[PATCHv2] clk: imx8m: register ARM A53 core clock

Z.Q. Hou zhiqiang.hou at nxp.com
Mon Jul 29 04:58:40 CEST 2024


Hi Marek,

> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: Saturday, July 27, 2024 4:38 AM
> To: Z.Q. Hou <zhiqiang.hou at nxp.com>; u-boot at lists.denx.de;
> trini at konsulko.com; lukma at denx.de; seanga2 at gmail.com;
> festevam at gmail.com; Peng Fan <peng.fan at nxp.com>
> Subject: Re: [PATCHv2] clk: imx8m: register ARM A53 core clock
> 
> On 7/26/24 6:01 PM, Z.Q. Hou wrote:
> > Hi Marek,
> 
> Hi,
> 
> >> -----Original Message-----
> >> From: Marek Vasut <marex at denx.de>
> >> Sent: Friday, July 26, 2024 10:05 PM
> >> To: Z.Q. Hou <zhiqiang.hou at nxp.com>; u-boot at lists.denx.de;
> >> trini at konsulko.com; lukma at denx.de; seanga2 at gmail.com;
> >> festevam at gmail.com; Peng Fan <peng.fan at nxp.com>
> >> Subject: Re: [PATCHv2] clk: imx8m: register ARM A53 core clock
> >>
> >> On 7/26/24 12:29 PM, Zhiqiang Hou wrote:
> >>> From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >>>
> >>> Register ARM A53 core clock for i.MX 8M Mini, Nano and Plus.
> >>
> >> Why is MX8M(Q) not included here ?
> >
> > I don't know iMX8MQ and it's not in my work plan.
> 
> Uh ... why are you adding this patch in the first place ?
> 
> Is there anything (driver? arch code?) which requires these clock ?

Yes, the cmd/cpu.c needs this to print the core frequency. The following patch enables the CONFIG_CPU feature:
http://patchwork.ozlabs.org/project/uboot/patch/20240726175108.34704-12-Zhiqiang.Hou@nxp.com/ 

> 
> MX8M(Q) is very similar to the rest of the MX8M from the software
> perspective, also up to quad-A53.

will have a look, but I'don't have a iMX8MQ board to verify the code.

Thanks,
Zhiqiang


More information about the U-Boot mailing list