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.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to