Attention is currently required from: Simon Buhrow, Nico Huber, Aarya.
Thomas Heijligen has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/67354 )
Change subject: spi.c: Add erasefn and opcodes for AT45 and S25F
......................................................................
Patch Set 2: Code-Review+1
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/67354/comment/fc8cfc1a_d64ae7e1
PS2, Line 7: spi.c: Add erasefn and opcodes for AT45 and S25F
To what? / To which lookup list?
File spi.c:
https://review.coreboot.org/c/flashrom/+/67354/comment/cd30af35_d9910c53
PS2, Line 50: &spi_erase_at45cs_sector, 0x50},
Nico, please have a look here at45db.c:417 spi_erase_at45cs_sector().
The opcode might be variable.
--
To view, visit
https://review.coreboot.org/c/flashrom/+/67354
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ief25932120f940dea0ffe161a79219deecb2e8ab
Gerrit-Change-Number: 67354
Gerrit-PatchSet: 2
Gerrit-Owner: Aarya
aarya.chaumal@gmail.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Simon Buhrow
Gerrit-Reviewer: Thomas Heijligen
src@posteo.de
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Simon Buhrow
Gerrit-Attention: Nico Huber
nico.h@gmx.de
Gerrit-Attention: Aarya
aarya.chaumal@gmail.com
Gerrit-Comment-Date: Tue, 06 Sep 2022 09:32:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment