[U-Boot-Users] Patch for drivers/cfi_flash.c; properly detect write errors.
Wolfgang Denk
wd at denx.de
Tue Feb 28 17:36:55 CET 2006
In message <430B8EC1.6090406 at lucent.com> you wrote:
>
> It seems that there is a minor problem in cfi_flash.c that prevents it
> from detecting that a flash write error has occurred.
>
> The first change in the attached patch is trivial, it just makes the
> strings output for a protected and a non-protected sector have the same
> length so the output looks prettier..
>
> The second change is in flash_full_status_check(). We have just called
> flash_status_check() and then did a switch() on the vendor. For intel,
> we *should* check if there are any error bits if the previous call
> returned ERR_OK (Otherwise we will have output an error message in
> flash_status_check() already.) The original code would only check for
> error bits if flash_status_check() returns ERR_TIMEOUT.
Applied. Thanks. But please provide a proper CHANGELOG entry next
time.
Best regards,
Wolfgang Denk
--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Any sufficiently advanced technology is indistinguishable from magic.
Clarke's Third Law - _Profiles of the Future_ (1962; rev. 1973)
``Hazards of Prophecy: The Failure of Imagination''
More information about the U-Boot
mailing list