Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1387122 > unrolled thread

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

Started byVinod Koul <vinod.koul@intel.com>
First post2016-04-26 05:40 +0200
Last post2016-04-26 18:30 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700  SoC Vinod Koul <vinod.koul@intel.com> - 2016-04-26 05:40 +0200
    Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada  3700 SoC Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2016-04-26 09:30 +0200
      Re: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700  SoC Vinod Koul <vinod.koul@intel.com> - 2016-04-26 18:30 +0200

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

FromVinod Koul <vinod.koul@intel.com>
Date2016-04-26 05:40 +0200
SubjectRe: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC
Message-ID<rs20V-By-3@gated-at.bofh.it>
On Thu, Apr 14, 2016 at 06:53:38PM +0200, Gregory CLEMENT wrote:
> From: Marcin Wojtas <mw@semihalf.com>
> 
> 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 <mw@semihalf.com>
> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
> ---
>  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 = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>;
> +				};
> +				xor11 {
> +					interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>;
> +				};
> +			};
> +
>  			sata@e0000 {
>  				compatible = "marvell,armada-3700-ahci";
>  				reg = <0xe0000 0x2000>;
> -- 
> 2.5.0
> 

-- 
~Vinod

[toc] | [next] | [standalone]


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

FromThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Date2016-04-26 09:30 +0200
SubjectRe: [PATCH v3 5/5] arm64: dts: marvell: add XOR node for Armada 3700 SoC
Message-ID<rs5Bv-3tR-15@gated-at.bofh.it>
In reply to#1387122
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 <mw@semihalf.com>
> > 
> > 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

[toc] | [prev] | [next] | [standalone]


#1387679

FromVinod Koul <vinod.koul@intel.com>
Date2016-04-26 18:30 +0200
Message-ID<rse27-2io-23@gated-at.bofh.it>
In reply to#1387186
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 <mw@semihalf.com>
> > > 
> > > 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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web