[U-Boot-Users] [patch] do not use cmd_reset uninitialized in cfi_flash.c

[Michael Schwingen <michael@schwingen.org>]

> 
> Right. But it will be an improvement to the current implementation, where a 
> random command is written as RESET command. And also an improvement to the 
> fixed AMD/Intel RESET command. So I vote for trying this solution. I'll test 
> on a few of mine platforms too.
> 
> Michael, could please you provide a patch?

Okay - sorry, I did not have time for as much testing as I planned. 

From a short test, it looks like AMD-style flash roms treat *any* unknown
command write as a reset, at least when in CFI Query mode, so issuing the
Intel reset command to AMD-style flashs seems safe (from the small sample I
have), plus the 3-cycle magic sequence should kick the state machine into
the right state even without a reset command. Since the AMD-style flashs
require the unlock sequence for real operation, I chose to try the AMD reset
command first, so that Intel flashs do no see an invalid command prior to
the CFI query.

I have tested the patch on AM29LV320-style flashs from Fujitsu and Macronix,
plus Intel StrataFlash.

cu
Michael