* Michael Turquette [off-list ref] [160216 17:17]:

Quoting Tony Lindgren (2016-02-12 13:20:08)

quoted

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

quoted

+- 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 at 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