Re: [PATCH] ASoC: da7219: read fmw property to get mclk for non-dts systems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 4, 2024 at 1:50 PM Curtis Malainey <cujomalainey@xxxxxxxxxx> wrote:
>
> On Mon, Mar 4, 2024 at 1:23 PM Mark Brown <broonie@xxxxxxxxxx> wrote:
> >
> > On Mon, Mar 04, 2024 at 01:14:43PM -0800, cujomalainey@xxxxxxxxxxxx wrote:
> >
>
>
> No problem, I can clean it up, I figured a good starting point would
> be to just revive the original that was sent in 2018
> https://mailman.alsa-project.org/hyperkitty/list/alsa-devel@xxxxxxxxxxxxxxxx/message/26IVUCF7KMKNL7LZWAWWNFF3KB2TURSA/
>
> We have been carrying this in our tree the whole time, just caught it
> and figure I would try and get it up stream again

So after a bunch of debugging on our 5.10 kernel it actually appears
this property is useless and the system is already attaching the mclk
under the clock defined in the property. Maybe someone pushed a
coreboot update to fix this later which rendered this patch useless?

>
> >
> >
> > Having a firmware property for this is obviously broken for DT systems,
> > this should be limited to ACPI systems if it's going to be there at all.
> > It would be nicer if it were implementeded by having some ACPI specific
> > code link whatever the configured clock name is to "mclk" - I don't know
> > if the clock API has an equivalent to regulator_register_supply_alias()
> > but that's the sort of thing I'm thinking of.
>
> I will take a look at this, note that it appears the original author
> is no longer at AMD as the emails are bouncing.

To answer this for the sake of sharing knowledge, there is a clk_add_alias API.




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux