[U-Boot-Users] Update for 5329
Stefan Roese
sr at denx.de
Thu Aug 2 07:23:21 CEST 2007
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
=====================================================================
More information about the U-Boot
mailing list