[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_JsqKQJzUOWE2oJsABH3Aa1cCD+526L0G7-WRH=LbKFxesyQ@mail.gmail.com>
Date: Tue, 13 Jan 2015 14:06:24 -0600
From: Rob Herring <robherring2@...il.com>
To: Linus Walleij <linus.walleij@...aro.org>
Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>
Subject: Re: [PATCH] pinctrl: fix up device tree bindings
On Sat, Jan 10, 2015 at 3:14 PM, Linus Walleij <linus.walleij@...aro.org> wrote:
> After the Nomadik pin controller was force migrated to generic pin
> control bindings, some leftovers in the documentation need to be
> cleaned up. The code and device trees are already migrated.
>
> Signed-off-by: Linus Walleij <linus.walleij@...aro.org>
Applied for 3.20. Thanks.
> ---
> .../devicetree/bindings/pinctrl/ste,nomadik.txt | 35 ++++++++++++----------
> 1 file changed, 20 insertions(+), 15 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/ste,nomadik.txt b/Documentation/devicetree/bindings/pinctrl/ste,nomadik.txt
> index 6b33b9f18e88..f63fcb3ed352 100644
> --- a/Documentation/devicetree/bindings/pinctrl/ste,nomadik.txt
> +++ b/Documentation/devicetree/bindings/pinctrl/ste,nomadik.txt
> @@ -16,17 +16,22 @@ mux function to select on those pin(s)/group(s), and various pin configuration
> parameters, such as input, output, pull up, pull down...
>
> The name of each subnode is not important; all subnodes should be enumerated
> -and processed purely based on their content.
> +and processed purely based on their content. The subnodes use the generic
> +pin multiplexing node layout from the standard pin control bindings
> +(see pinctrl-bindings.txt):
>
> -Required subnode-properties:
> -- ste,pins : An array of strings. Each string contains the name of a pin or
> - group.
> -
> -Optional subnode-properties:
> -- ste,function: A string containing the name of the function to mux to the
> +Required pin multiplexing subnode properties:
> +- function: A string containing the name of the function to mux to the
> pin or group.
> +- groups : An array of strings. Each string contains the name of a pin
> + group that will be combined with the function to form a multiplexing
> + set-up.
>
> -- ste,config: Handle of pin configuration node (e.g. ste,config = <&slpm_in_wkup_pdis>)
> +Required pin configuration subnode properties:
> +- pins: A string array describing the pins affected by the configuration
> + in the node.
> +- ste,config: Handle of pin configuration node
> + (e.g. ste,config = <&slpm_in_wkup_pdis>)
>
> - ste,input : <0/1/2>
> 0: input with no pull
> @@ -97,32 +102,32 @@ Example board file extract:
> uart0 {
> uart0_default_mux: uart0_mux {
> u0_default_mux {
> - ste,function = "u0";
> - ste,pins = "u0_a_1";
> + function = "u0";
> + pins = "u0_a_1";
> };
> };
> uart0_default_mode: uart0_default {
> uart0_default_cfg1 {
> - ste,pins = "GPIO0", "GPIO2";
> + pins = "GPIO0", "GPIO2";
> ste,input = <1>;
> };
>
> uart0_default_cfg2 {
> - ste,pins = "GPIO1", "GPIO3";
> + pins = "GPIO1", "GPIO3";
> ste,output = <1>;
> };
> };
> uart0_sleep_mode: uart0_sleep {
> uart0_sleep_cfg1 {
> - ste,pins = "GPIO0", "GPIO2";
> + pins = "GPIO0", "GPIO2";
> ste,config = <&slpm_in_wkup_pdis>;
> };
> uart0_sleep_cfg2 {
> - ste,pins = "GPIO1";
> + pins = "GPIO1";
> ste,config = <&slpm_out_hi_wkup_pdis>;
> };
> uart0_sleep_cfg3 {
> - ste,pins = "GPIO3";
> + pins = "GPIO3";
> ste,config = <&slpm_out_wkup_pdis>;
> };
> };
> --
> 1.9.3
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists