[PATCH 00/12] imx9: switch to bootstd for i.MX93/1 EVK/QSB and i.MX8M EVK

[Peng Fan]

Hi Simon,

> Subject: Re: [PATCH 00/12] imx9: switch to bootstd for i.MX93/1
> EVK/QSB and i.MX8M EVK
> 
> Hi Peng,
> 
> On Sun, 5 Jan 2025 at 18:14, Peng Fan <peng.fan at nxp.com> wrote:
> >
> > > Subject: Re: [PATCH 00/12] imx9: switch to bootstd for i.MX93/1
> > > EVK/QSB and i.MX8M EVK
> > >
> > > Hi Peng,
> > >
> > > On Sun, 5 Jan 2025 at 17:27, Peng Fan <peng.fan at oss.nxp.com>
> wrote:
> > > >
> > > > Hi Simon,
> > > >
> > > > On Sun, Jan 05, 2025 at 08:30:55AM +1300, Simon Glass wrote:
> > > > >Hi Peng,
> > > > >
> > > > >On Sat, 4 Jan 2025 at 16:19, Peng Fan (OSS)
> > > <peng.fan at oss.nxp.com> wrote:
> > > > >>
> > > > >> Switch to bootstd for NXP i.MX9/8M boards with a
> bsp_bootcmd
> > > as fallback.
> > > > >>
> > > > [...]
> > > > >>
> > > > >
> > > > >This also switches to a text environment, great!
> > > > >
> > > > >For bsp_bootcmd is it possible to create a bootmeth for your use
> > > case?
> > > > >If you would like to take a look (separate from this series), I
> > > > >could help with any issues that come up.
> > > >
> > > > I just give a quick look of bootmeth, seems needs extlinux or
> > > > bootscript (boot.scr). But both are used on NXP i.MX release as of
> > > > now. In NXP releases, we put all script in the U-Boot ENV.
> > > >
> > > > Do you have suggestions how to move to support bootmeth.
> > >
> > > I'm not entirely sure what you are trying to do with the bootcmd,
> >
> > With a single u-boot, we want to support Fedora/SUSE/etc and NXP
> Yocto
> > releases.
> >
> > So add a bsp_bootcmd as fallback to run NXP Yocto releases.
> > NXP Yocto releases not have extlinux.cfg, nor boot.scr, we just have
> > simple uboot hush command in bootcmd, simplied saying:
> > run loadimage; run loadfdt; run mmcargs; booti ${loadaddr} -
> > ${fdtaddr}
> >
> > > you could write your own bootmeth, e.g. using common code from
> the
> > > others. For example, bootmeth_script.c shows how to run a script
> and
> > > bootmeth_extlinux.c shows how to run an extlinux file. You could
> > > then create C code to perform your 'bsp' function, e.g. a new
> > > bootmeth_bsp.c.
> >
> > I will give a look, but this seems we need put the "run loadimage and
> etc"
> > in dtb and needs bootmeth_imx.c to support it.
> 
> Note that my suggestion here is as a follow-on to you current series,
> which I believe can be applied as is.
> 

I see.

> Anyway, as a starting point, yes you can use run_command() lots of
> times in the bootmeth to run the things you need. From there you can
> potentially change some commands to direct calls into the U-Boot
> functionality. The idea is to be able to locate the bootflow as a separate
> thing from booting it. I am quite happy to help as well.

It would be great if you could help here. Appreciate in advance.

Thanks,
Peng.

> 
> Regards,
> SImon