* Michael Turquette <mturquette@xxxxxxxxxxxx> [160216 17:17]:
> Quoting Tony Lindgren (2016-02-12 13:20:08)
> > +Required properties:
> > +- compatible : shall be one of "ti,dm814-adpll-s-clock" or
> > + "ti,dm814-adpll-lj-clock" depending on the type of the ADPLL
> > +- #clock-cells : from common clock binding; shall be set to 0.
>
> Thanks for the patch.
>
> clock-cells = 1 in both of the examples below (which looks right).
Oops thanks, will update to 1.
> > +- clocks : link phandles of parent clocks clkinp and clkinpulow, note
> > + that the adpll-s-clock also has an optional clkinphif
> > +- reg : address and length of the register set for controlling the ADPLL.
> > +
> > +Examples:
> > + adpll_mpu_ck: adpll@40 {
> > + #clock-cells = <1>;
> > + compatible = "ti,dm814-adpll-s-clock";
> > + reg = <0x40 0x40>;
> > + clocks = <&devosc_ck &devosc_ck &devosc_ck>;
> > + clock-names = "clkinp", "clkinpulow", "clkinphif";
> > + clock-indices = <0>, <1>, <2>, <3>;
>
> Why is clock-indices necessary?
Hmm yeah I don't think it's needed, so will drop.
Regards,
Tony
--
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
