[U-Boot] [PATCH] wandboard: Set default environment to use zImage

From: John Weber rjohnweber@gmail.com
Change the default environment to use zImage instead of uImage, because using zImage does not require a LOADADDR assignment when building the kernel to be consistent with the loadaddr environment variable in u-boot, decreasing the chance for error. This requires changes to the default environment to load a file named zImage instead of uImage, and to use the 'bootz' command instead of 'bootm' when booting the kernel.
The zImage works for FSL Linux's kernel fork versions 3.0.35, 3.10.9, and 3.10.17; this also works fine for mainline kernels. There is no reason to continue using uImage for default environment.
Signed-off-by: John Weber rjohnweber@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br --- include/configs/wandboard.h | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h index 5134b2e..3424124 100644 --- a/include/configs/wandboard.h +++ b/include/configs/wandboard.h @@ -115,7 +115,7 @@
#define CONFIG_EXTRA_ENV_SETTINGS \ "script=boot.scr\0" \ - "uimage=uImage\0" \ + "image=zImage\0" \ "console=ttymxc0\0" \ "splashpos=m,m\0" \ "fdt_high=0xffffffff\0" \ @@ -173,22 +173,22 @@ "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \ "bootscript=echo Running bootscript from mmc ...; " \ "source\0" \ - "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \ + "loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \ "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \ "mmcboot=echo Booting from mmc ...; " \ "run mmcargs; " \ "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \ "if run loadfdt; then " \ - "bootm ${loadaddr} - ${fdt_addr}; " \ + "bootz ${loadaddr} - ${fdt_addr}; " \ "else " \ "if test ${boot_fdt} = try; then " \ - "bootm; " \ + "bootz; " \ "else " \ "echo WARN: Cannot load the DT; " \ "fi; " \ "fi; " \ "else " \ - "bootm; " \ + "bootz; " \ "fi;\0" \ "netargs=setenv bootargs console=${console},${baudrate} " \ "root=/dev/nfs " \ @@ -200,19 +200,19 @@ "else " \ "setenv get_cmd tftp; " \ "fi; " \ - "${get_cmd} ${uimage}; " \ + "${get_cmd} ${image}; " \ "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \ "if ${get_cmd} ${fdt_addr} ${fdt_file}; then " \ - "bootm ${loadaddr} - ${fdt_addr}; " \ + "bootz ${loadaddr} - ${fdt_addr}; " \ "else " \ "if test ${boot_fdt} = try; then " \ - "bootm; " \ + "bootz; " \ "else " \ "echo WARN: Cannot load the DT; " \ "fi; " \ "fi; " \ "else " \ - "bootm; " \ + "bootz; " \ "fi;\0"
#define CONFIG_BOOTCOMMAND \ @@ -220,7 +220,7 @@ "if run loadbootscript; then " \ "run bootscript; " \ "else " \ - "if run loaduimage; then " \ + "if run loadimage; then " \ "run mmcboot; " \ "else run netboot; " \ "fi; " \

On Tue, Dec 31, 2013 at 6:44 PM, Otavio Salvador otavio@ossystems.com.br wrote:
From: John Weber rjohnweber@gmail.com
Change the default environment to use zImage instead of uImage, because using zImage does not require a LOADADDR assignment when building the kernel to be consistent with the loadaddr environment variable in u-boot, decreasing the chance for error. This requires changes to the default environment to load a file named zImage instead of uImage, and to use the 'bootz' command instead of 'bootm' when booting the kernel.
The zImage works for FSL Linux's kernel fork versions 3.0.35, 3.10.9, and 3.10.17; this also works fine for mainline kernels. There is no reason to continue using uImage for default environment.
Signed-off-by: John Weber rjohnweber@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br
Reviewed-by: Fabio Estevam fabio.estevam@freescale.com

Dear Otavio Salvador,
happy new year...
In message 1388522649-5316-1-git-send-email-otavio@ossystems.com.br you wrote:
Change the default environment to use zImage instead of uImage, because using zImage does not require a LOADADDR assignment when building the kernel to be consistent with the loadaddr environment variable in u-boot, decreasing the chance for error. This requires
This statement makes no sense. If you use the old legacy uImage format (which is deprecated, and should be replaced by FIT images), you specify the entry point address to the kernel, and the load address (i. e. where in RAM the image should be placed). These two parameters to themkimage commansd are registered in the image, and they are in no way related to any U-Boot environment variables, nor can any environment variable settings influence these.
Of course you should make sure that memory area where you dowenload the image does not overlap the area where it gets unpacked / copied to, but the same issues exist with zImage files.
Please fix the commit message to describe correctly what you want to acchieve.
Best regards,
Wolfgang Denk

On Tue, Dec 31, 2013 at 11:23 PM, Wolfgang Denk wd@denx.de wrote:
Dear Otavio Salvador,
happy new year...
You too! :-)
In message 1388522649-5316-1-git-send-email-otavio@ossystems.com.br you wrote:
Change the default environment to use zImage instead of uImage, because using zImage does not require a LOADADDR assignment when building the kernel to be consistent with the loadaddr environment variable in u-boot, decreasing the chance for error. This requires
This statement makes no sense. If you use the old legacy uImage format (which is deprecated, and should be replaced by FIT images), you specify the entry point address to the kernel, and the load address (i. e. where in RAM the image should be placed). These two parameters to themkimage commansd are registered in the image, and they are in no way related to any U-Boot environment variables, nor can any environment variable settings influence these.
Of course you should make sure that memory area where you dowenload the image does not overlap the area where it gets unpacked / copied to, but the same issues exist with zImage files.
Please fix the commit message to describe correctly what you want to acchieve.
Sent v2 fixing this; this is were more related to OE than U-Boot ;-)
participants (3)
-
Fabio Estevam
-
Otavio Salvador
-
Wolfgang Denk