Attention is currently required from: Nikolai Artemiev, Sergii Dmytruk, Stefan Reinauer.
View Change
2 comments:
File flashchips.c:
Patch Set #1, Line 2651: .tested = {.probe = NT, .read = NT, .erase = NT, .write = BAD, .wp = NT},
The chip seems to have only sector-based write protection and no block protection. […]
Thank you so much for catching! I put NA here (and also in the other chip).
Patch Set #1, Line 3720: .tested = {.probe = OK, .read = OK, .erase = BAD, .write = BAD, .wp = NT},
It's [this chip](https://pdf1.alldatasheet.com/datasheet-pdf/view/57384/CATALYST/CAT28F512. […]
Done
To view, visit change 83132. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-MessageType: comment
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I6ed8e04cd215865dc6a7d9415634dedbe3014ab5
Gerrit-Change-Number: 83132
Gerrit-PatchSet: 2
Gerrit-Owner: 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-Attention: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
Gerrit-Comment-Date: Fri, 21 Jun 2024 01:38:36 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>