Hello Thomas,
2015-02-28 10:58 GMT+01:00 Thomas Petazzoni thomas.petazzoni@free-electrons.com:
Dear Marcin Wojtas,
On Fri, 27 Feb 2015 21:22:30 +0100, Marcin Wojtas wrote:
Given all the facts, do you think that drivers/clk/mvebu/armada-38x.c is a right place for adding this PLL setting support?
Yes, sounds right to me. I was thinking that maybe it should be in a separate file armada-38x-pll.c, but that's probably a bit too much for something that is causing to be a simple driver.
I've got an update from Marvell - TDM and Audio are available in A39x and it has the same set of PLL registers like A38x. Shouldn't we place PLL configuration code in a separate file like clk-pll.c, or whatever name you suggest?
Have you sorted out how to handle the pin-muxing part of the problem?
I'll come back with this issue after implementing poc.
Best regards, Marcin