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

[Nico Huber (Code Review)]

Nico Huber 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;
> Unfortunately I have not found any documentation related to ICCRIBA.

It used to give some offset, but newer SPI guides just list it as reserved
or reserved with a specific value.
> I wonder if any PCH of CHIPSET_300_SERIES_CANNON_POINT has ICCRIBA as 0x34.

I guess all of them. The SPI guides say "Reserved. Set this field to 34h.".

It seems the original C620 models and Sunrise Point PCHs had the same issue,
hence the check for `NM` above. Maybe that's all we need on this path
too?