Thread (30 messages) 30 messages, 6 authors, 2011-08-15

[RFC PATCH 05/12] arm/dt: Tegra: Add nvidia,gpios property to GPIO controller

From: Stephen Warren <hidden>
Date: 2011-08-15 16:15:43
Also in: linux-devicetree, linux-tegra, lkml

Olof Johansson wrote at Sunday, August 14, 2011 1:02 AM:
On Fri, Aug 12, 2011 at 3:54 PM, Stephen Warren [off-list ref] wrote:
quoted
Add board-specific gpio node for Harmony and Seaboard. This lists the
GPIOs used by the board. Note that not all GPIOs that exist on the board
are listed; only those used by devices currently supported by device
tree.
I don't want to bikeshed over this, but something like
"nvidia,enabled-gpios" could be a bit more descriptive. Either way is
OK though.
Yes, that's a much better name.

Existing board files do have a Boolean enabled/disabled flag per GPIO,
which is currently always true,  whereas this patch series only implements
"enabled" entries. I thought about it briefly, and didn't think that
explicitly disabling GPIOs that the bootloader had already enabled was
likely to be useful. Do you agree? Possibly not since IIRC you added the
tables that contain that Boolean.
The property should be documented under the bindings Documentation/
hierarchy though.
Yup. The next posting will include this.

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