Re: [PATCH 3/4 v3][RFC] arm64: renesas: Add initial r8a7795 SoC support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux