Re: [PATCH v2 4/4] clk: fsl-sai: Add MCLK generation support

["Michael Walle" <michael@xxxxxxxx>]

Hi,

> > ..Which is the
> > normal use case for this pin. This driver was created because the
> > LS1028A doesn't have a MCLK pin, so we've "misused" the BCLK pin,
> > with the restriction that only integer dividers are possible.
>
> I have a system that is wired a bit unfortunately, I need to source 
> codec clock, where the codec is the clock consumer and needs to be able 
> to control the clock (SGTL5000). SAI MCLK is the only way I can get them 
> out of the pin I need, hence this patch.

Which is also the default case, no?

> > Also I'd expect that the imx
> > SoCs already supports the MCLK for audio applications. Isn't that
> > the case?
>
> That does not work if the MCLK has to be enabled/disabled by the MCLK 
> clock consumer .

Why's that?

Don't get me wrong. I don't have anything against this patch, I'm
just confused, why that isn't already working with the current MCLK
driver as this seems to be the usual requirements.

-michael

Attachment: signature.asc
Description: PGP signature