summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNico Huber <nico.h@gmx.de>2019-07-27 13:45:58 +0200
committerNico Huber <nico.h@gmx.de>2019-07-28 17:54:02 +0000
commiteaeb0b7892432f25cf1e518fa4a5211c526aa438 (patch)
treecc39e0814134cf2795a1171bd88e89407e9fdb69
parent1a64194307f41e6d25eacc8853b92c7017940082 (diff)
downloadcoreboot-eaeb0b7892432f25cf1e518fa4a5211c526aa438.tar.xz
sb/intel/common/spi: Fix opmenu setup
Remove a spurious reference: the `optype` field is already the pointer we want. Change-Id: I65eb3a519db9037c84750c5d40e3f19a1e360361 Signed-off-by: Nico Huber <nico.h@gmx.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/34596 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
-rw-r--r--src/southbridge/intel/common/spi.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/southbridge/intel/common/spi.c b/src/southbridge/intel/common/spi.c
index 3244808179..0095dd9ccb 100644
--- a/src/southbridge/intel/common/spi.c
+++ b/src/southbridge/intel/common/spi.c
@@ -1081,7 +1081,7 @@ void spi_finalize_ops(void)
optype |= (spi_config.ops[i].type & 3) << (i * 2);
writeb_(spi_config.ops[i].op, &cntlr->opmenu[i]);
}
- writew_(optype, &cntlr->optype);
+ writew_(optype, cntlr->optype);
}
__weak void intel_southbridge_override_spi(struct intel_swseq_spi_config *spi_config)