Attention is currently required from: Simon Buhrow, Nico Huber, Thomas Heijligen, Paul Menzel.
View Change
7 comments:
File flashrom.c:
Patch Set #43, Line 1054: assert(!ret);
``` […]
Done
Patch Set #43, Line 1055:
//verify erase
ret = check_erased_range(flashctx, start_addr, block_len);
assert(!ret);
//if erase fail??
//
This is not needed, there is a verify command at the end.
Done
Patch Set #43, Line 1065: msg_cinfo
s/msg_cdbg/
Done
Patch Set #43, Line 1076: assert(!ret);
``` […]
Done
Patch Set #43, Line 1078:
ret = verify_range(flashctx, newcontents + i, i, flashctx->chip->page_size);
assert(!ret);
This is not needed, there is a verify command at the end.
Done
Patch Set #43, Line 1082: msg_cinfo
s/msg_cdbg/
Done
Patch Set #43, Line 1088: assert(!ret);
``` […]
Done
To view, visit change 66104. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I29e3f2bd796759794184b125741a5abaac6f3ce8
Gerrit-Change-Number: 66104
Gerrit-PatchSet: 45
Gerrit-Owner: Aarya <aarya.chaumal@gmail.com>
Gerrit-Reviewer: Simon Buhrow
Gerrit-Reviewer: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Attention: Simon Buhrow
Gerrit-Attention: Nico Huber <nico.h@gmx.de>
Gerrit-Attention: Thomas Heijligen <src@posteo.de>
Gerrit-Attention: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Comment-Date: Thu, 08 Sep 2022 13:06:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Thomas Heijligen <src@posteo.de>
Gerrit-MessageType: comment