[flashrom-gerrit] Change in flashrom[master]: allow 0x34 as ICCRIBA for CHIPSET_C620_SERIES_LEWISBURG

[Jonathan Zhang (Code Review)]

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/44621 )

Change subject: allow 0x34 as ICCRIBA for CHIPSET_C620_SERIES_LEWISBURG
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/flashrom/+/44621/2/ich_descriptors.c 
File ich_descriptors.c:

https://review.coreboot.org/c/flashrom/+/44621/2/ich_descriptors.c@a943 
PS2, Line 943: 		return CHIPSET_300_SERIES_CANNON_POINT;
> The default path for 0x34 (without warning) leads here. If you change that, […]

Unfortunately I have not found any documentation related to ICCRIBA.
On DeltaLake DVT server, ICCRIBA is 0x34. When it uses original default (CHIPSET_300_SERIES_CANNON_POINT), flashrom fails, not able to parse the descriptor. But when CHIPSET_C620_SERIES_LEWISBURG is chosen, it works well.

I wonder if any PCH of CHIPSET_300_SERIES_CANNON_POINT has ICCRIBA as 0x34.