[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