2015-04-30 16:46:10

by Thomas.Betker

[permalink] [raw]
Subject: Re: [PATCH 0/3] Using SPI NOR flah on sunxi.

Hello Michal:

> I tried to connect a SPI NOR flash to my sunxi board and due to the
current
> sunxi SPI driver limitations it does not work.
>
> The SPI driver returns an error when more than 64 bytes are
> transferred at once
> due to lack of DMA support.

Wouldn't it be easier to fix the SPI driver to handle transfers larger
than 64 bytes, filling and draining the FIFO multiple times if neccessary?
(As far as I can tell, most SPI drivers do this.)

Just asking,
Thomas


2015-04-30 16:57:02

by Michal Suchanek

[permalink] [raw]
Subject: Re: [PATCH 0/3] Using SPI NOR flah on sunxi.

On 30 April 2015 at 18:30, <[email protected]> wrote:
> Hello Michal:
>
>> I tried to connect a SPI NOR flash to my sunxi board and due to the
> current
>> sunxi SPI driver limitations it does not work.
>>
>> The SPI driver returns an error when more than 64 bytes are
>> transferred at once
>> due to lack of DMA support.
>
> Wouldn't it be easier to fix the SPI driver to handle transfers larger
> than 64 bytes, filling and draining the FIFO multiple times if neccessary?
> (As far as I can tell, most SPI drivers do this.)
>

Yes, the intent is to fix this by adding dma support to the driver, eventually.

The patch might be still useful for other hardware with developing SPI support.

Thanks

Michal

2015-04-30 20:58:52

by Marek Vasut

[permalink] [raw]
Subject: Re: [PATCH 0/3] Using SPI NOR flah on sunxi.

On Thursday, April 30, 2015 at 06:56:18 PM, Michal Suchanek wrote:
> On 30 April 2015 at 18:30, <[email protected]> wrote:
> > Hello Michal:
> >> I tried to connect a SPI NOR flash to my sunxi board and due to the
> >
> > current
> >
> >> sunxi SPI driver limitations it does not work.
> >>
> >> The SPI driver returns an error when more than 64 bytes are
> >> transferred at once
> >> due to lack of DMA support.
> >
> > Wouldn't it be easier to fix the SPI driver to handle transfers larger
> > than 64 bytes, filling and draining the FIFO multiple times if
> > neccessary? (As far as I can tell, most SPI drivers do this.)
>
> Yes, the intent is to fix this by adding dma support to the driver,
> eventually.
>
> The patch might be still useful for other hardware with developing SPI
> support.

Please just fix the controller driver to correctly handle arbitrary transfer
lengths.

Best regards,
Marek Vasut

2015-05-20 23:54:59

by Brian Norris

[permalink] [raw]
Subject: Re: [PATCH 0/3] Using SPI NOR flah on sunxi.

On Thu, Apr 30, 2015 at 08:34:36PM +0200, Marek Vasut wrote:
> On Thursday, April 30, 2015 at 06:56:18 PM, Michal Suchanek wrote:
> > On 30 April 2015 at 18:30, <[email protected]> wrote:
> > > Hello Michal:
> > >> I tried to connect a SPI NOR flash to my sunxi board and due to the
> > >
> > > current
> > >
> > >> sunxi SPI driver limitations it does not work.
> > >>
> > >> The SPI driver returns an error when more than 64 bytes are
> > >> transferred at once
> > >> due to lack of DMA support.
> > >
> > > Wouldn't it be easier to fix the SPI driver to handle transfers larger
> > > than 64 bytes, filling and draining the FIFO multiple times if
> > > neccessary? (As far as I can tell, most SPI drivers do this.)
> >
> > Yes, the intent is to fix this by adding dma support to the driver,
> > eventually.
> >
> > The patch might be still useful for other hardware with developing SPI
> > support.
>
> Please just fix the controller driver to correctly handle arbitrary transfer
> lengths.

Just noticed this. Yes, that would definitely be a better option!

Brian