[PATCHv1 1/3] mp: imx9: add cpu command support

Z.Q. Hou zhiqiang.hou at nxp.com
Mon Jul 22 06:49:34 CEST 2024


Hi Simon,

> -----Original Message-----
> From: Simon Glass <sjg at chromium.org>
> Sent: Friday, July 19, 2024 11:05 PM
> To: Z.Q. Hou <zhiqiang.hou at nxp.com>
> Cc: u-boot at lists.denx.de; sbabic at denx.de; festevam at gmail.com;
> dl-uboot-imx <uboot-imx at nxp.com>; trini at konsulko.com; Peng Fan
> <peng.fan at nxp.com>; Tao Yang <Tao.Yang1 at nxp.com>
> Subject: Re: [PATCHv1 1/3] mp: imx9: add cpu command support
> 
> Hi Zhiqiang,
> 
> On Fri, 19 Jul 2024 at 12:28, Zhiqiang Hou <Zhiqiang.Hou at nxp.com> wrote:
> >
> > From: Tao Yang <Tao.Yang1 at nxp.com>
> >
> > Implement the cpu command to kick cpu core to run barematel or RTOS
> > applications.
> >
> > Signed-off-by: Tao Yang <Tao.Yang1 at nxp.com>
> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> > ---
> >  arch/arm/mach-imx/imx9/Makefile |  4 +-
> >  arch/arm/mach-imx/imx9/mp.c     | 81
> +++++++++++++++++++++++++++++++++
> >  2 files changed, 84 insertions(+), 1 deletion(-)  create mode 100644
> > arch/arm/mach-imx/imx9/mp.c
> 
> This seems to be adding more code to a deprecated API. It should use
> UCLASS_CPU and driver model. See the comment at the top of cpu_func.h

Thanks for the guide! I saw the cmd/cpu.c provided a new 'cpu' command based on UCLASS_CPU, I'll add a 'release' subcommand.

Thanks,
Zhiqiang


More information about the U-Boot mailing list