On Mon, Jan 13, 2014 at 22:27 -0800, Paul Walmsley wrote: > > On Thu, 19 Dec 2013, Stephen Warren wrote: > > >On 12/19/2013 05:49 AM, Paul Walmsley wrote: > >>Add basic DT bindings for the DFLL IP block for the NVIDIA Tegra114 SoC. > > > >>diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra114-dfll.txt b/Documentation/devicetree/bindings/clock/nvidia,tegra114-dfll.txt > > > >>+- clocks : Must contain an array of two-cell arrays, one per clock. > >>+ DFLL source clocks. At minimum this should include the > >>+ reference clock source and the IP block's main clock > >>+ source. Also it should contain the DFLL's I2C controller > >>+ clock source. The format is <&clock-provider-phandle > >>+ clock-id>. > > > >Entries in "clocks" aren't two cells, they're a phandle plus as many > >cells as the node referenced by the phandle specifies. > > It's worth noting that the clock binding documentation itself refers > to pairs: > > ---- > > clocks: List of phandle and clock specifier pairs, one pair > for each clock input to the device. Note: if the > clock provider specifies '0' for #clock-cells, then > only the phandle portion of the pair will appear. > > ---- > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/clock/clock-bindings.txt#n50 > > But given the ambiguity of that documentation, I basically agree, so > have changed it to: Please note that there neither is an ambiguity nor a conflict here, and that you actually acknowledge what Stephen said: Clocks get referenced by a pair which consists of a phandle for the clock provider and a clock specifier. The clock specifier is made of as many cells as the clock provider's #clock-cells property defines (including none, or any other number than 1). This is exactly what Stephen said: A "clocks" item does not need to have two cells. The pair of phandle and clock specifier don't necessarily translate into two cells, instead the number of cells depends on the clock provider. Also note that the phandle is specifically _not_ part of the clock specifier. virtually yours Gerhard Sittig -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr. 5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@xxxxxxx -- 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
