[PATCHv1 1/2] mp: imx8m: add cpu command support

Z.Q. Hou zhiqiang.hou at nxp.com
Fri Jul 19 15:50:46 CEST 2024


Hi Fabio,

Thanks a lot for your comments!

> -----Original Message-----
> From: Fabio Estevam <festevam at gmail.com>
> Sent: Friday, July 19, 2024 7:33 PM
> To: Z.Q. Hou <zhiqiang.hou at nxp.com>
> Cc: u-boot at lists.denx.de; sbabic at denx.de; dl-uboot-imx
> <uboot-imx at nxp.com>; trini at konsulko.com; Peng Fan <peng.fan at nxp.com>;
> Yi Zhao <yi.zhao at nxp.com>; Jiafei Pan <jiafei.pan at nxp.com>
> Subject: Re: [PATCHv1 1/2] mp: imx8m: add cpu command support
> 
> On Fri, Jul 19, 2024 at 6:47 AM Zhiqiang Hou <Zhiqiang.Hou at nxp.com> wrote:
> >
> > From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >
> > Implement the cpu command to kick cpu core to run barematel or RTOS
> > applications.
> 
> s/barematel/baremetal

Will fix.

> 
> Please add documentation explaining how to run this command, how it was
> tested, etc.

This patch is to add the needed functions to enable cmd/mp.c, which is well known, so I don't think it's necessary to add an example to the commit log.

> 
> > --- /dev/null
> > +++ b/arch/arm/mach-imx/imx8m/mp.c
> 
> What does mp mean?

Short of 'multiprocessor', the same mean as cmd/mp.c.

Thanks,
Zhiqiang


More information about the U-Boot mailing list