Attention is currently required from: Nikolai Artemiev, Stefan Reinauer, rogeryou.
View Change
2 comments:
Patchset:
File flashchips.c:
Patch Set #2, Line 10308: STATUS2
Should this be CONFIG ? TB bit is defined in configuration register (Table 8).
I hope I am not mistaken.
Commit message says you tested WP, could you maybe give a detailed list of WP commands that you tested?
To view, visit change 78142. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I43de9ed123b9736c04d070754bcf9c32be5697ad
Gerrit-Change-Number: 78142
Gerrit-PatchSet: 2
Gerrit-Owner: rogeryou
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: rogeryou
Gerrit-Attention: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Attention: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Comment-Date: Sat, 30 Sep 2023 08:40:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment