Hi Chris > -----Original Message----- > From: Chris Packham [mailto:judge.pack...@gmail.com] > Sent: Wednesday, December 06, 2017 9:49 AM > To: Ofer Heifetz <of...@marvell.com>; Scott Wood <o...@buserror.net>; > Scott Wood <scottw...@freescale.com>; Tom Rini <tr...@konsulko.com> > Cc: u-boot <u-boot@lists.denx.de>; Nadav Haklai <nad...@marvell.com>; > Stefan Roese <s...@denx.de>; Ezequiel Garcia > <ezequ...@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, <of...@marvell.com> wrote: > > From: Ofer Heifetz <of...@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 <of...@marvell.com> > > Tested-by: Ofer Heifetz <of...@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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
