Re: [PATCH v3 1/3] dt-bindings: pwm: imx: Add optional clock '32k'

[Marek Vasut]

On 9/11/24 11:10 PM, Frank Li wrote:
> On Wed, Sep 11, 2024 at 10:28:52PM +0200, Marek Vasut wrote:
> > On 9/10/24 9:07 PM, Frank Li wrote:
> > > The pwm in imx8qxp mipi subsystem require one extra '32k' clock. So
> > > increase maxItems for clock and clock-names.
> >
> > This mentions MIPI subsystem, but the IP in question here is PWM.
>
> Here, mipi just name of subsystem, not related MIPI IP at all.
>
> There are many IP in i.MX8QXP mipi subsystem, such as i2c, PWM, MIPI PHY,
> MIPI controller, PLL, clock-gate, ...
>
> > Are you sure the clock are assigned to the correct IP ?
> >
> > Shouldn't the clock be assigned to some MIPI IP instead ?
>
> Are both question still validate if treat 'mipi' just name of subsystem.
>
> > Could you please clarify this in the commit message ?
>
> 'mipi' just name of subsystem because the major ip is for MIPI. is word
> 'mipi-subsystem' better?
Let's find out.

What is the 32kHz clock used for in the PWM block ?