Hi Chris
-----Original Message----- From: Chris Packham [mailto:judge.packham@gmail.com] Sent: Wednesday, December 06, 2017 9:49 AM To: Ofer Heifetz oferh@marvell.com; Scott Wood oss@buserror.net; Scott Wood scottwood@freescale.com; Tom Rini trini@konsulko.com Cc: u-boot u-boot@lists.denx.de; Nadav Haklai nadavh@marvell.com; Stefan Roese sr@denx.de; Ezequiel Garcia ezequiel@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@marvell.com wrote:
From: Ofer Heifetz oferh@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@marvell.com Tested-by: Ofer Heifetz oferh@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.