Hi Fabio,

Thanks a lot for your comments!

> -----Original Message-----
> From: Fabio Estevam <feste...@gmail.com>
> Sent: Friday, July 19, 2024 7:33 PM
> To: Z.Q. Hou <zhiqiang....@nxp.com>
> Cc: u-boot@lists.denx.de; sba...@denx.de; dl-uboot-imx
> <uboot-...@nxp.com>; tr...@konsulko.com; Peng Fan <peng....@nxp.com>;
> Yi Zhao <yi.z...@nxp.com>; Jiafei Pan <jiafei....@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....@nxp.com> wrote:
> >
> > From: Hou Zhiqiang <zhiqiang....@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

Reply via email to