
6 Jun
2013
6 Jun
'13
3:33 p.m.
On Thu, Jun 06, 2013 at 08:49:53PM +0800, Nicolin Chen wrote:
On Wed, Jun 05, 2013 at 12:55:44PM +0100, Mark Brown wrote:
Since it's easy to define a fixed rate clock (there's a generic driver for that) I'd just require the user to provide a clock API clock and fix the rate using that. This is going to be less error prone and makes the code simpler.
I tried to use fixed rate clock as below:
data->codec_clk = devm_clk_get(&codec_dev->dev, NULL); if (IS_ERR(data->codec_clk)) { of_fixed_clk_setup(codec_np); data->codec_clk = clk_get(NULL, codec_np->name);
No, this is silly. Have the board define the clock in the DT.