RE: [PATCH v9 00/23] Support spread spectrum clocking for i.MX8M PLLs
From: Peng Fan
Date: Fri Jan 24 2025 - 18:47:57 EST
> Subject: Re: [PATCH v9 00/23] Support spread spectrum clocking for
> i.MX8M PLLs
>
> On 18/01/2025 13:39, Dario Binacchi wrote:
> > The series adds support for spread spectrum clocking for
> i.MX8M{M,N,P}
> > PLLs (audio, video and DRAM). It has been tested for the video PLL on
> > boards using i.MX8MN and i.MX8MP.
> >
> > Changes in v9:
> > - Add 'Reviewed-by' tag of Peng Fan for imx8mn platform patches
> > - Fix building warning raised by the kernel test robot for patch
> > v8, 11/18 clk: imx: add support for i.MX8MN anatop clock driver
> > - Add patches for imx8m{m,p} platforms:
> > - 23/23 clk: imx8mm: support spread spectrum clock generation
> > - 22/23 clk: imx: add support for i.MX8MM anatop clock driver
> > - 21/23 clk: imx8mp: support spread spectrum clock generation
> > - 20/23 clk: imx8mp: rename ccm_base to base
> > - 19/23 clk: imx: add support for i.MX8MP anatop clock driver
> I responded in particular patch, but let's respond for formality to cover
> letter as well.
Sorry to come into the binding part at so late stage. I just
got a request for i.MX95 SCMI CLK a few days ago which is impossible
for vendor specific properties.
But I am not sure "assigned-clock-sscs" is welcomed by maintainers.
Please also help to check whether this is usable for your patches
to enable SSC for i.MX8M PLL.
>
> The bindings here might get obsolete already with work:
> https://github.com/devicetree-org/dt-schema/pull/154
Not intend to waste to your effort.
I also replied in the PR to express my apologize.
Thanks,
Peng.
>
> Best regards,
> Krzysztof