summaryrefslogtreecommitdiff
path: root/src/southbridge
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2019-06-11 11:10:25 +0200
committerArthur Heymans <arthur@aheymans.xyz>2019-06-18 13:11:41 +0000
commit816aaba399537349b068afbd9bde702d00243f54 (patch)
treea161c4dd3a5bb5f09f5337ce76b1732fd376feaf /src/southbridge
parent21c5d43d723d5a52d5b4c0060a94d56bdf297424 (diff)
downloadcoreboot-816aaba399537349b068afbd9bde702d00243f54.tar.xz
sb/intel/spi: Check for the SPI lock bit during runtime
The SPI swseq controller can be locked in other parts of the code, for instance when it's locked down in the finalize section. The driver has to be made aware of that. The simpler solution is to not keep track of the state and simply read out the lock bit on each SPI transfer. Change-Id: Ifcd5121b89d6f80fc1c1368786982d0d9fa1bf61 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/33388 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/southbridge')
-rw-r--r--src/southbridge/intel/common/spi.c18
1 files changed, 12 insertions, 6 deletions
diff --git a/src/southbridge/intel/common/spi.c b/src/southbridge/intel/common/spi.c
index f447515e58..d9a77c3bb5 100644
--- a/src/southbridge/intel/common/spi.c
+++ b/src/southbridge/intel/common/spi.c
@@ -38,8 +38,6 @@
static int spi_is_multichip(void);
-static int g_ichspi_lock = 0;
-
struct ich7_spi_regs {
uint16_t spis;
uint16_t spic;
@@ -294,7 +292,6 @@ void spi_init(void)
cntlr->data = (uint8_t *)ich7_spi->spid;
cntlr->databytes = sizeof(ich7_spi->spid);
cntlr->status = (uint8_t *)&ich7_spi->spis;
- g_ichspi_lock = readw_(&ich7_spi->spis) & HSFS_FLOCKDN;
cntlr->control = &ich7_spi->spic;
cntlr->bbar = &ich7_spi->bbar;
cntlr->preop = &ich7_spi->preop;
@@ -304,7 +301,6 @@ void spi_init(void)
ich9_spi = (struct ich9_spi_regs *)(rcrb + 0x3800);
cntlr->ich9_spi = ich9_spi;
hsfs = readw_(&ich9_spi->hsfs);
- g_ichspi_lock = hsfs & HSFS_FLOCKDN;
cntlr->hsfs = hsfs;
cntlr->opmenu = ich9_spi->opmenu;
cntlr->menubytes = sizeof(ich9_spi->opmenu);
@@ -336,6 +332,16 @@ void spi_init(void)
pci_write_config8(dev, 0xdc, bios_cntl | 0x1);
}
+static int spi_locked(void)
+{
+ struct ich_spi_controller *cntlr = &g_cntlr;
+ if (CONFIG(SOUTHBRIDGE_INTEL_I82801GX)) {
+ return !!(readw_(&cntlr->ich7_spi->spis) & HSFS_FLOCKDN);
+ } else {
+ return !!(readw_(&cntlr->ich9_spi->hsfs) | HSFS_FLOCKDN);
+ }
+}
+
static void spi_init_cb(void *unused)
{
spi_init();
@@ -407,7 +413,7 @@ static int spi_setup_opcode(spi_transaction *trans)
trans->opcode = trans->out[0];
spi_use_out(trans, 1);
- if (!g_ichspi_lock) {
+ if (!spi_locked()) {
/* The lock is off, so just use index 0. */
writeb_(trans->opcode, cntlr->opmenu);
optypes = readw_(cntlr->optype);
@@ -552,7 +558,7 @@ static int spi_ctrlr_xfer(const struct spi_slave *slave, const void *dout,
* in order to prevent the Management Engine from
* issuing a transaction between WREN and DATA.
*/
- if (!g_ichspi_lock)
+ if (!spi_locked())
writew_(trans.opcode, cntlr->preop);
return 0;
}