From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregory.clement@free-electrons.com (Gregory CLEMENT) Date: Tue, 19 Sep 2017 16:25:58 +0200 Subject: [PATCH] arm64: dt marvell: extend AP806 system controller size In-Reply-To: (Baruch Siach's message of "Fri, 15 Sep 2017 10:50:07 +0300") References: Message-ID: <877ewusr2h.fsf@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Baruch, On ven., sept. 15 2017, Baruch Siach wrote: > Extend the container size to 0x2000 to include the gpio controller at > offset 0x1040. > > While at it, add start address notation to the gpio node name to match > its 'offset' property. > > Signed-off-by: Baruch Siach I made a quick test on a mcbin and I confirm that the watchdog is here and usable. Tested-by: Gregory CLEMENT and Applied on mvebu/dt64 Thanks, Gregory > --- > 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 4d360713ed12..30d48ecf46e0 100644 > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > @@ -254,7 +254,7 @@ > > ap_syscon: system-controller at 6f4000 { > compatible = "syscon", "simple-mfd"; > - reg = <0x6f4000 0x1000>; > + reg = <0x6f4000 0x2000>; > > ap_clk: clock { > compatible = "marvell,ap806-clock"; > @@ -265,7 +265,7 @@ > compatible = "marvell,ap806-pinctrl"; > }; > > - ap_gpio: gpio { > + ap_gpio: gpio at 1040 { > compatible = "marvell,armada-8k-gpio"; > offset = <0x1040>; > ngpios = <20>; > -- > 2.14.1 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory CLEMENT Subject: Re: [PATCH] arm64: dt marvell: extend AP806 system controller size Date: Tue, 19 Sep 2017 16:25:58 +0200 Message-ID: <877ewusr2h.fsf@free-electrons.com> References: Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: (Baruch Siach's message of "Fri, 15 Sep 2017 10:50:07 +0300") Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Baruch Siach Cc: Jason Cooper , Andrew Lunn , Sebastian Hesselbarth , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Baruch, On ven., sept. 15 2017, Baruch Siach wrote: > Extend the container size to 0x2000 to include the gpio controller at > offset 0x1040. > > While at it, add start address notation to the gpio node name to match > its 'offset' property. > > Signed-off-by: Baruch Siach I made a quick test on a mcbin and I confirm that the watchdog is here and usable. Tested-by: Gregory CLEMENT and Applied on mvebu/dt64 Thanks, Gregory > --- > 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 4d360713ed12..30d48ecf46e0 100644 > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > @@ -254,7 +254,7 @@ > > ap_syscon: system-controller@6f4000 { > compatible = "syscon", "simple-mfd"; > - reg = <0x6f4000 0x1000>; > + reg = <0x6f4000 0x2000>; > > ap_clk: clock { > compatible = "marvell,ap806-clock"; > @@ -265,7 +265,7 @@ > compatible = "marvell,ap806-pinctrl"; > }; > > - ap_gpio: gpio { > + ap_gpio: gpio@1040 { > compatible = "marvell,armada-8k-gpio"; > offset = <0x1040>; > ngpios = <20>; > -- > 2.14.1 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html