diff options
author | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2009-09-05 01:16:30 +0000 |
---|---|---|
committer | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2009-09-05 01:16:30 +0000 |
commit | f3ec3d0f16e7e42bf0ab5d85fc78d9d140bdb045 (patch) | |
tree | 2d17169c2f7f720fc40ac6184b15d46572d3086f /flashchips.c | |
parent | b9e89f150c9d221c77ac2fe524996eef5b7b79ca (diff) | |
download | flashrom-f3ec3d0f16e7e42bf0ab5d85fc78d9d140bdb045.zip flashrom-f3ec3d0f16e7e42bf0ab5d85fc78d9d140bdb045.tar.gz |
Quite a few probe functions in flashrom are copies of probe_jedec with additional lock bit printing or other glue
Make them call probe_jedec instead. Use the correct reset sequence for 82802AB.
Detailed explanation: The reset sequence before ID reading was correct,
so ID always worked. But the reset sequence after ID reading was a copy-paste
leftover from probe_jedec and didn't have any effect. I dug up flash_and_burn
from the freebios-v1 tree and found out that 82802ab.c was indeed a copy
of jedec.c with lots of experimental unannotated #if 0 and #if 1. About the
wait_82802ab change: Before the patch, wait_82802ab entered read status mode,
switched to ID mode, then tried an incorrect and unsupported JEDEC command
to exit ID mode. Nobody ever saw that this failed because all subsequent
function calls had the correct reset sequence at the beginning. With the patch,
wait_82802ab enters read status mode, then switches back to read mode with
the official reset command.
Corresponding to flashrom svn r717.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
Diffstat (limited to 'flashchips.c')
0 files changed, 0 insertions, 0 deletions