Patchwork [U-Boot,2/2] eXMeritus HWW-1U-1A: Minor environment variable tweaks

login
register
mail settings
Submitter Kyle Moffett
Date Dec. 16, 2011, 3:26 a.m.
Message ID <1324006013-24494-2-git-send-email-Kyle.D.Moffett@boeing.com>
Download mbox | patch
Permalink /patch/131776/
State Accepted
Commit 7985f7e3ff51ad8024d69a209893f68a2a5d3983
Headers show

Comments

Kyle Moffett - Dec. 16, 2011, 3:26 a.m.
Most of the ethernet connections are internal links with specialized
hardware and are not useful for "dhcp" or general-purpose networking;
U-Boot should not be cycling through them.  Force the primary external
network interface in "ethprime" and disable the interface cycling with
"ethrotate=no".

Additionally, the environment variable "preboot" has its own config
option and means something entirely different from what the HWW-1U-1A
variable was intended for.  Rename the board variable to "setbootargs"
to avoid potential confusion.

Finally, fix an incorrect address for the kernel in FLASH memory.

Signed-off-by: Kyle Moffett <Kyle.D.Moffett@boeing.com>
Cc: Andy Fleming <afleming@gmail.com>
Cc: Kumar Gala <kumar.gala@freescale.com>
---
 include/configs/HWW1U1A.h |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)
Kumar Gala - Jan. 13, 2012, 6:55 p.m.
On Dec 15, 2011, at 9:26 PM, Kyle Moffett wrote:

> Most of the ethernet connections are internal links with specialized
> hardware and are not useful for "dhcp" or general-purpose networking;
> U-Boot should not be cycling through them.  Force the primary external
> network interface in "ethprime" and disable the interface cycling with
> "ethrotate=no".
> 
> Additionally, the environment variable "preboot" has its own config
> option and means something entirely different from what the HWW-1U-1A
> variable was intended for.  Rename the board variable to "setbootargs"
> to avoid potential confusion.
> 
> Finally, fix an incorrect address for the kernel in FLASH memory.
> 
> Signed-off-by: Kyle Moffett <Kyle.D.Moffett@boeing.com>
> Cc: Andy Fleming <afleming@gmail.com>
> Cc: Kumar Gala <kumar.gala@freescale.com>
> ---
> include/configs/HWW1U1A.h |    7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)

applied to 85xx

- k

Patch

diff --git a/include/configs/HWW1U1A.h b/include/configs/HWW1U1A.h
index 123fc57..01597b9 100644
--- a/include/configs/HWW1U1A.h
+++ b/include/configs/HWW1U1A.h
@@ -460,11 +460,14 @@  const char *hww1u1a_get_ps1(void);
 
 /* Extra environment parameters */
 #define CONFIG_EXTRA_ENV_SETTINGS					\
-	"preboot=setenv bootargs \"${bootargs} "CONFIG_BOOTARGS_DYNAMIC"\"\0" \
+	"ethprime=e1000#0\0"						\
+	"ethrotate=no\0"						\
+	"setbootargs=setenv bootargs "					\
+			"\"${bootargs} "CONFIG_BOOTARGS_DYNAMIC"\"\0"	\
 	"perf_mode=performance\0"					\
 	"hwconfig="	"fsl_ddr:ctlr_intlv=bank,bank_intlv=cs0_cs1;"	\
 			"usb1:dr_mode=host,phy_type=ulpi\0"		\
-	"flkernel=0xe8020000\0"						\
+	"flkernel=0xe8000000\0"						\
 	"flinitramfs=0xe8800000\0"					\
 	"fldevicetree=0xeff20000\0"					\
 	"flbootm=bootm ${flkernel} ${flinitramfs} ${fldevicetree}\0"	\