Edward O'Callaghan submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: build bot (Jenkins): Verified Sam McNally: Looks good to me, approved
flashrom.c: Trivial code style fix

Change-Id: Iacdb62067a8d22261d4eabe73ad96168eb11417c
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/72968
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sam McNally <sammc@google.com>
---
M flashrom.c
1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/flashrom.c b/flashrom.c
index d17e04b..62e5757 100644
--- a/flashrom.c
+++ b/flashrom.c
@@ -1282,8 +1282,7 @@
ret = 1;
}
}
- if(curr_eraseblock_count > prev_eraseblock_count)
- {
+ if (curr_eraseblock_count > prev_eraseblock_count) {
msg_gerr("ERROR: Flash chip %s erase function %i is not "
"in order. Please report a bug at flashrom@flashrom.org\n",
chip->name, k);

To view, visit change 72968. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Iacdb62067a8d22261d4eabe73ad96168eb11417c
Gerrit-Change-Number: 72968
Gerrit-PatchSet: 3
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Sam McNally <sammc@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged