Attention is currently required from: Nikolai Artemiev, Peter Marheine, Thomas Heijligen.

Peter Marheine uploaded patch set #7 to this change.

View Change

The following approvals got outdated and were removed: Verified+1 by build bot (Jenkins)

tests/erase: record the opcode for each erase

This allows tests to verify that the correct opcode is used when
erasing, which is required to unit-test the fix to issue #525 where in
some situations an incorrect erase opcode will be used.

BUG=https://ticket.coreboot.org/issues/525

Change-Id: I3983fe42c2e7f06668a1bd20d2db7fafa93b8043
Signed-off-by: Peter Marheine <pmarheine@chromium.org>
---
M flashrom.c
M include/flash.h
M tests/chip.c
M tests/erase_func_algo.c
M tests/meson.build
5 files changed, 157 insertions(+), 58 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/51/82251/7

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

Gerrit-MessageType: newpatchset
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I3983fe42c2e7f06668a1bd20d2db7fafa93b8043
Gerrit-Change-Number: 82251
Gerrit-PatchSet: 7
Gerrit-Owner: Peter Marheine <pmarheine@chromium.org>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Reviewer: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Thomas Heijligen <src@posteo.de>
Gerrit-Attention: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Attention: Peter Marheine <pmarheine@chromium.org>