[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/common/spi: Check the SPI lock bit before setting FPR

[Arthur Heymans (Code Review)]

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

Change subject: sb/intel/common/spi: Check the SPI lock bit before setting FPR
......................................................................

sb/intel/common/spi: Check the SPI lock bit before setting FPR

Change-Id: Ib0b63c3b0342c62aeabb5c6e418eb9811fc6597d
Signed-off-by: Arthur Heymans 
---
M src/southbridge/intel/common/spi.c
1 file changed, 6 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/33389/1

diff --git a/src/southbridge/intel/common/spi.c b/src/southbridge/intel/common/spi.c
index 3cecd4f..6f1d54e 100644
--- a/src/southbridge/intel/common/spi.c
+++ b/src/southbridge/intel/common/spi.c
@@ -1011,6 +1011,12 @@
    int fpr;
    uint32_t *fpr_base;

+	if (spi_locked) {
+		printk(BIOS_ERR,
+		       "ERROR: SPI already locked, can't setup FPR!\n");
+		return -1;
+	}
+
    fpr_base = cntlr->fpr;

/* Find first empty FPR */