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

Re: [PATCH v4 07/10] pwm: tegra: Add device tree support

From: Stephen Warren <hidden>
Date: 2012-03-20 15:33:01
Also in: linux-arm-kernel, linux-tegra

Possibly related (same subject, not in this thread)

On 03/20/2012 02:48 AM, Thierry Reding wrote:
* Stephen Warren wrote:
quoted
On 03/14/2012 09:56 AM, Thierry Reding wrote:
quoted
Add auxdata to instantiate the PWFM controller from a device tree,
include the corresponding nodes in the dtsi files for Tegra 20 and
Tegra 30 and add binding documentation.

Signed-off-by: Thierry Reding <redacted>
quoted
+++ b/Documentation/devicetree/bindings/pwm/tegra-pwm.txt
Can this please be named nvidia,tegra20-pwm.txt so that if we need
different bindings for any future Tegra PWM, we won't have any filename
conflicts?
That file already documents the bindings for Tegra 20 and Tegra 30, that's
why I chose the move generic name. Could this perhaps be documented within
the file instead? Or should I rather add a nvidia,tegra30-pwm.txt with a
reference to the nvidia,tegra20-pwm.txt?
I think it's fine for a file of name nvidia,tegra20-pwm.txt to document
both Tegra20 and Tegra30; the file is named after the first/earliest
compatible value it documents. The issue is more that /if/ say TegraNN
needs a different binding, I want the naming consistent (e.g.
nvidia,tegraNN-pwm.txt) rather than having the old file with one style
and the new file named using a different style.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help