From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 2/2] ARM: shmobile: r8a7794: add GPIO DT support Date: Wed, 25 Feb 2015 17:04:53 +0300 Message-ID: <54EDD685.1010802@cogentembedded.com> References: <6954455.LyoCrfjM8R@wasted.cogentembedded.com> <3082195.zNdUtZ8oMM@wasted.cogentembedded.com> <2292882.LbM0mNWfOj@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <2292882.LbM0mNWfOj@avalon> Sender: linux-sh-owner@vger.kernel.org To: Laurent Pinchart Cc: horms@verge.net.au, linux-sh@vger.kernel.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, devicetree@vger.kernel.org, magnus.damm@gmail.com, linux@arm.linux.org.uk, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hello. On 2/25/2015 1:11 PM, Laurent Pinchart wrote: >> Describe GPIO[0-6] controllers in the R8A7794 device tree. >> Based on original patch by Hisashi Nakamura >> . >> Signed-off-by: Sergei Shtylyov >> --- >> arch/arm/boot/dts/r8a7794.dtsi | 84 +++++++++++++++++++++++++++++++++++++ >> 1 file changed, 84 insertions(+) >> >> Index: renesas/arch/arm/boot/dts/r8a7794.dtsi >> =================================================================== >> --- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi >> +++ renesas/arch/arm/boot/dts/r8a7794.dtsi >> @@ -50,6 +50,90 @@ [...] >> + gpio1: gpio@e6051000 { >> + compatible = "renesas,gpio-r8a7794", "renesas,gpio-rcar"; >> + reg = <0 0xe6051000 0 0x50>; >> + interrupts = <0 5 IRQ_TYPE_LEVEL_HIGH>; >> + #gpio-cells = <2>; >> + gpio-controller; >> + gpio-ranges = <&pfc 0 32 32>; > This GPIO block has 26 GPIOs only. [...] >> + gpio5: gpio@e6055000 { >> + compatible = "renesas,gpio-r8a7794", "renesas,gpio-rcar"; >> + reg = <0 0xe6055000 0 0x50>; >> + interrupts = <0 9 IRQ_TYPE_LEVEL_HIGH>; >> + #gpio-cells = <2>; >> + gpio-controller; >> + gpio-ranges = <&pfc 0 160 32>; > This GPIO block has 28 GPIOs only. >> + #interrupt-cells = <2>; >> + interrupt-controller; >> + clocks = <&mstp9_clks R8A7794_CLK_GPIO5>; >> + }; >> + >> + gpio6: gpio@e6055400 { >> + compatible = "renesas,gpio-r8a7794", "renesas,gpio-rcar"; >> + reg = <0 0xe6055400 0 0x50>; >> + interrupts = <0 10 IRQ_TYPE_LEVEL_HIGH>; >> + #gpio-cells = <2>; >> + gpio-controller; >> + gpio-ranges = <&pfc 0 192 32>; > This GPIO block has 26 GPIOs only. Yes, I know; I just wasn't sure what to do with the "gpio-ranges" prop of the following GPIO node. Should I keep the base GPIO # or update it as well? WBR, Sergei