[U-Boot-Users] Update for 5329

[Stefan Roese <sr@denx.de>]

Hi Craig,

On Wednesday 01 August 2007, Craig Day wrote:
> I've ported u-boot 1.1.6 over to the LPC3180 (arm926ejs core) and have
> included NAND support.  The nand_write_ecc function in
> drivers/nand/nand_base.c appears to be writing pages correctly, but the
> pointer to the source data in memory for nand_verify_pages does not seem to
> be updated correctly. Specifically, line 1707 in nand_base.c contains:
> this->data_poi = bufstart; However, the problem is that bufstart is never
> updated, and always points to the beginning of the data source, causing all
> nand_page_verify calls to compare the newest written pages in NAND flash to
> the beginning of the data source. After the first block NAND writing fails
> due to nand_verify_pages failing. To fix this I added in:
> bufstart += (1 << this->phys_erase_shift);

Thanks for the detailed explanation. But even better would be to send a patch 
(best with a signed-off line etc.). A patch sometimes explains a problem 
betten than such an explanation. And if you are correct and it fixes a real 
problem, the patch could be applied directly.

> Have you seen this problem? Or has anyone else run into this problem? Or is
> there something I'm missing?

We are using the current NAND driver successfully on multiple boards.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================