[U-Boot] [PATCH 1/2] mx28evk: extend default environment

From: Otavio Salvador otavio@ossystems.com.br
The environment has been based on mx53loco but keeping the possibility to easy change the default console device as Freescale and mainline kernels differ on the device name.
Signed-off-by: Otavio Salvador otavio@ossystems.com.br Cc: Fabio Estevam fabio.estevam@freescale.com Cc: Stefano Babic sbabic@denx.de --- include/configs/mx28evk.h | 44 ++++++++++++++++++++++++++++++++++++++------ 1 file changed, 38 insertions(+), 6 deletions(-)
diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h index 02f3366..b165285 100644 --- a/include/configs/mx28evk.h +++ b/include/configs/mx28evk.h @@ -222,7 +222,7 @@ #define CONFIG_SETUP_MEMORY_TAGS #define CONFIG_BOOTDELAY 3 #define CONFIG_BOOTFILE "uImage" -#define CONFIG_BOOTCOMMAND "run bootcmd_net" + #define CONFIG_LOADADDR 0x42000000 #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
@@ -230,13 +230,45 @@ * Extra Environments */ #define CONFIG_EXTRA_ENV_SETTINGS \ - "console_fsl=console=ttyAM0" \ - "console_mainline=console=ttyAMA0" \ - "netargs=setenv bootargs console=${console_mainline}" \ + "script=boot.scr\0" \ + "uimage=uImage\0" \ + "console_fsl=ttyAM0\0" \ + "console_mainline=ttyAMA0\0" \ + "console=${console_mainline}\0" \ + "mmcdev=0\0" \ + "mmcpart=2\0" \ + "mmcroot=/dev/mmcblk0p3 rw\0" \ + "mmcrootfstype=ext3 rootwait\0" \ + "mmcargs=setenv bootargs console=${console},${baudrate} " \ + "root=${mmcroot} " \ + "rootfstype=${mmcrootfstype}\0" \ + "loadbootscript=" \ + "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \ + "bootscript=echo Running bootscript from mmc ...; " \ + "source\0" \ + "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \ + "mmcboot=echo Booting from mmc ...; " \ + "run mmcargs; " \ + "bootm\0" \ + "netargs=setenv bootargs console=${console},${baudrate} " \ "root=/dev/nfs " \ - "ip=dhcp nfsroot=${serverip}:${nfsroot}\0" \ - "bootcmd_net=echo Booting from net ...; " \ + "ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \ + "netboot=echo Booting from net ...; " \ "run netargs; " \ "dhcp ${uimage}; bootm\0" \ +/* + * Default Boot command + */ +#define CONFIG_BOOTCOMMAND \ + "if mmc rescan ${mmcdev}; then " \ + "if run loadbootscript; then " \ + "run bootscript; " \ + "else " \ + "if run loaduimage; then " \ + "run mmcboot; " \ + "else run netboot; " \ + "fi; " \ + "fi; " \ + "else run netboot; fi"
#endif /* __CONFIG_H */

From: Otavio Salvador otavio@ossystems.com.br
The macro mistakenly referred to 32bit struct instead of 8bit one.
Signed-off-by: Otavio Salvador otavio@ossystems.com.br Cc: Fabio Estevam fabio.estevam@freescale.com Cc: Marek Vasut marex@denx.de --- arch/arm/include/asm/arch-mx28/regs-common.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/include/asm/arch-mx28/regs-common.h b/arch/arm/include/asm/arch-mx28/regs-common.h index 94b512d..d2e1953 100644 --- a/arch/arm/include/asm/arch-mx28/regs-common.h +++ b/arch/arm/include/asm/arch-mx28/regs-common.h @@ -70,7 +70,7 @@ struct mx28_register_32 { #define mx28_reg_8(name) \ union { \ struct { __mx28_reg_8(name) }; \ - struct mx28_register_32 name##_reg; \ + struct mx28_register_8 name##_reg; \ };
#define mx28_reg_32(name) \

Dear Fabio Estevam,
From: Otavio Salvador otavio@ossystems.com.br
The environment has been based on mx53loco but keeping the possibility to easy change the default console device as Freescale and mainline kernels differ on the device name.
Signed-off-by: Otavio Salvador otavio@ossystems.com.br Cc: Fabio Estevam fabio.estevam@freescale.com Cc: Stefano Babic sbabic@denx.de
Can we possibly unify it with what's already in m28evk? Also, it'd be cool if we had some default -- very generic -- env. Can we go in that direction?
include/configs/mx28evk.h | 44 ++++++++++++++++++++++++++++++++++++++------ 1 file changed, 38 insertions(+), 6 deletions(-)
diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h index 02f3366..b165285 100644 --- a/include/configs/mx28evk.h +++ b/include/configs/mx28evk.h @@ -222,7 +222,7 @@ #define CONFIG_SETUP_MEMORY_TAGS #define CONFIG_BOOTDELAY 3 #define CONFIG_BOOTFILE "uImage" -#define CONFIG_BOOTCOMMAND "run bootcmd_net"
#define CONFIG_LOADADDR 0x42000000 #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
@@ -230,13 +230,45 @@
- Extra Environments
*/ #define CONFIG_EXTRA_ENV_SETTINGS \
- "console_fsl=console=ttyAM0" \
- "console_mainline=console=ttyAMA0" \
- "netargs=setenv bootargs console=${console_mainline}" \
- "script=boot.scr\0" \
- "uimage=uImage\0" \
- "console_fsl=ttyAM0\0" \
- "console_mainline=ttyAMA0\0" \
- "console=${console_mainline}\0" \
- "mmcdev=0\0" \
- "mmcpart=2\0" \
- "mmcroot=/dev/mmcblk0p3 rw\0" \
- "mmcrootfstype=ext3 rootwait\0" \
- "mmcargs=setenv bootargs console=${console},${baudrate} " \
"root=${mmcroot} " \
"rootfstype=${mmcrootfstype}\0" \
- "loadbootscript=" \
"fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \
- "bootscript=echo Running bootscript from mmc ...; " \
"source\0" \
- "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \
- "mmcboot=echo Booting from mmc ...; " \
"run mmcargs; " \
"bootm\0" \
- "netargs=setenv bootargs console=${console},${baudrate} " \ "root=/dev/nfs " \
"ip=dhcp nfsroot=${serverip}:${nfsroot}\0" \
- "bootcmd_net=echo Booting from net ...; " \
"ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp\0" \
- "netboot=echo Booting from net ...; " \ "run netargs; " \ "dhcp ${uimage}; bootm\0" \
+/*
- Default Boot command
- */
+#define CONFIG_BOOTCOMMAND \
- "if mmc rescan ${mmcdev}; then " \
"if run loadbootscript; then " \
"run bootscript; " \
"else " \
"if run loaduimage; then " \
"run mmcboot; " \
"else run netboot; " \
"fi; " \
"fi; " \
- "else run netboot; fi"
#endif /* __CONFIG_H */
Best regards, Marek Vasut

On Thu, May 10, 2012 at 2:23 PM, Marek Vasut marex@denx.de wrote:
Dear Fabio Estevam,
From: Otavio Salvador otavio@ossystems.com.br
The environment has been based on mx53loco but keeping the possibility to easy change the default console device as Freescale and mainline kernels differ on the device name.
Signed-off-by: Otavio Salvador otavio@ossystems.com.br Cc: Fabio Estevam fabio.estevam@freescale.com Cc: Stefano Babic sbabic@denx.de
Can we possibly unify it with what's already in m28evk? Also, it'd be cool if we had some default -- very generic -- env. Can we go in that direction?
I have prepared a set of commits in this direction. Please take a look at:
https://github.com/otavio/u-boot/commit/5745dfbfa1bd69b23c73dc3d2b2c2932c338... https://github.com/otavio/u-boot/commit/d8b1ea575177038f6e37f28563f00c926001...
Is that what you expect? I want to put sdcard based update commands before sending it for merging.

On Fri, May 11, 2012 at 10:54 AM, Otavio Salvador otavio@ossystems.com.br wrote:
Can we possibly unify it with what's already in m28evk? Also, it'd be cool if we had some default -- very generic -- env. Can we go in that direction?
I have prepared a set of commits in this direction. Please take a look at:
https://github.com/otavio/u-boot/commit/5745dfbfa1bd69b23c73dc3d2b2c2932c338... https://github.com/otavio/u-boot/commit/d8b1ea575177038f6e37f28563f00c926001...
Is that what you expect? I want to put sdcard based update commands before sending it for merging.
Fabio has pointed a example for me of 'magnesium' board config. Is it what you expect? it does make sense for me to make a common header file for both boards and others that might come later.

Dear Otavio Salvador,
On Fri, May 11, 2012 at 10:54 AM, Otavio Salvador
otavio@ossystems.com.br wrote:
Can we possibly unify it with what's already in m28evk? Also, it'd be cool if we had some default -- very generic -- env. Can we go in that direction?
I have prepared a set of commits in this direction. Please take a look at:
https://github.com/otavio/u-boot/commit/5745dfbfa1bd69b23c73dc3d2b2c2932c 338e750 https://github.com/otavio/u-boot/commit/d8b1ea575177038f6e37f28563f00c92 60016ac9
Is that what you expect? I want to put sdcard based update commands before sending it for merging.
Fabio has pointed a example for me of 'magnesium' board config. Is it what you expect? it does make sense for me to make a common header file for both boards and others that might come later.
Something like that, but you're missing update_sd_firmware ;-)
Best regards, Marek Vasut

On 11/05/2012 16:10, Otavio Salvador wrote:
On Fri, May 11, 2012 at 10:54 AM, Otavio Salvador otavio@ossystems.com.br wrote:
Can we possibly unify it with what's already in m28evk? Also, it'd be cool if we had some default -- very generic -- env. Can we go in that direction?
I have prepared a set of commits in this direction. Please take a look at:
https://github.com/otavio/u-boot/commit/5745dfbfa1bd69b23c73dc3d2b2c2932c338... https://github.com/otavio/u-boot/commit/d8b1ea575177038f6e37f28563f00c926001...
Is that what you expect? I want to put sdcard based update commands before sending it for merging.
Hi Otavio,
Fabio has pointed a example for me of 'magnesium' board config. Is it what you expect? it does make sense for me to make a common header file for both boards and others that might come later.
This is an example, but the imx27lite and the magnesium are very close. Of course, you could prepare some kind of mx28-common, that all boards can include.
Another example is amcc-common.h for PowerPC 4xx. Up now we have not a real "common" environment, that spreads among boards. Each board defines its own set - it is not bad to define some common scripts, that a board maintainer can use.
Best regards, Stefano Babic

On Fri, May 11, 2012 at 11:25 AM, Stefano Babic sbabic@denx.de wrote:
Another example is amcc-common.h for PowerPC 4xx. Up now we have not a real "common" environment, that spreads among boards. Each board defines its own set - it is not bad to define some common scripts, that a board maintainer can use.
Right; I will work on that.
participants (4)
-
Fabio Estevam
-
Marek Vasut
-
Otavio Salvador
-
Stefano Babic