Attention is currently required from: Anastasia Klimchuk, Nikolai Artemiev.
Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/70349 )
Change subject: tree/: Change chip restore data type from uint8_t to void ptr
......................................................................
Patch Set 9: Code-Review+1
(2 comments)
File s25f.c:
https://review.coreboot.org/c/flashrom/+/70349/comment/0811f358_0e4fd1d2
PS5, Line 295: if (!data)
: return 1;
Yes, that's a typical pattern, to have `msg_perr` with out of memory message.
Done
File spi25_statusreg.c:
https://review.coreboot.org/c/flashrom/+/70349/comment/44ba6399_25939800
PS5, Line 313: if (!data)
: return 1;
ditto
Done
--
To view, visit
https://review.coreboot.org/c/flashrom/+/70349
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I311b468a4b0349f4da9584c12b36af6ec2394527
Gerrit-Change-Number: 70349
Gerrit-PatchSet: 9
Gerrit-Owner: Nikolai Artemiev
nartemiev@google.com
Gerrit-Reviewer: Anastasia Klimchuk
aklm@chromium.org
Gerrit-Reviewer: Edward O'Callaghan
quasisec@chromium.org
Gerrit-Reviewer: Sergii Dmytruk
sergii.dmytruk@3mdeb.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Anastasia Klimchuk
aklm@chromium.org
Gerrit-Attention: Nikolai Artemiev
nartemiev@google.com
Gerrit-Comment-Date: Wed, 18 Jan 2023 12:07:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Edward O'Callaghan
quasisec@chromium.org
Comment-In-Reply-To: Anastasia Klimchuk
aklm@chromium.org
Comment-In-Reply-To: Sergii Dmytruk
sergii.dmytruk@3mdeb.com
Gerrit-MessageType: comment