[alsa-devel] [PATCH 1/2] ASoC: rt5682: Add CCF usage for providing I2S clks
Mark Brown
broonie at kernel.org
Mon Feb 17 13:36:50 CET 2020
On Mon, Feb 17, 2020 at 10:59:55AM +0000, Derek [方德義] wrote:
> > Subject: Re: [PATCH 1/2] ASoC: rt5682: Add CCF usage for providing I2S clks
> > > + if (parent_rate != CLK_PLL2_FIN)
> > > + dev_warn(component->dev, "clk %s only support %d Hz input\n",
> > > + clk_name, CLK_PLL2_FIN);
> > > + if (rate != CLK_48) {
> > > + dev_warn(component->dev, "clk %s only support %d Hz output\n",
> > > + clk_name, CLK_48);
> > > + rate = CLK_48;
> > > + }
> > Are these genuine restrictions of the hardware or is this just being
> > hard coded in the driver?
> It's hard coded for an application case.
> Generate a 48kHz clk with a parent 48 MHz clk.
I see... obviously this is really not good quality of implementation
although it doesn't actually break any external interfaces so I guess we
could take it. I would at least want to see some clear comments in both
the code and the changelog explaining that this limitation is just a
temporary hack until a better implementation is done though.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20200217/b2bf7d20/attachment.sig>
More information about the Alsa-devel
mailing list