From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keerthy Subject: Re: [PATCH v3 2/2] ARM: dts: keystone-k2g-evm: Add gpio nodes Date: Wed, 26 Jul 2017 18:59:58 +0530 Message-ID: References: <1501051524-5990-1-git-send-email-j-keerthy@ti.com> <1501051524-5990-2-git-send-email-j-keerthy@ti.com> <1fb21a17-c38e-539f-7cd3-b72bd819a531@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1fb21a17-c38e-539f-7cd3-b72bd819a531@ti.com> Sender: linux-gpio-owner@vger.kernel.org To: Suman Anna , ssantosh@kernel.org, robh+dt@kernel.org Cc: devicetree@vger.kernel.org, linus.walleij@linaro.org, t-kristo@ti.com, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, fcooper@ti.com, nsekhar@ti.com List-Id: devicetree@vger.kernel.org On Wednesday 26 July 2017 06:33 PM, Suman Anna wrote: > Hi Keerthy, > > On 07/26/2017 01:45 AM, Keerthy wrote: >> 66AK2G has 2 instances of gpio. The first one has all the 144 GPIOs >> functional( 9 banks with 16 gpios = 144). The second instance has > > nit, space before ( and not after. Sure. > >> only the GPIO0:GPIO67 functional and rest are marked reserved. > > s/keystone-k2g-evm/keystone-k2g/ on the patch header please, otherwise > this patch looks good. Okay. > >> >> Signed-off-by: Keerthy >> --- >> >> The patch depends on the below series: >> >> http://marc.info/?l=linux-arm-kernel&m=150034845427555&w=2 >> http://marc.info/?l=linux-arm-kernel&m=150034856627618&w=2 > > FYI, these are already picked up my Santosh and are part of linux-next. Yes saw that now. I will send v4. > > regards > Suman > >> >> Driver patches related are already posted: >> >> https://www.spinics.net/lists/arm-kernel/msg594862.html >> >> Changes in v2: >> >> * Split the documentation part into a separate patch. >> >> arch/arm/boot/dts/keystone-k2g.dtsi | 42 +++++++++++++++++++++++++++++++++++++ >> 1 file changed, 42 insertions(+) >> >> diff --git a/arch/arm/boot/dts/keystone-k2g.dtsi b/arch/arm/boot/dts/keystone-k2g.dtsi >> index bf4d1fa..f9ff299 100644 >> --- a/arch/arm/boot/dts/keystone-k2g.dtsi >> +++ b/arch/arm/boot/dts/keystone-k2g.dtsi >> @@ -15,6 +15,7 @@ >> >> #include >> #include >> +#include >> >> / { >> compatible = "ti,k2g","ti,keystone"; >> @@ -168,5 +169,46 @@ >> #reset-cells = <2>; >> }; >> }; >> + >> + gpio0: gpio@2603000 { >> + compatible = "ti,keystone-k2g-gpio", "ti,keystone-gpio"; >> + reg = <0x02603000 0x100>; >> + gpio-controller; >> + #gpio-cells = <2>; >> + >> + interrupts = , >> + , >> + , >> + , >> + , >> + , >> + , >> + , >> + ; >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + ti,ngpio = <144>; >> + ti,davinci-gpio-unbanked = <0>; >> + clocks = <&k2g_clks 0x001b 0x0>; >> + clock-names = "gpio"; >> + }; >> + >> + gpio1: gpio@260a000 { >> + compatible = "ti,keystone-k2g-gpio", "ti,keystone-gpio"; >> + reg = <0x0260a000 0x100>; >> + gpio-controller; >> + #gpio-cells = <2>; >> + interrupts = , >> + , >> + , >> + , >> + ; >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + ti,ngpio = <68>; >> + ti,davinci-gpio-unbanked = <0>; >> + clocks = <&k2g_clks 0x001c 0x0>; >> + clock-names = "gpio"; >> + }; >> }; >> }; >> >