Change in flashrom[staging]: ichspi: Disable software sequencing by default for Skylake

[Nico Huber (Code Review)]

Nico Huber has submitted this change and it was merged. ( https://review.coreboot.org/22274 )

Change subject: ichspi: Disable software sequencing by default for Skylake
......................................................................

ichspi: Disable software sequencing by default for Skylake

Skylake is a mess, especially with coreboot. We have now a present and
configured software sequencing interface with SCGO supposedly being
readonly (Apollo Lake has that feature and a strap documented, Skylake
behaviour might be the same). As we can't easily check if it's read-
only, just enable hardware sequencing by default (even if the software
sequencing interface seems usable).

Change-Id: I8a13fb9c3ca679b3f7d39ad1dc56d5efdc80045b
Signed-off-by: Nico Huber 
Reviewed-on: https://review.coreboot.org/22274
Tested-by: build bot (Jenkins) 
Reviewed-by: David Hendricks 
---
M ichspi.c
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  David Hendricks: Looks good to me, approved

diff --git a/ichspi.c b/ichspi.c
index ec81628..859d55f 100644
--- a/ichspi.c
+++ b/ichspi.c
@@ -1951,6 +1951,11 @@
    		ich_spi_mode = ich_hwseq;
    	}

+		if (ich_spi_mode == ich_auto && ich_gen == CHIPSET_100_SERIES_SUNRISE_POINT) {
+			msg_pdbg("Enabling hardware sequencing by default for 100 series PCH.\n");
+			ich_spi_mode = ich_hwseq;
+		}
+
    	if (ich_spi_mode == ich_hwseq) {
    		if (!desc_valid) {
    			msg_perr("Hardware sequencing was requested "