Re: [PATCH] clk: add flags to distinguish xtal clocks

From: Felipe Balbi
Date: Mon Nov 11 2013 - 14:44:51 EST


Hi,

On Mon, Nov 11, 2013 at 09:27:58AM -0700, Stephen Warren wrote:
> On 11/08/2013 11:00 AM, Felipe Balbi wrote:
> > From: Luciano Coelho <luca@xxxxxxxxx>
> >
> > Add a flag that indicate whether the clock is a crystal or not.
> >
> > Additionally, parse a new device tree binding in clk-fixed-rate to set
> > this flag.
> >
> > If clock-xtal isn't set, the clock framework will assume clock to be
> > generated by an oscillator. There's only one user for this binding
> > right now which is Texas Instruments' WiLink devices which need to know
> > details about the clock in order to initialize the underlying WiFi HW
> > correctly.
>
> Why on earth does it care? Surely the WiFi HW doesn't care about
> crystal-vs-non-crystal, but rather some facet of the clock signal that
> the type of source implies. Shouldn't the DT property describe that
> facet of the signal, rather than the reason why it has that facet?

well, if you can figure out what that facet is, then _do_ tell. Luca has
tried for months to get that information with no success.

--
balbi

Attachment: signature.asc
Description: Digital signature