On Wed, 2016-02-10 at 02:06 +0200, Vladimir Zapolskiy wrote: > Hi Sylvain, > > On 09.02.2016 20:47, slemieux.tyco@xxxxxxxxx wrote: > > From: Sylvain Lemieux <slemieux@xxxxxxxxxxx> > > > > This patch add an optional parameter to setup the > > xtal clock frequency output. > > > > Signed-off-by: Sylvain Lemieux <slemieux@xxxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/clock/nxp,lpc3220-clk.txt | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/clock/nxp,lpc3220-clk.txt b/Documentation/devicetree/bindings/clock/nxp,lpc3220-clk.txt > > index 20cbca3..ce51759 100644 > > --- a/Documentation/devicetree/bindings/clock/nxp,lpc3220-clk.txt > > +++ b/Documentation/devicetree/bindings/clock/nxp,lpc3220-clk.txt > > @@ -10,6 +10,9 @@ Required properties: > > - clock-names: list of external oscillator clock names, must contain > > "xtal_32k" and may have optional "xtal" > > > > +Optional properties: > > +- xtal,clock-output-frequency: define the "xtal" clock frequency output in Hz > > + > > this one looks wrong in sense that from patch 1/2 the property configures > a PLL output clock rate, which is not an oscillator (xtal) clock rate. > > I believe switching to "assigned-clock-rates" should make this change > obsolete. > Agreed, this change is not needed; Sylvain > > Examples: > > > > /* System Control Block */ > > > > -- > With best wishes, > Vladimir -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
