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. > 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
