On Fri, May 31, 2024 at 10:48:12AM -0400, Elinor Montmasson wrote:
From: "Mark Brown" broonie@kernel.org
So you're trying to use this as the audio clock? There's no code that enables the clock which seems worrying, and I'd expect that if the device is using it's own clock the device would be querying it directly via the clock API rather than this. This all seems really confused.
It's not specifically the audio clock, I am merely using this in the machine driver to let the user the possibility to configure the CPU DAI sysclock frequency. The CPU DAI and codec drivers already manage their own clocks.
I would expect that if the clocks used by the devices are configured via the clock API then the drivers for those devices will configure themselves via the clock API. I still don't understand what this change is intended to accomplish.