[U-Boot] [PATCH v2] lsxl: unset ncip for rescue mode

Michael Walle michael at walle.cc
Thu Oct 4 18:54:25 CEST 2012


Instead of using the serverip we get from the DHCP server, implicitly use
the broadcast address, which is automatically set when no ncip environment
variable is set. That way it isn't necessary to use a special DHCP
configuration to set the netconsole peer.

Signed-off-by: Michael Walle <michael at walle.cc>
Cc: Prafulla Wadaskar <prafulla at marvell.com>
---

The original patch subject was "lsxl: set ncip to broadcast address".

v2:
 - don't hardcode IP address
 - adjust comments in lsxl.c

 board/buffalo/lsxl/lsxl.c |    5 ++---
 include/configs/lsxl.h    |    2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/board/buffalo/lsxl/lsxl.c b/board/buffalo/lsxl/lsxl.c
index b3f31d6..8756abf 100644
--- a/board/buffalo/lsxl/lsxl.c
+++ b/board/buffalo/lsxl/lsxl.c
@@ -49,9 +49,8 @@
  * you can do this only with a working network connection. Therefore, a random
  * ethernet address is generated if none is set and a DHCP request is sent.
  * After a successful DHCP response is received, the network settings are
- * configured and the ncip parameter is set to the serverip. Eg. for a working
- * resuce mode, you should set 'next-server' to the host where the netconsole
- * client is started.
+ * configured and the ncip is unset. Therefore, all netconsole packets are
+ * broadcasted.
  * Additionally, the bootsource is set to 'rescue'.
  */
 
diff --git a/include/configs/lsxl.h b/include/configs/lsxl.h
index 0db559c..86c4bce 100644
--- a/include/configs/lsxl.h
+++ b/include/configs/lsxl.h
@@ -146,7 +146,7 @@
 	"config_nc_dhcp=setenv autoload_old ${autoload}; "		\
 		"setenv autoload no "					\
 		"&& bootp "						\
-		"&& setenv ncip ${serverip} "				\
+		"&& setenv ncip "					\
 		"&& setenv autoload ${autoload_old}; "			\
 		"setenv autoload_old\0"					\
 	"standard_env=setenv ipaddr; setenv netmask; setenv serverip; "	\
-- 
1.7.2.5



More information about the U-Boot mailing list