diff options
author | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2013-08-12 22:58:43 +0000 |
---|---|---|
committer | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2013-08-12 22:58:43 +0000 |
commit | c4f44df55fbe624e283ce3db83e1dea6925fd5da (patch) | |
tree | 23f602150eecb4599d675e45ff1a61640953d52e /Documentation | |
parent | 463dd6953eed14930ae9819c7194ea1cc3b8ef40 (diff) | |
download | flashrom-c4f44df55fbe624e283ce3db83e1dea6925fd5da.tar.gz flashrom-c4f44df55fbe624e283ce3db83e1dea6925fd5da.tar.bz2 flashrom-c4f44df55fbe624e283ce3db83e1dea6925fd5da.zip |
Fix verification operation
I broke this in r1702 where I enabled avoidance of the verification step if
we did not modify anything in the erase/write step. The problem is that
all_skipped is initialized to true and hence it would only ever verify if
there have been changes noted in the erase/write step. This obviously
breaks the verification operation (-v/--verify) because there we never
enter the erase/write loop.
The better alternative would be to enable (the implicit) verification
in the write loop and not in cli_classic.c. This would require a bigger
change due to the existance of dont_verify_it. Eventually this is the
right thing to do but not so shortly before a release.
Corresponding to flashrom svn r1707.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions