Am Dienstag, den 17.12.2013, 14:00 +0000 schrieb Mark Brown:
On Tue, Dec 17, 2013 at 11:52:44AM +0100, Lucas Stach wrote:
This corrects the Tegra AC97 clock handling to be in line with the devicetree. Audio PLL is controlled by the machine driver, only AC97 host controller clock remains to be independent.
I don't understand this commit message at all I'm afraid. What does it mean to "be in line with the devicetree" and what is the purpose of this change?
Clocks were added to the DT after the Tegra AC97 driver was initially pushed upstream and the layout changed a bit in the process. Stephen Warren convinced me that it's the right thing to let the Audio PLL be controlled by the ASoC machine driver, rather than the AC97 host controller driver.
So currently the host controller driver asks for clocks that aren't there in the DT. This change corrects the clock handling so that the host controller driver only ask for it's own clock, but the Audio PLL handling is pushed into the machine driver.
Regards, Lucas
I should also have mentioned on the previous patch: please use subject lines matching the subsystem. You've got "ASOC" not "ASoC".