[PATCH 4/6 v2] ARM: integrator: initial device tree support

On Thu, Sep 6, 2012 at 4:19 PM, Arnd Bergmann [off-list ref] wrote:

On Thursday 06 September 2012, Linus Walleij wrote:

quoted

On Thu, Sep 6, 2012 at 3:20 PM, Rob Herring [off-list ref] wrote:

quoted

[Me]

quoted

I can of course skip the aliases and go for the timer nodes
I want directly, but that isn't any helpful for people reading the
device tree, is it?

Well, the dts doesn't really need to describe how Linux is using things.

The arm,integrator-clocksource and arm,integrator-clockevent
are renamed arm,timer-primary and arm,timer-secondary in the
last patch set.

Are you OK with this for now, going forward? I'd be happy to
take a round refactoring it for the next cycle as outlined
below.

If the timers are really free for use by anything and you want to
encode how Linux uses them, I would suggest naming the aliases
"linux.clocksource" and "linux,clockevent" to make it clear that
you are not describing the hardware here but rather picking
a configuration.

So the renaming of the aliases to arm,timer-primary and
arm-timer-secondary is to be taken as a hint to the OS to use
these two as primary and secondary respectively.

linux.foo seems bad in comparison, but if you insist...

Yours,
Linus Walleij