cmd: Kconfig: Fix submenu for network commands
authorPaul Barker <paul.barker.ct@bp.renesas.com>
Fri, 7 Mar 2025 15:15:29 +0000 (15:15 +0000)
committerJerome Forissier <jerome.forissier@linaro.org>
Mon, 31 Mar 2025 07:46:25 +0000 (09:46 +0200)
The Kconfig parser seems to get confused by the current if conditions
following CMD_NET and displays all network command options directly in
the "Command line interface" menu instead of in a "Network commands"
submenu.

To help out Kconfig we can simplify the if conditions, so that the
definition of CMD_NET is followed immediately by an if/endif block that
contains all network command options. We can also remove nested checks
for CMD_NET or (NET || NET_LWIP).

Fixes: 98ad145db61a ("net: lwip: add DHCP support and dhcp commmand")
Signed-off-by: Paul Barker <paul.barker.ct@bp.renesas.com>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
cmd/Kconfig

index 642cc11..ce8393c 100644 (file)
@@ -1833,12 +1833,10 @@ menuconfig CMD_NET
        bool "Network commands"
        default y
 
-endif
+if CMD_NET
 
 if NET
 
-if CMD_NET
-
 config CMD_BOOTP
        bool "bootp"
        default y
@@ -2079,7 +2077,6 @@ config IPV6_ROUTER_DISCOVERY
        help
          Will automatically perform router solicitation on first IPv6
          network operation
-endif  # if CMD_NET
 
 config CMD_ETHSW
        bool "ethsw"
@@ -2096,10 +2093,6 @@ config CMD_WOL
 
 endif  # if NET
 
-if NET || NET_LWIP
-
-if CMD_NET
-
 config CMD_DHCP
        bool "dhcp"
        select PROT_DHCP_LWIP if NET_LWIP
@@ -2175,8 +2168,6 @@ config WGET_HTTPS
        help
          Enable TLS over http for wget.
 
-endif  # if CMD_NET
-
 config CMD_PXE
        bool "pxe"
        select PXE_UTILS
@@ -2184,7 +2175,9 @@ config CMD_PXE
        help
          Boot image via network using PXE protocol
 
-endif  # if NET || NET_LWIP
+endif  # if CMD_NET
+
+endif # NET || NET_LWIP
 
 menu "Misc commands"