From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm64: dts: marvell: fix number of GPIOs in Armada AP806 description
Date: Wed, 30 Aug 2017 15:51:02 +0200 [thread overview]
Message-ID: <874lsp4115.fsf@free-electrons.com> (raw)
In-Reply-To: <20170830101537.8260-1-thomas.petazzoni@free-electrons.com> (Thomas Petazzoni's message of "Wed, 30 Aug 2017 12:15:37 +0200")
Hi Thomas,
On mer., ao?t 30 2017, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:
> The Armada AP806 has 20 pins, and therefore 20 GPIOs (from 0 to 19
> included) and not 19 pins. Therefore, we fix the Device Tree
> description for the GPIO controller.
>
> Before this patch:
>
> $ cat /sys/kernel/debug/pinctrl/f06f4000.system-controller:pinctrl/gpio-ranges
> GPIO ranges handled:
> 0: mvebu-gpio GPIOS [0 - 19] PINS [0 - 19]
> 0: f06f4000.system-controller:gpio GPIOS [0 - 18] PINS [0 - 18]
>
> After this patch:
>
> $ cat /sys/kernel/debug/pinctrl/f06f4000.system-controller:pinctrl/gpio-ranges
> GPIO ranges handled:
> 0: mvebu-gpio GPIOS [0 - 19] PINS [0 - 19]
> 0: f06f4000.system-controller:gpio GPIOS [0 - 19] PINS [0 - 19]
>
> Fixes: 63dac0f4924b9 ("arm64: dts: marvell: add gpio support for Armada 7K/8K")
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Applied on mvebu/fixes
Thanks,
Gregory
> ---
> Note: this is a fix, that should go in the 4.13 cycle, as
> 63dac0f4924b9 was introduced in 4.13.
> ---
> arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi
> index 1eb1f1e..4d36071 100644
> --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi
> @@ -268,10 +268,10 @@
> ap_gpio: gpio {
> compatible = "marvell,armada-8k-gpio";
> offset = <0x1040>;
> - ngpios = <19>;
> + ngpios = <20>;
> gpio-controller;
> #gpio-cells = <2>;
> - gpio-ranges = <&ap_pinctrl 0 0 19>;
> + gpio-ranges = <&ap_pinctrl 0 0 20>;
> };
> };
> };
> --
> 2.9.4
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
prev parent reply other threads:[~2017-08-30 13:51 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-30 10:15 [PATCH] arm64: dts: marvell: fix number of GPIOs in Armada AP806 description Thomas Petazzoni
2017-08-30 13:51 ` Gregory CLEMENT [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=874lsp4115.fsf@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.