Attention is currently required from: Nikolai Artemiev, Stefan Reinauer.
Vasily Galkin has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/77530?usp=email )
Change subject: flashchips: Add WP features for Winbond W25X20
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
This "patch series"="relation chain" is currently shown as "Merge conflict" in Gerrit. They are "spurious" - unrelated chip change in flashchips.c, easily handled by autorebasing (tested locally).
Should I rebase this on master tip, or it is fine to keep in in current state?
I'm new to Gerrit, so I'm not sure if rebasing is "harmless" or would initate new review round (that wound be unneeded in this case, since no manual tunings would be done during rebasing)
--
To view, visit
https://review.coreboot.org/c/flashrom/+/77530?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I82c0cc52ca2a78d27f513234cc12d3e09d8905a5
Gerrit-Change-Number: 77530
Gerrit-PatchSet: 1
Gerrit-Owner: Vasily Galkin
Gerrit-Reviewer: Anastasia Klimchuk
aklm@chromium.org
Gerrit-Reviewer: Nikolai Artemiev
nartemiev@google.com
Gerrit-Reviewer: Sergii Dmytruk
sergii.dmytruk@3mdeb.com
Gerrit-Reviewer: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Attention: Nikolai Artemiev
nartemiev@google.com
Gerrit-Comment-Date: Fri, 01 Sep 2023 12:49:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment