Re: [PATCH 4/5] gpio: of: Add support to have multiple gpios in gpio-hog

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

 





On Thursday 17 March 2016 09:16 PM, Rob Herring wrote:
On Thu, Mar 10, 2016 at 05:23:55PM +0530, Laxman Dewangan wrote:
On this case, we have already property "line-name" and passed the name
of the gpio via this property.
The property names is "line-name" which is good for one string. We can
support other property "line-names" with multiple string per GPIO index.

line-names = "wlan-reset", "wlan-enable";
Then what happens when someone wants to selectively disable gpio hogs?

status = "okay", "disabled", "okay";

While I often push things to fewer nodes and more compact descriptions,
I don't think that is the right direction in this case.

I dont think we need to support the individual gpio to be enable/disable by status. We need to support the status property at node level only. if individual gpio need to be enabled/disabled by status then it need to break in different hog nodes.

This is same as like we have in pincontrol where we can provide the list of pin names for some configuration under same node.


There is currently a discussion about the future bindings for subnodes in GPIO
controller nodes. Please have a look at these two mail threads:

	"Device tree binding documentation for gpio-switch"
	"gpio: of: Add support to have multiple gpios in gpio-hog"
Second one is this patch only. Is it by intention?

The binding details about the gpio-switch and names are given by property
"lable". I think property "label" is standard way of going forward i.e. I
post similar patch for gpio-keys device name from DT after got review
comment.

So here,  we can have the gpio names  under property "label" or "labels".
label is standard. labels you just made up.

Yaah, lables for plural only. Otherwise no issue with "label".


Or am I missing anything?
The point is the more one off changes I see that are all inter-related,
the less willing I am to accept any that don't consider all the cases.
The inter-relationship here is how do we describe gpio lines that don't
otherwise have a connection to another node and how to deal with them if
that changes.



--
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