[EXT] Re: [PATCH v2 2/2] board: imx8mp: add boot.cmd for distro boot on iMX8MP

[Dennis Gilmore]

boot.scr is only in distro boot because prior to standardising in
extlinux.conf or grub vie efi, distros provided boot.scr files that
were tailored for each board with  detailed staps and memory locations
to load the kernel, dtb, and initramfs. the scripts were wrapped by
mkimage and at this point I would consider it to be a legacy thing
that possibly should be considered for deprecation. grub, extlinux, or
boot.scr should all be provided by the distro. u-boot just nees to
load whats provided.

Dennis

On Fri, Dec 18, 2020 at 5:11 AM Alice Guo (OSS) <alice.guo at oss.nxp.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Fabio Estevam <festevam at gmail.com>
> > Sent: 2020年12月18日 19:03
> > To: Alice Guo (OSS) <alice.guo at oss.nxp.com>
> > Cc: Stefano Babic <sbabic at denx.de>; Peng Fan <peng.fan at nxp.com>;
> > dl-uboot-imx <uboot-imx at nxp.com>; Ye Li <ye.li at nxp.com>; U-Boot-Denx
> > <u-boot at lists.denx.de>; Alice Guo <alice.guo at nxp.com>; Otavio Salvador
> > <otavio at ossystems.com.br>; andrey.zhizhikin at leica-geosystems.com
> > Subject: [EXT] Re: [PATCH v2 2/2] board: imx8mp: add boot.cmd for distro boot
> > on iMX8MP
> >
> > Caution: EXT Email
> >
> > Hi Alice,
> >
> > On Fri, Dec 18, 2020 at 4:20 AM Alice Guo (OSS) <alice.guo at oss.nxp.com>
> > wrote:
> >
> > > +++ b/board/freescale/imx8mp_evk/boot.cmd
> > > @@ -0,0 +1,25 @@
> > > +setenv bootargs console=${console} root=${mmcroot};
> > > +
> > > +for boot_target in ${boot_targets};
> >
> > This is not how distro boot support works.
> >
> > boot.scr or extlinux.conf scripts should be provided by the distro itself (Yocto,
> > Buildroot, Debian, etc), and not from U-Boot.
> >
> > Each distro has its own partitioning mechanism and may store the kernel, dtb in
> > different locations.
>
> Hi,
> Can you tell me where I can see more details about boot.scr? I didn't see that " boot.scr or extlinux.conf scripts should be provided by the distro itself".
> Thank you.
>
> Alice