Thread (1 message) 1 message, 1 author, 2012-03-04

Re: [PATCH v3 07/10] arm/tegra: Add PWFM controller device tree probing

From: Arnd Bergmann <hidden>
Date: 2012-03-04 20:39:14
Also in: linux-arm-kernel, linux-tegra

Possibly related (same subject, not in this thread)

On Saturday 03 March 2012, Thierry Reding wrote:
Not enough information to check signature validity.  Show Details
  * Stephen Warren wrote:
quoted
Thierry Reding wrote at Wednesday, February 22, 2012 8:17 AM:
quoted
Add auxdata to instantiate a device tree for the PWFM controller and
include a corresponding node in the device tree.
quoted
diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi
quoted
+   pwm: pwm@7000a000 {
+           compatible = "nvidia,tegra20-pwm";
+           reg = <0x7000a000 0x100>;
+           #pwm-cells = <2>;
+   };
The compatible value probably should list both Tegra30 and Tegra20, so
we can know exactly which HW is present, just in case we need to turn
on some bug-fix only for one of the variants:

    compatible = "nvidia,tegra30-pwm", "nvidia,tegra20-pwm";
I'm confused. If I know exactly that the hardware is Tegra30 (which it
definitely should be if I include tegra30.dtsi), then why list "tegra20-pwm"
as compatible?

Or did you mean to list tegra30-pwm as compatible value in the PWM driver?
We generally list the oldest version of the device that a specific version
is compatible to, in order to use the same driver unmodified.

E.g. an ns16550a serial port would list both ns16550a and i8250 in its
compatible fields so it works with operating systems that only know about
i8250.

It's not actually all that useful when you add device tree support for
two versions of the same device at the same time, but I think that
it's good style anyway, in particular when you consider other users of
the device tree data besides the Linux kernel.

	Arnd
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help