Message ID | da713212dcf4c096b04b831026df3dd0bc5e99d1.1517920010.git.michal.simek@xilinx.com |
---|---|
State | Changes Requested |
Delegated to: | Tom Rini |
Headers | show |
Series | [U-Boot,RFC] Kconfig: cmd: Make networking command dependent on NET | expand |
On Tue, Feb 06, 2018 at 01:26:52PM +0100, Michal Simek wrote: > Enable networking command only when NET is enabled. > And remove selecting NET for CMD_NET > > Signed-off-by: Michal Simek <michal.simek@xilinx.com> I tried taking this, but as-is, now we have a big problem due to "everyone" getting NET selected here. Making NET default y fixes _most_ things except for the following boards that blow up related to EFI loader: aarch64: + thunderx_88xx arm: + at91sam9rlek_mmc warp zynq_zc770_xm011_x16 mx23evk bcm23550_w1d s5pc210_univer sal cairo topic_miamilite omap3_pandora topic_miami origen bcm28155_ap tricorder zynq_zc770_xm0 11 tricorder_flash s5p_goni trats aspenite at91sam9rlek_nandflash trats2 at91sam9rlek_dataflash At this point, I'm going to defer this back to you. I think the concept of the patch is right but it needs a bit more work. Thanks!
Hi Tom, On 14.2.2018 21:41, Tom Rini wrote: > On Tue, Feb 06, 2018 at 01:26:52PM +0100, Michal Simek wrote: > >> Enable networking command only when NET is enabled. >> And remove selecting NET for CMD_NET >> >> Signed-off-by: Michal Simek <michal.simek@xilinx.com> > > I tried taking this, but as-is, now we have a big problem due to > "everyone" getting NET selected here. Making NET default y fixes _most_ > things except for the following boards that blow up related to EFI > loader: > aarch64: + thunderx_88xx > arm: + at91sam9rlek_mmc warp zynq_zc770_xm011_x16 mx23evk bcm23550_w1d s5pc210_univer > sal cairo topic_miamilite omap3_pandora topic_miami origen bcm28155_ap tricorder zynq_zc770_xm0 > 11 tricorder_flash s5p_goni trats aspenite at91sam9rlek_nandflash trats2 at91sam9rlek_dataflash > > At this point, I'm going to defer this back to you. I think the concept > of the patch is right but it needs a bit more work. Thanks! > I have sent v2 of this with fixing above boards. I expect you have nice system for building. If there is any issue please let me know I will look at that boards. Thanks, Michal
diff --git a/cmd/Kconfig b/cmd/Kconfig index 790c40456de5..aedfd03c68ef 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -970,9 +970,10 @@ endmenu menu "Network commands" +if NET + config CMD_NET bool "bootp, tftpboot" - select NET default y help Network commands. @@ -1049,6 +1050,8 @@ config CMD_ETHSW operations such as enabling / disabling a port and viewing/maintaining the filtering database (FDB) +endif + endmenu menu "Misc commands"
Enable networking command only when NET is enabled. And remove selecting NET for CMD_NET Signed-off-by: Michal Simek <michal.simek@xilinx.com> --- There is a huge impact on defconfigs because of select NET. But this change makes sense to do but it just needs to be syncup properly. Do we have a tool for this kind of change? --- cmd/Kconfig | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)