David Hendricks has posted comments on this change. ( https://review.coreboot.org/17950 )
Change subject: Add option (-A, --noverify-all) to not verify the whole chip after write ......................................................................
Patch Set 7: Code-Review+1
(2 comments)
Code looks good, I just have some cosmetic nits.
https://review.coreboot.org/#/c/17950/7/cli_classic.c File cli_classic.c:
PS7, Line 58: don't auto-verify whole chip Instead of saying what it doesn't do, maybe it would be clearer to say what it does, for example "verify included regions only\n"
PS7, Line 58: A I guess I'd prefer 'V' for "verify included". Might be worth asking a few others.