diff mbox series

[U-Boot,v2,05/11] net: Improve menu options and help for BOOTP options

Message ID 1523651200-31456-6-git-send-email-joe.hershberger@ni.com
State Accepted
Commit 8df69d9
Delegated to: Joe Hershberger
Headers show
Series net: Clean up the menus and dependencies among commands and options | expand

Commit Message

Joe Hershberger April 13, 2018, 8:26 p.m. UTC 
The options were pretty unhelpful, so improve them some.

Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
Reviewed-by: Chris Packham <judge.packham@gmail.com>
---

Changes in v2: None

 cmd/Kconfig | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

Comments

Joe Hershberger April 15, 2018, 4:16 a.m. UTC | #1 
Hi Joe,

https://patchwork.ozlabs.org/patch/898099/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git

Thanks!
-Joe
diff mbox series

Patch

diff --git a/cmd/Kconfig b/cmd/Kconfig
index a89d5ac..6664379 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -1042,23 +1042,32 @@  config CMD_DHCP
 	  Boot image via network using DHCP/TFTP protocol
 
 config BOOTP_BOOTPATH
-	bool "Enable BOOTP BOOTPATH"
+	bool "Request & store 'rootpath' from BOOTP/DHCP server"
 	depends on CMD_BOOTP
+	help
+	  Even though the config is called BOOTP_BOOTPATH, it stores the
+	  path in the variable 'rootpath'.
 
 config BOOTP_DNS
-	bool "Enable bootp DNS"
+	bool "Request & store 'dnsip' from BOOTP/DHCP server"
 	depends on CMD_BOOTP
+	help
+	  The primary DNS server is stored as 'dnsip'. If two servers are
+	  returned, you must set BOOTP_DNS2 to store that second server IP
+	  also.
 
 config BOOTP_GATEWAY
-	bool "Enable BOOTP gateway"
+	bool "Request & store 'gatewayip' from BOOTP/DHCP server"
 	depends on CMD_BOOTP
 
 config BOOTP_HOSTNAME
-	bool "Enable BOOTP hostname"
+	bool "Request & store 'hostname' from BOOTP/DHCP server"
 	depends on CMD_BOOTP
+	help
+	  The name may or may not be qualified with the local domain name.
 
 config BOOTP_SUBNETMASK
-	bool "Enable BOOTP subnetmask"
+	bool "Request & store 'netmask' from BOOTP/DHCP server"
 	depends on CMD_BOOTP
 
 config BOOTP_PXE