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