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


Groups > linux.kernel > #1339301 > unrolled thread

Re: [PATCH 3/5] ARM: davinci: dm365: Add dma_slave_map to edma

Started bySekhar Nori <nsekhar@ti.com>
First post2016-02-22 11:00 +0100
Last post2016-02-22 11:10 +0100
Articles 2 — 1 participant

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 3/5] ARM: davinci: dm365: Add dma_slave_map to edma Sekhar Nori <nsekhar@ti.com> - 2016-02-22 11:00 +0100
    Re: [PATCH 3/5] ARM: davinci: dm365: Add dma_slave_map to edma Sekhar Nori <nsekhar@ti.com> - 2016-02-22 11:10 +0100

#1339301 — Re: [PATCH 3/5] ARM: davinci: dm365: Add dma_slave_map to edma

FromSekhar Nori <nsekhar@ti.com>
Date2016-02-22 11:00 +0100
SubjectRe: [PATCH 3/5] ARM: davinci: dm365: Add dma_slave_map to edma
Message-ID<r4Vrz-3HO-1@gated-at.bofh.it>
On Tuesday 02 February 2016 06:13 PM, Peter Ujfalusi wrote:
> Provide the dma_slave_map to edma which will allow us to move the drivers
> to the new, simpler dmaengine API and we can remove the DMA resources also
> for the devices.
> 
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>

> FIX: dm365

dropped this "note to self" :)

> ---
>  arch/arm/mach-davinci/dm365.c | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
> 
> diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c
> index 01843fbcc9ea..d26dc2b4339a 100644
> --- a/arch/arm/mach-davinci/dm365.c
> +++ b/arch/arm/mach-davinci/dm365.c
> @@ -17,6 +17,7 @@
>  #include <linux/serial_8250.h>
>  #include <linux/platform_device.h>
>  #include <linux/dma-mapping.h>
> +#include <linux/dmaengine.h>
>  #include <linux/spi/spi.h>
>  #include <linux/platform_data/edma.h>
>  #include <linux/platform_data/gpio-davinci.h>
> @@ -862,9 +863,30 @@ static s8 dm365_queue_priority_mapping[][2] = {
>  	{-1, -1},
>  };
>  
> +static const struct dma_slave_map da365_edma_map[] = {

This should be "dm365_edma_map".

Fixed both locally while applying.

Regards,
Sekhar

[toc] | [next] | [standalone]


#1339313

FromSekhar Nori <nsekhar@ti.com>
Date2016-02-22 11:10 +0100
Message-ID<r4VBi-42I-37@gated-at.bofh.it>
In reply to#1339301
On Monday 22 February 2016 03:28 PM, Sekhar Nori wrote:
> On Tuesday 02 February 2016 06:13 PM, Peter Ujfalusi wrote:
>> Provide the dma_slave_map to edma which will allow us to move the drivers
>> to the new, simpler dmaengine API and we can remove the DMA resources also
>> for the devices.
>>
>> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> 
>> FIX: dm365
> 
> dropped this "note to self" :)
> 
>> ---
>>  arch/arm/mach-davinci/dm365.c | 22 ++++++++++++++++++++++
>>  1 file changed, 22 insertions(+)
>>
>> diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c
>> index 01843fbcc9ea..d26dc2b4339a 100644
>> --- a/arch/arm/mach-davinci/dm365.c
>> +++ b/arch/arm/mach-davinci/dm365.c
>> @@ -17,6 +17,7 @@
>>  #include <linux/serial_8250.h>
>>  #include <linux/platform_device.h>
>>  #include <linux/dma-mapping.h>
>> +#include <linux/dmaengine.h>
>>  #include <linux/spi/spi.h>
>>  #include <linux/platform_data/edma.h>
>>  #include <linux/platform_data/gpio-davinci.h>
>> @@ -862,9 +863,30 @@ static s8 dm365_queue_priority_mapping[][2] = {
>>  	{-1, -1},
>>  };
>>  
>> +static const struct dma_slave_map da365_edma_map[] = {
> 
> This should be "dm365_edma_map".
> 
> Fixed both locally while applying.

Similar fixes needed on patches 2,4,5 as well. Made those too.

Thanks,
Sekhar

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web