[U-Boot] [RFC PATCH v3 01/10] mtd: nand: pxa3xx_nand: Increase initial buffer size

[Ofer Heifetz]

Hi Chris

> -----Original Message-----
> From: Chris Packham [mailto:judge.packham at gmail.com]
> Sent: Wednesday, December 06, 2017 9:49 AM
> To: Ofer Heifetz <oferh at marvell.com>; Scott Wood <oss at buserror.net>;
> Scott Wood <scottwood at freescale.com>; Tom Rini <trini at konsulko.com>
> Cc: u-boot <u-boot at lists.denx.de>; Nadav Haklai <nadavh at marvell.com>;
> Stefan Roese <sr at denx.de>; Ezequiel Garcia
> <ezequiel at vanguardiasur.com.ar>
> Subject: Re: [RFC PATCH v3 01/10] mtd: nand: pxa3xx_nand: Increase initial
> buffer size
> 
> Hi Ofer,
> 
> On Wed, Dec 6, 2017 at 7:55 PM,  <oferh at marvell.com> wrote:
> > From: Ofer Heifetz <oferh at marvell.com>
> >
> > The initial buffer is used for the initial commands used to detect a
> > flash device (STATUS, READID and PARAM).
> >
> > ONFI param page is 256 bytes, and there are three redundant copies to
> > be read. JEDEC param page is 512 bytes, and there are also three
> > redundant copies to be read. Hence this buffer should be at least
> > 512 x 3. This commits rounds the buffer size to 2048.
> >
> > [ Linux commit c16340973fcb6461474a9f811f7f3ff2f946b24c ]
> >
> > Cc: Ezequiel Garcia <ezequiel at vanguardiasur.com.ar>
> > Signed-off-by: Chris Packham <judge.packham at gmail.com>
> > Reviewed-by: Ofer Heifetz <oferh at marvell.com>
> > Tested-by: Ofer Heifetz <oferh at marvell.com>
> 
> Thanks for testing these. Since I sent the initial series I hadn't heard anything
> back (the nand maintainer seems to be inactive).
> 
> Did you manage to rebase this series against u-boot#master? I've just got my
> hands on a DB-88F78460-AMC board I was planning on porting it to upstream
> u-boot when I find some time. It would be another good test platform for
> these changes.

I have worked on u-boot 2017.03 branch for testing this patchset, without this 8bit ECC
Failed to boot ubifs and bootrom gave ECC errors, as I see it the patch from Linux originated by Thomas 
Is mandatory to allow 8bit ECC to work.