Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/83143?usp=email )
Change subject: ich_descriptor: Fix chipset_names index for Intel Meteor Lake ......................................................................
ich_descriptor: Fix chipset_names index for Intel Meteor Lake
commit hash 85b977151b8f57 (ichspi.c: Add support for region 9 and beyond in Meteor Lake) moved the Intel Meteor Lake macro in programmer.h, causing flashrom to display an incorrect chipset name for Meteor Lake platforms.
This patch updates the corresponding chipset_names index to resolve this issue.
TEST=Verified correct chipset name in flashrom output for Meteor Lake chipset (google/rex0 board).
Change-Id: Ic09cf0474c980369bcbf90924d45f697bc1b0a0d Signed-off-by: Subrata Banik subratabanik@google.com Reviewed-on: https://review.coreboot.org/c/flashrom/+/83143 Reviewed-by: Anastasia Klimchuk aklm@chromium.org Reviewed-by: Sam McNally sammc@google.com Reviewed-by: Hsuan-ting Chen roccochen@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M ich_descriptors.c 1 file changed, 2 insertions(+), 1 deletion(-)
Approvals: Hsuan-ting Chen: Looks good to me, approved Sam McNally: Looks good to me, but someone else must approve build bot (Jenkins): Verified Anastasia Klimchuk: Looks good to me, approved
diff --git a/ich_descriptors.c b/ich_descriptors.c index b11a78d..6de4316 100644 --- a/ich_descriptors.c +++ b/ich_descriptors.c @@ -121,7 +121,8 @@ "9 series Wildcat Point", "9 series Wildcat Point LP", "100 series Sunrise Point", "C620 series Lewisburg", "C740 series Emmitsburg", "300 series Cannon Point", "400 series Comet Point", "500 series Tiger Point", "600 series Alder Point", - "Meteor Lake", "Apollo Lake", "Gemini Lake", "Jasper Lake", "Elkhart Lake", + "Apollo Lake", "Gemini Lake", "Jasper Lake", "Elkhart Lake", + "Meteor Lake", }; if (cs < CHIPSET_ICH8 || cs - CHIPSET_ICH8 + 1 >= ARRAY_SIZE(chipset_names)) cs = 0;