Hi Mike, On Wed, Aug 19, 2015 at 9:49 AM, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > On Tue, Aug 18, 2015 at 2:20 AM, Michael Turquette > <mturquette@xxxxxxxxxxxx> wrote: >> Also, while you're thinking about the perfect clock binding, please do >> consider dropping clock-output-names if you can. Specifying clock-names >> alongside the clocks property inside of the consumer node is a bit more >> elegant in my opinion. This is also a bit easier if you think about >> expressing your clock data with C inside of your provider driver. > > Makes sense. I don't think anything relies on the "clock-output-names" > ... currently. BTW, was "dropping clock-output-names" a general comment for all clock drivers, or specific to the SoC's Clock Pulse Generator? For e.g. "fixed-factor-clock", the driver falls back to using the node's name if "clock-output-names" is not present. But e.g. for MSTP clocks, that can't be done, as the clocks wouldn't have names in that case (single node with multiple clocks). Unless we start fabricating them from the node name and the indices. > I was going to use it for identifying the GIC clock, though: That applies to MSTP clocks ("intc-sys"), not the main CPG block clocks. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- 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