[U-Boot] [PATCH v1 24/26] colibri_imx6: switch to zimage

Marcel Ziswiler
2019-02-08T21:48:13Z

Hi Lukasz

On Fri, 2019-02-08 at 22:42 +0100, Lukasz Majewski wrote:
> Hi Marcel,
>
> > From: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
> >
> > Switch to the generic compressed Kernel image type (zImage) instead
> > of
> > the U-Boot specific uImage format.
> >
>
> I would recommend switching to use fitImage instead of zImage.

Thanks, we are, of course, aware of fitImage and we do use it e.g. for
the Toradex Easy Installer. However, while our former BSPs still booted
from uImage our current BSPs now boot from zImage. In the future, we
plan to fully integrate distroboot which basically makes this choice
here obsolete anyway.

> You can look on display5 board for a reference.
>
> > While at it also fix device tree naming for MMC DFU.
> >
> > Signed-off-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
> > Acked-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> >
> > ---
> >
> > include/configs/colibri_imx6.h | 22 +++++++++++-----------
> > 1 file changed, 11 insertions(+), 11 deletions(-)
> >
> > diff --git a/include/configs/colibri_imx6.h
> > b/include/configs/colibri_imx6.h index 6e056aab31..a6a823ee1f
> > 100644
> > --- a/include/configs/colibri_imx6.h
> > +++ b/include/configs/colibri_imx6.h
> > @@ -108,17 +108,17 @@
> > #include <config_distro_bootcmd.h>
> > #undef BOOTENV_RUN_NET_USB_START
> > #define BOOTENV_RUN_NET_USB_START ""
> > -#else
> > +#else /* CONFIG_SPL_BUILD */
> > #define BOOTENV
> > -#endif
> > +#endif /* CONFIG_SPL_BUILD */
> >
> > #define DFU_ALT_EMMC_INFO \
> > "u-boot.imx raw 0x2 0x3ff mmcpart 0;" \
> > "boot part 0 1;" \
> > "rootfs part 0 2;" \
> > - "uImage fat 0 1;" \
> > - "imx6q-colibri-eval-v3.dtb fat 0 1;" \
> > - "imx6q-colibri-cam-eval-v3.dtb fat 0 1"
> > + "zImage fat 0 1;" \
> > + "imx6dl-colibri-eval-v3.dtb fat 0 1;" \
> > + "imx6dl-colibri-cam-eval-v3.dtb fat 0 1"
> >
> > #define EMMC_BOOTCMD \
> > "emmcargs=ip=off root=/dev/mmcblk0p2 rw,noatime
> > rootfstype=ext4 " \ @@ -128,7 +128,7 @@
> > "${vidargs}; echo Booting from internal eMMC
> > chip...; " \ "run emmcdtbload; load mmc 0:1 ${kernel_addr_r} "
> > \ "${boot_file} && run fdt_fixup && " \
> > - "bootm ${kernel_addr_r} ${dtbparam}\0" \
> > + "bootz ${kernel_addr_r} ${dtbparam}\0" \
> > "emmcdtbload=setenv dtbparam; load mmc 0:1 ${fdt_addr_r} " \
> > "${fdt_file} && setenv dtbparam \" - ${fdt_addr_r}\"
> > && true\0"
> > @@ -148,11 +148,11 @@
> > "setenv bootargs ${defargs} ${nfsargs} ${setupargs}
> > " \ "${vidargs}; echo Booting via DHCP/TFTP/NFS...; " \
> > "run nfsdtbload; dhcp ${kernel_addr_r} " \
> > - "&& run fdt_fixup && bootm ${kernel_addr_r}
> > ${dtbparam}\0" \
> > + "&& run fdt_fixup && bootz ${kernel_addr_r}
> > ${dtbparam}\0" \ "nfsdtbload=setenv dtbparam; tftp ${fdt_addr_r}
> > ${fdt_file} " \ "&& setenv dtbparam \" - ${fdt_addr_r}\" && true\0"
> >
> > -#define SD_BOOTCMD \
> > +#define SD_BOOTCMD \
> > "sdargs=ip=off root=/dev/mmcblk1p2 rw,noatime
> > rootfstype=ext4 " \ "rootwait\0" \
> > "sdboot=run setup; " \
> > @@ -160,7 +160,7 @@
> > "${vidargs}; echo Booting from SD card; " \
> > "run sddtbload; load mmc 1:1 ${kernel_addr_r} " \
> > "${boot_file} && run fdt_fixup && " \
> > - "bootm ${kernel_addr_r} ${dtbparam}\0" \
> > + "bootz ${kernel_addr_r} ${dtbparam}\0" \
> > "sddtbload=setenv dtbparam; load mmc 1:1 ${fdt_addr_r} " \
> > "${fdt_file} && setenv dtbparam \" - ${fdt_addr_r}\"
> > && true\0"
> > @@ -171,7 +171,7 @@
> > "${usbargs} ${vidargs}; echo Booting from USB
> > stick...; " \ "usb start && run usbdtbload; load usb 0:1
> > ${kernel_addr_r} " \ "${boot_file} && run fdt_fixup && " \
> > - "bootm ${kernel_addr_r} ${dtbparam}\0" \
> > + "bootz ${kernel_addr_r} ${dtbparam}\0" \
> > "usbdtbload=setenv dtbparam; load usb 0:1 ${fdt_addr_r} " \
> > "${fdt_file} && setenv dtbparam \" - ${fdt_addr_r}\"
> > && true\0"
> > @@ -182,7 +182,7 @@
> > "run distro_bootcmd ; " \
> > "usb start ; " \
> > "setenv stdout serial,vga ; setenv stdin
> > serial,usbkbd\0" \
> > - "boot_file=uImage\0" \
> > + "boot_file=zImage\0" \
> > "console=ttymxc0\0" \
> > "defargs=enable_wait_mode=off
> > galcore.contiguousSize=50331648\0" \ "dfu_alt_info="
> > DFU_ALT_EMMC_INFO "\0" \
>
>
>
> Best regards,
>
> Lukasz Majewski
>
> --
>
> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> lukma at denx.de

Best regards - Mit freundlichen GrÃ¼ssen - Meilleures salutations

Marcel Ziswiler
Platform Manager Embedded Linux

Toradex AG
Altsagenstrasse 5 | 6048 Horw/Luzern | Switzerland | T: +41 41 500 48 00
(main line) | Direct: +41 41 500 48 10