[coreboot-gerrit] Change in coreboot[master]: soc/intel/baytrail: Don't reinitialize SPI after lockdown

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36008 )

Change subject: soc/intel/baytrail: Don't reinitialize SPI after lockdown
......................................................................

soc/intel/baytrail: Don't reinitialize SPI after lockdown

With the common southbridge SPI code reinitialization after lockdown
is not necessary, hence the SMM finalize call becomes a no-op.

Change-Id: Ie73a0adc120731d541a772e09f3482902771b9eb
Signed-off-by: Arthur Heymans 
---
M src/soc/intel/baytrail/smihandler.c
M src/soc/intel/baytrail/southcluster.c
2 files changed, 0 insertions(+), 23 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/36008/1

diff --git a/src/soc/intel/baytrail/smihandler.c b/src/soc/intel/baytrail/smihandler.c
index 7de9d96..7804253 100644
--- a/src/soc/intel/baytrail/smihandler.c
+++ b/src/soc/intel/baytrail/smihandler.c
@@ -233,22 +233,6 @@
 }
 #endif

-static void finalize(void)
-{
-	static int finalize_done;
-
-	if (finalize_done) {
-		printk(BIOS_DEBUG, "SMM already finalized.\n");
-		return;
-	}
-	finalize_done = 1;
-
-#if CONFIG(SPI_FLASH_SMM)
-	/* Re-init SPI driver to handle locked BAR */
-	spi_init();
-#endif
-}
-
 /*
  * soc_legacy: A payload (Depthcharge) has indicated that the
  *   legacy payload (SeaBIOS) is being loaded. Switch devices that are
@@ -353,10 +337,6 @@
    	southbridge_smi_gsmi();
    	break;
 #endif
-	case APM_CNT_FINALIZE:
-		finalize();
-		break;
-
    case APM_CNT_LEGACY:
    	soc_legacy();
    	break;
diff --git a/src/soc/intel/baytrail/southcluster.c b/src/soc/intel/baytrail/southcluster.c
index 3c0c07f..139d2be 100644
--- a/src/soc/intel/baytrail/southcluster.c
+++ b/src/soc/intel/baytrail/southcluster.c
@@ -577,9 +577,6 @@
    	write32(spi + UVSCC, cfg.uvscc);
    	write32(spi + LVSCC, cfg.lvscc | VCL);
    }
-
-	printk(BIOS_DEBUG, "Finalizing SMM.\n");
-	outb(APM_CNT_FINALIZE, APM_CNT);
 }

BOOT_STATE_INIT_ENTRY(BS_OS_RESUME, BS_ON_ENTRY, finalize_chipset, NULL);