[flashrom-gerrit] Change in flashrom[staging]: ichspi: Add support for Intel Skylake

[David Hendricks (Code Review)]

David Hendricks has posted comments on this change. ( https://review.coreboot.org/18962 )

Change subject: ichspi: Add support for Intel Skylake
......................................................................

Patch Set 9:

Let's go with a comment, then. `17-16` still isn't obvious since the reader still needs to look at how it's used in the code and find an older datasheet to infer what is happening.

As an example:
/* HSFC and HSFS 16-bit registers are combined into the 32-bit BIOS_HSFSTS_CTL register in the Sunrise Point datasheet, however we still treat them separately in order to re-use code. */

This also has the advantage of being searchable, in case somebody wants to grep for BIOS_HSFSTS_CTL to see how it's used.