From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH RESEND 3/4] ARM: mvebu: remove unneeded dmacap, * properties in Device Trees
Date: Thu, 10 Sep 2015 11:00:54 +0200 [thread overview]
Message-ID: <87d1xq8x89.fsf@free-electrons.com> (raw)
In-Reply-To: <558169F2.2070608@free-electrons.com> (Gregory CLEMENT's message of "Wed, 17 Jun 2015 14:37:06 +0200")
Hi Thomas,
On mer., juin 17 2015, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
> Hi Thomas,
>
> On 17/06/2015 14:12, Thomas Petazzoni wrote:
>> The dmacap,* properties are now ignored by the mv_xor driver, who
>> knows what are the capabilities of the hardware. For the DMA_XOR and
>> DMA_MEMCPY capabilities, there are no changes: they were anyway
>> mentionned in all DTs. The DMA_MEMSET operation (enabled by
>> dmacap,memset) was ignored since the removal of DMA_MEMSET support
>> from mv_xor. Also, now the DMA_INTERRUPT capability is enabled for all
>> SoCs.
>>
>> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>
> Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
>
> I will wait for that 'dmaengine: mv_xor: remove support for dmacap,* DT
> properties' was applied before applying it.
Actually this patch does not apply anymore on recent kernel. Could you
rebase it on 4.3-rc1 when it will be released and then submit it again?
Thanks,
Gregory
>
>
> Thanks,
>
> Gregory
>
>> ---
>> Note: this patch should only be applied once 'dmaengine: mv_xor:
>> remove support for dmacap,* DT properties' is applied.
>> ---
>> arch/arm/boot/dts/armada-370.dtsi | 10 ----------
>> arch/arm/boot/dts/armada-375.dtsi | 10 ----------
>> arch/arm/boot/dts/armada-38x.dtsi | 10 ----------
>> arch/arm/boot/dts/armada-39x.dtsi | 10 ----------
>> arch/arm/boot/dts/armada-xp.dtsi | 10 ----------
>> arch/arm/boot/dts/dove.dtsi | 8 --------
>> 6 files changed, 58 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/armada-370.dtsi b/arch/arm/boot/dts/armada-370.dtsi
>> index 00b50db5..cb68579 100644
>> --- a/arch/arm/boot/dts/armada-370.dtsi
>> +++ b/arch/arm/boot/dts/armada-370.dtsi
>> @@ -278,14 +278,9 @@
>>
>> xor00 {
>> interrupts = <51>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor01 {
>> interrupts = <52>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> @@ -297,14 +292,9 @@
>>
>> xor10 {
>> interrupts = <94>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor11 {
>> interrupts = <95>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>> };
>> diff --git a/arch/arm/boot/dts/armada-375.dtsi b/arch/arm/boot/dts/armada-375.dtsi
>> index f076ff8..ebee84a 100644
>> --- a/arch/arm/boot/dts/armada-375.dtsi
>> +++ b/arch/arm/boot/dts/armada-375.dtsi
>> @@ -476,14 +476,9 @@
>>
>> xor00 {
>> interrupts = <GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor01 {
>> interrupts = <GIC_SPI 23 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> @@ -496,14 +491,9 @@
>>
>> xor10 {
>> interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor11 {
>> interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi
>> index 218a2ac..1710523 100644
>> --- a/arch/arm/boot/dts/armada-38x.dtsi
>> +++ b/arch/arm/boot/dts/armada-38x.dtsi
>> @@ -456,14 +456,9 @@
>>
>> xor00 {
>> interrupts = <GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor01 {
>> interrupts = <GIC_SPI 23 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> @@ -476,14 +471,9 @@
>>
>> xor10 {
>> interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor11 {
>> interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> diff --git a/arch/arm/boot/dts/armada-39x.dtsi b/arch/arm/boot/dts/armada-39x.dtsi
>> index ecd1318..9c510d8 100644
>> --- a/arch/arm/boot/dts/armada-39x.dtsi
>> +++ b/arch/arm/boot/dts/armada-39x.dtsi
>> @@ -331,14 +331,9 @@
>>
>> xor00 {
>> interrupts = <GIC_SPI 22 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor01 {
>> interrupts = <GIC_SPI 23 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> @@ -351,14 +346,9 @@
>>
>> xor10 {
>> interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor11 {
>> interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> diff --git a/arch/arm/boot/dts/armada-xp.dtsi b/arch/arm/boot/dts/armada-xp.dtsi
>> index 013d63f..9c46054 100644
>> --- a/arch/arm/boot/dts/armada-xp.dtsi
>> +++ b/arch/arm/boot/dts/armada-xp.dtsi
>> @@ -209,14 +209,9 @@
>>
>> xor10 {
>> interrupts = <51>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor11 {
>> interrupts = <52>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>>
>> @@ -229,14 +224,9 @@
>>
>> xor00 {
>> interrupts = <94>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> xor01 {
>> interrupts = <95>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> - dmacap,memset;
>> };
>> };
>> };
>> diff --git a/arch/arm/boot/dts/dove.dtsi b/arch/arm/boot/dts/dove.dtsi
>> index 9ad8295..e71c552 100644
>> --- a/arch/arm/boot/dts/dove.dtsi
>> +++ b/arch/arm/boot/dts/dove.dtsi
>> @@ -261,14 +261,10 @@
>>
>> channel0 {
>> interrupts = <39>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>>
>> channel1 {
>> interrupts = <40>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> };
>>
>> @@ -281,14 +277,10 @@
>>
>> channel0 {
>> interrupts = <42>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>>
>> channel1 {
>> interrupts = <43>;
>> - dmacap,memcpy;
>> - dmacap,xor;
>> };
>> };
>>
>>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2015-09-10 9:00 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-17 12:12 [PATCH RESEND 0/4] dmaengine: mv_xor improvements Thomas Petazzoni
2015-06-17 12:12 ` [PATCH RESEND 1/4] dmaengine: mv_xor: add suspend/resume support Thomas Petazzoni
2015-06-17 12:12 ` [PATCH RESEND 2/4] dmaengine: mv_xor: remove support for dmacap, * DT properties Thomas Petazzoni
2015-06-17 12:12 ` [PATCH RESEND 3/4] ARM: mvebu: remove unneeded dmacap, * properties in Device Trees Thomas Petazzoni
2015-06-17 12:37 ` Gregory CLEMENT
2015-09-10 9:00 ` Gregory CLEMENT [this message]
2015-06-17 12:12 ` [PATCH RESEND 4/4] ARM: mvebu: use armada-380-xor on Armada 38x and 39x Thomas Petazzoni
2015-06-17 12:35 ` Gregory CLEMENT
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87d1xq8x89.fsf@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.