Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Vinod Koul]

On Tue, Apr 26, 2016 at 09:28:11AM +0200, Thomas Petazzoni wrote:
> Hello Vinod,
> 
> On Tue, 26 Apr 2016 09:05:36 +0530, Vinod Koul wrote:
> > On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> > > From: Marcin Wojtas 
> > > 
> > > Armada 3700 SoC comprise one dual-channel XOR engine and this
> > > patch adds its according representation.
> > 
> > This need an ACK from ARM folks..
> 
> Well, this patch has the SoB from Gregory Clement, who is one of the
> maintainers of the ARM Marvell Armada platforms. And he should be the
> one taking this patch in his tree (especially since a lot of others
> Armada 3700 DT changes are going to be made).
> 
> So I guess that the intent is that you take patches 1 to 4, and patch 5
> will be taken by Gregory.

sounds good to me then

thanks
-- 
~Vinod

Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Vinod Koul]

On Tue, Apr 26, 2016 at 09:28:11AM +0200, Thomas Petazzoni wrote:
> Hello Vinod,
> 
> On Tue, 26 Apr 2016 09:05:36 +0530, Vinod Koul wrote:
> > On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> > > From: Marcin Wojtas 
> > > 
> > > Armada 3700 SoC comprise one dual-channel XOR engine and this
> > > patch adds its according representation.
> > 
> > This need an ACK from ARM folks..
> 
> Well, this patch has the SoB from Gregory Clement, who is one of the
> maintainers of the ARM Marvell Armada platforms. And he should be the
> one taking this patch in his tree (especially since a lot of others
> Armada 3700 DT changes are going to be made).
> 
> So I guess that the intent is that you take patches 1 to 4, and patch 5
> will be taken by Gregory.

sounds good to me then

thanks
-- 
~Vinod

Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Thomas Petazzoni]

Hello Vinod,

On Tue, 26 Apr 2016 09:05:36 +0530, Vinod Koul wrote:
> On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> > From: Marcin Wojtas 
> > 
> > Armada 3700 SoC comprise one dual-channel XOR engine and this
> > patch adds its according representation.
> 
> This need an ACK from ARM folks..

Well, this patch has the SoB from Gregory Clement, who is one of the
maintainers of the ARM Marvell Armada platforms. And he should be the
one taking this patch in his tree (especially since a lot of others
Armada 3700 DT changes are going to be made).

So I guess that the intent is that you take patches 1 to 4, and patch 5
will be taken by Gregory.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Thomas Petazzoni]

Hello Vinod,

On Tue, 26 Apr 2016 09:05:36 +0530, Vinod Koul wrote:
> On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> > From: Marcin Wojtas 
> > 
> > Armada 3700 SoC comprise one dual-channel XOR engine and this
> > patch adds its according representation.
> 
> This need an ACK from ARM folks..

Well, this patch has the SoB from Gregory Clement, who is one of the
maintainers of the ARM Marvell Armada platforms. And he should be the
one taking this patch in his tree (especially since a lot of others
Armada 3700 DT changes are going to be made).

So I guess that the intent is that you take patches 1 to 4, and patch 5
will be taken by Gregory.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Vinod Koul]

On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> From: Marcin Wojtas 
> 
> Armada 3700 SoC comprise one dual-channel XOR engine and this
> patch adds its according representation.

This need an ACK from ARM folks..

> 
> Signed-off-by: Marcin Wojtas 
> Signed-off-by: Gregory CLEMENT 
> ---
>  arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 13 +
>  1 file changed, 13 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi 
> b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> index ba9df7ff2a72..932783d82ea9 100644
> --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> @@ -112,6 +112,19 @@
>   status = "disabled";
>   };
>  
> + xor@60900 {
> + compatible = "marvell,armada-3700-xor";
> + reg = <0x60900 0x100
> +0x60b00 0x100>;
> +
> + xor10 {
> + interrupts =  IRQ_TYPE_LEVEL_HIGH>;
> + };
> + xor11 {
> + interrupts =  IRQ_TYPE_LEVEL_HIGH>;
> + };
> + };
> +
>   sata@e {
>   compatible = "marvell,armada-3700-ahci";
>   reg = <0xe 0x2000>;
> -- 
> 2.5.0
> 

-- 
~Vinod

Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Vinod Koul]

On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> From: Marcin Wojtas 
> 
> Armada 3700 SoC comprise one dual-channel XOR engine and this
> patch adds its according representation.

This need an ACK from ARM folks..

> 
> Signed-off-by: Marcin Wojtas 
> Signed-off-by: Gregory CLEMENT 
> ---
>  arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 13 +
>  1 file changed, 13 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi 
> b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> index ba9df7ff2a72..932783d82ea9 100644
> --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> @@ -112,6 +112,19 @@
>   status = "disabled";
>   };
>  
> + xor@60900 {
> + compatible = "marvell,armada-3700-xor";
> + reg = <0x60900 0x100
> +0x60b00 0x100>;
> +
> + xor10 {
> + interrupts =  IRQ_TYPE_LEVEL_HIGH>;
> + };
> + xor11 {
> + interrupts =  IRQ_TYPE_LEVEL_HIGH>;
> + };
> + };
> +
>   sata@e {
>   compatible = "marvell,armada-3700-ahci";
>   reg = <0xe 0x2000>;
> -- 
> 2.5.0
> 

-- 
~Vinod

[PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Gregory CLEMENT]

From: Marcin Wojtas 

Armada 3700 SoC comprise one dual-channel XOR engine and this
patch adds its according representation.

Signed-off-by: Marcin Wojtas 
Signed-off-by: Gregory CLEMENT 
---
 arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 13 +
 1 file changed, 13 insertions(+)

diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi 
b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
index ba9df7ff2a72..932783d82ea9 100644
--- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
@@ -112,6 +112,19 @@
status = "disabled";
};
 
+   xor@60900 {
+   compatible = "marvell,armada-3700-xor";
+   reg = <0x60900 0x100
+  0x60b00 0x100>;
+
+   xor10 {
+   interrupts = ;
+   };
+   xor11 {
+   interrupts = ;
+   };
+   };
+
sata@e {
compatible = "marvell,armada-3700-ahci";
reg = <0xe 0x2000>;
-- 
2.5.0

[PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC

[Gregory CLEMENT]

From: Marcin Wojtas 

Armada 3700 SoC comprise one dual-channel XOR engine and this
patch adds its according representation.

Signed-off-by: Marcin Wojtas 
Signed-off-by: Gregory CLEMENT 
---
 arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 13 +
 1 file changed, 13 insertions(+)

diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi 
b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
index ba9df7ff2a72..932783d82ea9 100644
--- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
@@ -112,6 +112,19 @@
status = "disabled";
};
 
+   xor@60900 {
+   compatible = "marvell,armada-3700-xor";
+   reg = <0x60900 0x100
+  0x60b00 0x100>;
+
+   xor10 {
+   interrupts = ;
+   };
+   xor11 {
+   interrupts = ;
+   };
+   };
+
sata@e {
compatible = "marvell,armada-3700-ahci";
reg = <0xe 0x2000>;
-- 
2.5.0