Change in ...coreboot[master]: sb/intel/i82801gx: Use common code to lock down SPI

[Arthur Heymans (Code Review)]

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

Change subject: sb/intel/i82801gx: Use common code to lock down SPI
......................................................................

sb/intel/i82801gx: Use common code to lock down SPI

Change-Id: Ie5ae18e576b7cf7450ab7aa2c12244a92a452d62
Signed-off-by: Arthur Heymans 
---
M src/southbridge/intel/i82801gx/lpc.c
1 file changed, 1 insertion(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/33043/1

diff --git a/src/southbridge/intel/i82801gx/lpc.c b/src/southbridge/intel/i82801gx/lpc.c
index b226b23..33cf941 100644
--- a/src/southbridge/intel/i82801gx/lpc.c
+++ b/src/southbridge/intel/i82801gx/lpc.c
@@ -611,9 +611,6 @@
    }
 }

-#define SPIBAR16(x) RCBA16(0x3020 + x)
-#define SPIBAR32(x) RCBA32(0x3020 + x)
-
 static void lpc_final(struct device *dev)
 {
    u16 tco1_cnt;
@@ -623,8 +620,7 @@

spi_finalize_ops();

-	/* Lock SPIBAR */
-	SPIBAR16(0) = SPIBAR16(0) | (1 << 15);
+	spi_lock_down();

/* BIOS Interface Lockdown */
    RCBA32(0x3410) |= 1 << 0;

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33043 )

Change subject: sb/intel/i82801gx: Use common code to lock down SPI
......................................................................

Patch Set 3: Code-Review+1

[Arthur Heymans (Code Review)]

Arthur Heymans has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/33043 )

Change subject: sb/intel/i82801gx: Use common code to lock down SPI
......................................................................

Abandoned