• Carl-Daniel Hailfinger's avatar
    Unify some probe functions that basically correspond to probe_jedec() · 4e9cebb2
    Carl-Daniel Hailfinger authored
    
    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: default avatarCarl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
    Acked-by: default avatarStefan Reinauer <stepan@coresystems.de>
    4e9cebb2
82802ab.c 5.23 KB