diff mbox

[V2,1/3] pinctrl: Document "function" + "pins" pinmux binding

Message ID 1427757416-14491-2-git-send-email-abrestic@chromium.org
State New
Headers show

Commit Message

Andrew Bresticker March 30, 2015, 11:16 p.m. UTC
Currently the "function" + "groups" combination is the only documented
format for pinmux nodes, although many drivers use "function" + "pins".
Update the generic pinctrl binding to include the "function" + "pins"
combination as well.

Signed-off-by: Andrew Bresticker <abrestic@chromium.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Pawel Moll <pawel.moll@arm.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Ian Campbell <ijc+devicetree@hellion.org.uk>
Cc: Kumar Gala <galak@codeaurora.org>
---
New for v2.
---
 Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Linus Walleij April 7, 2015, 1:20 p.m. UTC | #1
On Tue, Mar 31, 2015 at 1:16 AM, Andrew Bresticker
<abrestic@chromium.org> wrote:

> Currently the "function" + "groups" combination is the only documented
> format for pinmux nodes, although many drivers use "function" + "pins".
> Update the generic pinctrl binding to include the "function" + "pins"
> combination as well.
>
> Signed-off-by: Andrew Bresticker <abrestic@chromium.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Pawel Moll <pawel.moll@arm.com>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: Ian Campbell <ijc+devicetree@hellion.org.uk>
> Cc: Kumar Gala <galak@codeaurora.org>
> ---
> New for v2.

Patch applied, as it just documents what some drivers are
already doing. Not much to be discussed there.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt b/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt
index 47d84b6..f7688e2 100644
--- a/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt
+++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt
@@ -133,6 +133,9 @@  pin multiplexing nodes:
 
 function		- the mux function to select
 groups			- the list of groups to select with this function
+			  (either this or "pins" must be specified)
+pins			- the list of pins to select with this function (either
+			  this or "groups" must be specified)
 
 Example:
 
@@ -144,6 +147,10 @@  state_1_node_a {
 	function = "spi0";
 	groups = "spi0pins";
 };
+state_2_node_a {
+	function = "i2c0";
+	pins = "mfio29", "mfio30";
+};
 
 == Generic pin configuration node content ==