Change in flashrom[master]: ichspi: Add Alder Lake support

[Edward O'Callaghan (Code Review)]

Attention is currently required from: Nico Huber, Rizwan Qureshi, Angel Pons.
Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62251 )

Change subject: ichspi: Add Alder Lake support
......................................................................

Patch Set 13:

(1 comment)

Patchset:

PS12: 
Sorry I think I didn't write well. We are saying the same thing.
> Saying it's hard to predict the result of future work.

Exactly this!

As for the enum, sorry I meant to say enum values. That is to say, a enum value that represents multiple chipsets that are the same and folding up the individual 500 and 600 together as an example.

Also like you said, I wanted to be consistent with the current approach which is what I meant by not being half-baked that is to say, doing something new but only for subset of chipsets. For example https://review.coreboot.org/c/flashrom/+/62580 could be needed and probably some other consistency cleanups? Subrata and I plan to discuss to have a audit over things to look for incomplete cases.

Resolved?