Hi Simon,

> -----Original Message-----
> From: Simon Glass <[email protected]>
> Sent: Friday, July 19, 2024 11:05 PM
> To: Z.Q. Hou <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> dl-uboot-imx <[email protected]>; [email protected]; Peng Fan
> <[email protected]>; Tao Yang <[email protected]>
> Subject: Re: [PATCHv1 1/3] mp: imx9: add cpu command support
> 
> Hi Zhiqiang,
> 
> On Fri, 19 Jul 2024 at 12:28, Zhiqiang Hou <[email protected]> wrote:
> >
> > From: Tao Yang <[email protected]>
> >
> > Implement the cpu command to kick cpu core to run barematel or RTOS
> > applications.
> >
> > Signed-off-by: Tao Yang <[email protected]>
> > Signed-off-by: Hou Zhiqiang <[email protected]>
> > ---
> >  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

Reply via email to