Attention is currently required from: Angel Pons, Nikolai Artemiev, Stefan Reinauer, mans0n+coreboot@gorani.run.
View Change
2 comments:
Patchset:
Patch Set #1:
Hi, […]
Sungbo, thank you so much for your contribution, sorry that your patch was lost and hanging for so long.
We had some structural changes in flashchips.c file since the time you sent the patch. So I made the changes to your chip definition and rebased your patch on head.
You did it right with reusing model_id of ST_M45PE80 and adding a comment.
File flashchips.c:
Patch Set #3, Line 20776: SPI_PRETTYPRINT_STATUS_REGISTER_BP2_SRWD
I am not entirely sure about this one, we have 4 different functions to prettyprint with BP2 and it would be ideal if another reviewer checks this is correct function. Thank you!
Initial patch had plain prettyprint, but I think we can use one of BP2 ones?
To view, visit change 63516. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib51225426d8d1a381d45af3574e5ba2bf02837aa
Gerrit-Change-Number: 63516
Gerrit-PatchSet: 3
Gerrit-Owner: mans0n+coreboot@gorani.run
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
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-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Attention: mans0n+coreboot@gorani.run
Gerrit-Attention: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Attention: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Comment-Date: Fri, 25 Aug 2023 09:27:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: mans0n+coreboot@gorani.run
Gerrit-MessageType: comment