Re: [PATCH] Documentation: minor correction of gpio dts example

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

 



On Thu, Jan 28, 2016 at 12:38 AM,  <wr.phallsma@xxxxxxxxx> wrote:
> From: Per Hallsmark <per.hallsmark@xxxxxxxxxxxxx>
>
> As the example brings up two ranges of 10 pins each, the count
> should be 10 in both range definitions.
>
> Signed-off-by: Per Hallsmark <per.hallsmark@xxxxxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/gpio/gpio.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/gpio/gpio.txt b/Documentation/devicetree/bindings/gpio/gpio.txt
> index 069cdf6..ad3d64b 100644
> --- a/Documentation/devicetree/bindings/gpio/gpio.txt
> +++ b/Documentation/devicetree/bindings/gpio/gpio.txt
> @@ -252,7 +252,7 @@ Example 1:
>                 compatible = "fsl,qe-pario-bank-e", "fsl,qe-pario-bank";
>                 reg = <0x1460 0x18>;
>                 gpio-controller;
> -               gpio-ranges = <&pinctrl1 0 20 10>, <&pinctrl2 10 50 20>;
> +               gpio-ranges = <&pinctrl1 0 20 10>, <&pinctrl2 10 50 10>;

Looks correct wrt. the description that follows, thanks!

Reviewed-by: Alexandre Courbot <acourbot@xxxxxxxxxx>

One nit: please make sure to run scripts/get_maintainer.pl on your
patch and email the maintainers of the subsystem you are submitting to
directly. It usually makes your email more visible in their inboxes
than if you send to the mailing-list only.
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux