[PATCH v1] spi: davinci: Support DMA transfers larger than 65535 words

2011-03-14 Thread Michael Williamson
The current davinci SPI driver, in DMA mode, is limited to 65535 words for a single transfer. Modify the driver by configuring a 3 dimensional EDMA transfer to support up to 65535x65535 words. Signed-off-by: Michael Williamson Tested-by: Stefano Babic --- changes since v0: - Added spaces

Re: [PATCH] spi: davinci: Support DMA transfers larger than 65535 words

2011-03-14 Thread Michael Williamson
On 3/14/2011 6:25 AM, Sergei Shtylyov wrote: > Hello. > > On 13-03-2011 17:34, Michael Williamson wrote: > >> The current davinci SPI driver, in DMA mode, is limited to 65535 >> words for a single transfer. Modify the driver by configuring a >> 3 dimensional

[PATCH] spi: davinci: Use correct length parameter to dma_map_single calls

2011-03-13 Thread Michael Williamson
The davinci spi driver provides an option to use DMA transfers for data. In the dma_map_single() call, the driver is passing the number of words to be transfered for the mapping size. It should be the number of bytes. Signed-off-by: Michael Williamson --- Note: I only have 8 bit devices to

[PATCH] spi: davinci: Support DMA transfers larger than 65535 words

2011-03-13 Thread Michael Williamson
The current davinci SPI driver, in DMA mode, is limited to 65535 words for a single transfer. Modify the driver by configuring a 3 dimensional EDMA transfer to support up to 65535x65535 words. Signed-off-by: Michael Williamson --- This patch is an attempt to address the issue raised here: http

[PATCH v2] davinci: spi: move event queue parameter to platform data

2011-02-08 Thread Michael Williamson
for spi0 out of resources array and into platform data. Signed-off-by: Michael Williamson Acked-by: Sekhar Nori --- Changes since v1: - Add Sekhar's Ack. - Really fix the typo. This time for sure (blew the format patch on last go around). arch/arm/mach-davinci/dm

[PATCH v1] davinci: spi: move event queue parameter to platform data

2011-02-04 Thread Michael Williamson
for spi0 out of resources array and into platform data. Signed-off-by: Michael Williamson --- Changes since v0: - combined separate platform patches with driver patch to avoid potential regression if doing a bissect. - fixed up typo per comments. arch/arm/mach-davinci/dm355.c

Re: [PATCH 1/3] davinci: spi: move event_q parameter to platform data

2011-02-04 Thread Michael Williamson
Hi Sergei, On 2/4/2011 7:03 AM, Sergei Shtylyov wrote: > Hello. > > On 04-02-2011 3:34, Michael Williamson wrote: > >> For DMA operation, the davinci spi driver needs an eventq number. >> Currently, this number is passed as a IORESOURCE_DMA. This is not >> corre

[PATCH 1/3] davinci: spi: move event_q parameter to platform data

2011-02-03 Thread Michael Williamson
For DMA operation, the davinci spi driver needs an eventq number. Currently, this number is passed as a IORESOURCE_DMA. This is not correct, as the eventq is not a DMA channel. Pass the eventq via the platform data structure instead. Signed-off-by: Michael Williamson --- arch/arm/mach-davinci

[PATCH 3/3] davinci: dm365: move spi0 eventq assignment from resources to platform data

2011-02-03 Thread Michael Williamson
Move the spi0 eventq assignment out of resources array and into platform data. Signed-off-by: Michael Williamson --- arch/arm/mach-davinci/dm365.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c index

[PATCH 2/3] davinci: dm355: move spi0 eventq assignment from resources to platform data

2011-02-03 Thread Michael Williamson
Move the eventq assignment for spi0 out of resources and into platform data. Signed-off-by: Michael Williamson --- arch/arm/mach-davinci/dm355.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-davinci/dm355.c b/arch/arm/mach-davinci/dm355.c index

[PATCH 0/3] spi: davinci: move eventq parameter to platform data

2011-02-03 Thread Michael Williamson
forms have not been tested. Michael Williamson (3): davinci: spi: move event_q parameter to platform data davinci: dm355: move spi0 eventq assignment from resources to platform data davinci: dm365: move spi0 eventq assignment from resources to platform data arch/arm/mach-davinci/dm

Re: [PATCH 00/49] spi: davinci: re-write existing driver

2010-12-11 Thread Michael Williamson
On 11/18/2010 08:14 AM, Nori, Sekhar wrote: > Hi Grant, > > On Thu, Nov 18, 2010 at 17:01:24, Nori, Sekhar wrote: >> On Thu, Nov 18, 2010 at 12:23:24, Nori, Sekhar wrote: >>> >>> Here is the pull request: >>> >>> The following changes since commit 0a5b871ea4c6bfb2723ac2ffc7ef5c32452abb89: >>> Li

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-10-11 Thread Michael Williamson
On 10/11/2010 12:57 PM, Nori, Sekhar wrote: > Hi Mike, > > On Sat, Oct 09, 2010 at 18:25:54, Michael Williamson wrote: >> On 10/08/2010 03:22 PM, Michael Williamson wrote: > >>> On 10/6/2010 11:37 AM, Nori, Sekhar wrote: > >>>> On Mon, Sep 20,

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-10-11 Thread Michael Williamson
On 10/11/2010 12:57 PM, Nori, Sekhar wrote: > Hi Mike, > > On Sat, Oct 09, 2010 at 18:25:54, Michael Williamson wrote: >> On 10/08/2010 03:22 PM, Michael Williamson wrote: > >>> On 10/6/2010 11:37 AM, Nori, Sekhar wrote: > >>>> On Mon, Sep 20,

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-10-09 Thread Michael Williamson
On 10/08/2010 03:22 PM, Michael Williamson wrote: > Hi Sekhar, > > On 10/6/2010 11:37 AM, Nori, Sekhar wrote: > >> Hi Mike, >> >> On Mon, Sep 20, 2010 at 23:12:53, Michael Williamson wrote: >>> >>> Let me know if you want testing on the DMA porti

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-10-08 Thread Michael Williamson
Hi Sekhar, On 10/6/2010 11:37 AM, Nori, Sekhar wrote: > Hi Mike, > > On Mon, Sep 20, 2010 at 23:12:53, Michael Williamson wrote: >> >> Let me know if you want testing on the DMA portion of the patch (when your >> ready, of course). > > I just finished pushi

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-10-06 Thread Michael Williamson
Hi Sekhar, On 10/06/2010 11:37 AM, Nori, Sekhar wrote: > Hi Mike, > > On Mon, Sep 20, 2010 at 23:12:53, Michael Williamson wrote: >> >> Let me know if you want testing on the DMA portion of the patch (when your >> ready, of course). > > I just finished pushi

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-09-20 Thread Michael Williamson
On 9/20/2010 11:08 AM, Nori, Sekhar wrote: > Hi Michael, > > On Sat, Sep 18, 2010 at 18:38:13, Michael Williamson wrote: >> >> * I'm not sure I understand why there is a "intr_line" field in the platform >> data and then a possibility to configure

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-09-18 Thread Michael Williamson
[...] > I had looking at the original spi code. Nice work, Nori, for break the patch s/Nori/Sekhar Sorry about that, Sekhar. -Mike -- Start uncovering the many advantages of virtual appliances and start using them to

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-09-18 Thread Michael Williamson
On 09/14/2010 09:26 AM, Nori, Sekhar wrote: > Hi Mike, > > On Tue, Sep 14, 2010 at 18:40:56, Michael Williamson wrote: >> On 9/14/2010 3:14 AM, Nori, Sekhar wrote: >> >>> On Tue, Sep 14, 2010 at 11:34:59, Caglar Akyuz wrote: >>> >>>> Yes, your pa

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-09-14 Thread Michael Williamson
On 9/14/2010 3:14 AM, Nori, Sekhar wrote: > On Tue, Sep 14, 2010 at 11:34:59, Caglar Akyuz wrote: > >> Yes, your patches seems ok and mostly complete. I'm waiting for hardware to >> test those, will let you know when I do some testing. > > Thanks! To save time, I haven't tested these patches mys

Re: [PATCH v5 1/1] davinci: spi: replace existing driver

2010-09-13 Thread Michael Williamson
Hi Sekhar, On 08/23/2010 05:30 AM, Nori, Sekhar wrote: > On Mon, Aug 23, 2010 at 14:24:10, Caglar Akyuz wrote: >> On Monday 23 August 2010 07:28:48 am Nori, Sekhar wrote: >>> Hi Caglar, >>> >> >> Hi, >> >> [...] >> >>> I have been spending time on this (admittedly on and off) since I wrote >>> th