Attention is currently required from: Patrick Rudolph.
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/78301?usp=email )
Change subject: ich_descriptors: Fix table printing
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/flashrom/+/78301/comment/daa34073_4927976e :
PS2, Line 10: Convert "unknown" to "DevE"
Why it was "unknown" before? Is Device Expansion the names from SPI guides?
Older chipsets and thus older versions of the SPI programming guide had less regions defined. Today there are 16 well known regions, even though they might not be actually used by the chipset.
--
To view, visit
https://review.coreboot.org/c/flashrom/+/78301?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I0d10e2771c7a27c1a73ed53a33e68a04eb9e1e00
Gerrit-Change-Number: 78301
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Rudolph
rudolphpatrick05@gmail.com
Gerrit-Reviewer: Anastasia Klimchuk
aklm@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Attention: Patrick Rudolph
rudolphpatrick05@gmail.com
Gerrit-Comment-Date: Tue, 24 Oct 2023 06:17:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Anastasia Klimchuk
aklm@chromium.org
Gerrit-MessageType: comment