All of lore.kernel.org
 help / color / mirror / Atom feed
* [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 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

* [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

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.