* [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-15 7:50 ` Baruch Siach 0 siblings, 0 replies; 10+ messages in thread From: Baruch Siach @ 2017-09-15 7:50 UTC (permalink / raw) To: linux-arm-kernel 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 <baruch@tkos.co.il> --- 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 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-15 7:50 ` Baruch Siach 0 siblings, 0 replies; 10+ messages in thread From: Baruch Siach @ 2017-09-15 7:50 UTC (permalink / raw) To: Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA, Baruch Siach 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 <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> --- 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 -- 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 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-19 14:25 ` Gregory CLEMENT 0 siblings, 0 replies; 10+ messages in thread From: Gregory CLEMENT @ 2017-09-19 14:25 UTC (permalink / raw) To: linux-arm-kernel Hi Baruch, On ven., sept. 15 2017, Baruch Siach <baruch@tkos.co.il> 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 <baruch@tkos.co.il> I made a quick test on a mcbin and I confirm that the watchdog is here and usable. Tested-by: Gregory CLEMENT <gregory.clement@free-electrons.com> 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-19 14:25 ` Gregory CLEMENT 0 siblings, 0 replies; 10+ messages in thread From: Gregory CLEMENT @ 2017-09-19 14:25 UTC (permalink / raw) To: Baruch Siach Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA Hi Baruch, On ven., sept. 15 2017, Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> 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 <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> I made a quick test on a mcbin and I confirm that the watchdog is here and usable. Tested-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-19 16:08 ` Baruch Siach 0 siblings, 0 replies; 10+ messages in thread From: Baruch Siach @ 2017-09-19 16:08 UTC (permalink / raw) To: linux-arm-kernel Hi Gregory, On Tue, Sep 19, 2017 at 04:25:58PM +0200, Gregory CLEMENT wrote: > On ven., sept. 15 2017, Baruch Siach <baruch@tkos.co.il> 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 <baruch@tkos.co.il> > > I made a quick test on a mcbin and I confirm that the watchdog is here > and usable. > > Tested-by: Gregory CLEMENT <gregory.clement@free-electrons.com> > and > Applied on mvebu/dt64 This is meant to be a reply to https://patchwork.kernel.org/patch/9947525/, right? Anyway, I see the right patch[1] at mvebu/dt64. Thanks, baruch [1] http://git.infradead.org/linux-mvebu.git/commitdiff/e34ffe32f6e7ae9191d14226ff9d8c0c47400a71 > > --- > > 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>; -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il - ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-19 16:08 ` Baruch Siach 0 siblings, 0 replies; 10+ messages in thread From: Baruch Siach @ 2017-09-19 16:08 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, devicetree-u79uwXL29TY76Z2rM5mHXA, Jason Cooper, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Sebastian Hesselbarth Hi Gregory, On Tue, Sep 19, 2017 at 04:25:58PM +0200, Gregory CLEMENT wrote: > On ven., sept. 15 2017, Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> 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 <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> > > I made a quick test on a mcbin and I confirm that the watchdog is here > and usable. > > Tested-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> > and > Applied on mvebu/dt64 This is meant to be a reply to https://patchwork.kernel.org/patch/9947525/, right? Anyway, I see the right patch[1] at mvebu/dt64. Thanks, baruch [1] http://git.infradead.org/linux-mvebu.git/commitdiff/e34ffe32f6e7ae9191d14226ff9d8c0c47400a71 > > --- > > 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>; -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org - tel: +972.52.368.4656, http://www.tkos.co.il - -- 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] arm64: dt marvell: extend AP806 system controller size 2017-09-19 16:08 ` Baruch Siach @ 2017-09-20 7:51 ` Gregory CLEMENT -1 siblings, 0 replies; 10+ messages in thread From: Gregory CLEMENT @ 2017-09-20 7:51 UTC (permalink / raw) To: linux-arm-kernel Hi Baruch, On mar., sept. 19 2017, Baruch Siach <baruch@tkos.co.il> wrote: > Hi Gregory, > > On Tue, Sep 19, 2017 at 04:25:58PM +0200, Gregory CLEMENT wrote: >> On ven., sept. 15 2017, Baruch Siach <baruch@tkos.co.il> 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 <baruch@tkos.co.il> >> >> I made a quick test on a mcbin and I confirm that the watchdog is here >> and usable. >> >> Tested-by: Gregory CLEMENT <gregory.clement@free-electrons.com> >> and >> Applied on mvebu/dt64 > > This is meant to be a reply to https://patchwork.kernel.org/patch/9947525/, > right? Yes! Sorry for hte confusion. > > Anyway, I see the right patch[1] at mvebu/dt64. > > Thanks, > baruch > > [1] http://git.infradead.org/linux-mvebu.git/commitdiff/e34ffe32f6e7ae9191d14226ff9d8c0c47400a71 > >> > --- >> > 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>; > > -- > http://baruch.siach.name/blog/ ~. .~ Tk Open Systems > =}------------------------------------------------ooO--U--Ooo------------{= > - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il - -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-20 7:51 ` Gregory CLEMENT 0 siblings, 0 replies; 10+ messages in thread From: Gregory CLEMENT @ 2017-09-20 7:51 UTC (permalink / raw) To: Baruch Siach Cc: Andrew Lunn, devicetree-u79uwXL29TY76Z2rM5mHXA, Jason Cooper, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Sebastian Hesselbarth Hi Baruch, On mar., sept. 19 2017, Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> wrote: > Hi Gregory, > > On Tue, Sep 19, 2017 at 04:25:58PM +0200, Gregory CLEMENT wrote: >> On ven., sept. 15 2017, Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> 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 <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> >> >> I made a quick test on a mcbin and I confirm that the watchdog is here >> and usable. >> >> Tested-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> >> and >> Applied on mvebu/dt64 > > This is meant to be a reply to https://patchwork.kernel.org/patch/9947525/, > right? Yes! Sorry for hte confusion. > > Anyway, I see the right patch[1] at mvebu/dt64. > > Thanks, > baruch > > [1] http://git.infradead.org/linux-mvebu.git/commitdiff/e34ffe32f6e7ae9191d14226ff9d8c0c47400a71 > >> > --- >> > 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>; > > -- > http://baruch.siach.name/blog/ ~. .~ Tk Open Systems > =}------------------------------------------------ooO--U--Ooo------------{= > - baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org - tel: +972.52.368.4656, http://www.tkos.co.il - -- 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-19 14:39 ` Gregory CLEMENT 0 siblings, 0 replies; 10+ messages in thread From: Gregory CLEMENT @ 2017-09-19 14:39 UTC (permalink / raw) To: linux-arm-kernel Hi Baruch, On ven., sept. 15 2017, Baruch Siach <baruch@tkos.co.il> wrote: > Extend the container size to 0x2000 to include the gpio controller at > offset 0x1040. > Actually it is a fix as I don't see how we can use the gpio if it is not in the range. So I amended your patch by adding: Fixes: 63dac0f4924b ("arm64: dts: marvell: add gpio support for Armada 7K/8K") Cc: <stable@vger.kernel.org> And I also modified the title to: arm64: dt marvell: Fix AP806 system controller size > While at it, add start address notation to the gpio node name to match > its 'offset' property. > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> Applied on mvebu/fixes 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm64: dt marvell: extend AP806 system controller size @ 2017-09-19 14:39 ` Gregory CLEMENT 0 siblings, 0 replies; 10+ messages in thread From: Gregory CLEMENT @ 2017-09-19 14:39 UTC (permalink / raw) To: Baruch Siach Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA Hi Baruch, On ven., sept. 15 2017, Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> wrote: > Extend the container size to 0x2000 to include the gpio controller at > offset 0x1040. > Actually it is a fix as I don't see how we can use the gpio if it is not in the range. So I amended your patch by adding: Fixes: 63dac0f4924b ("arm64: dts: marvell: add gpio support for Armada 7K/8K") Cc: <stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> And I also modified the title to: arm64: dt marvell: Fix AP806 system controller size > While at it, add start address notation to the gpio node name to match > its 'offset' property. > > Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org> Applied on mvebu/fixes 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-09-20 7:51 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-09-15 7:50 [PATCH] arm64: dt marvell: extend AP806 system controller size Baruch Siach 2017-09-15 7:50 ` Baruch Siach 2017-09-19 14:25 ` Gregory CLEMENT 2017-09-19 14:25 ` Gregory CLEMENT 2017-09-19 16:08 ` Baruch Siach 2017-09-19 16:08 ` Baruch Siach 2017-09-20 7:51 ` Gregory CLEMENT 2017-09-20 7:51 ` Gregory CLEMENT 2017-09-19 14:39 ` Gregory CLEMENT 2017-09-19 14:39 ` Gregory CLEMENT
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.