[flashrom-gerrit] Change in flashrom[master]: ich_descriptors.c: Fix PCH detection for Tiger Lake

[Nico Huber (Code Review)]

Attention is currently required from: Michał Żygowski, Angel Pons.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/55577 )

Change subject: ich_descriptors.c: Fix PCH detection for Tiger Lake
......................................................................

Patch Set 1:

(1 comment)

File ich_descriptors.c:

https://review.coreboot.org/c/flashrom/+/55577/comment/5015de18_ffbc244e 
PS1, Line 955: CHIPSET_300_SERIES_CANNON_POINT
> I also have a suggestion: we could reuse the coreboot's ifdtool detection function guess_ifd_2_chips […]

It's certainly possible if you want to pursue this. However, ifdtool
was often unreliable in the past. And that function is using strap
settings and not structural fields like we are used to. I'm not sure
how well that works and it would need some more restructuring of
the code here to access strap information.