Re: [PATCH FIX] spi: bcm-qspi: switch back to reading flash using smaller chunks

[Kamal Dasu <kdasu.kdev@xxxxxxxxx>]

"So you don't care about what the actual problem is  or do you confirm
that the IP does not support 512B chunk reads and switching to 512B
was
buggy in the first place"

I will look into it for sure with the specific SoC being used.
Meanwhile I think changing back to 256 would be right thing to do.

Kamal
On Sat, Oct 13, 2018 at 1:45 AM Boris Brezillon
<boris.brezillon@xxxxxxxxxxx> wrote:
>
> Hi Kamal,
>
> On Fri, 12 Oct 2018 18:06:06 -0400
> Kamal Dasu <kdasu.kdev@xxxxxxxxx> wrote:
>
> > Rafal,
> >
> > "switching back to 256bytes is not helping us understanding what's
> > going on here.. "
> >
> > The read take a different path and use hw acceleration. . It is to do
> > with the BSPI_RAF buffer size. I think  it's safe to keep it at 256
> > and will not impact performance either.
>
> So you don't care about what the actual problem is, or do you confirm
> that the IP does not support 512B chunk reads and switching to 512B was
> buggy in the first place?
>
> Boris