From eaeb0b7892432f25cf1e518fa4a5211c526aa438 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Sat, 27 Jul 2019 13:45:58 +0200 Subject: sb/intel/common/spi: Fix opmenu setup MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove a spurious reference: the `optype` field is already the pointer we want. Change-Id: I65eb3a519db9037c84750c5d40e3f19a1e360361 Signed-off-by: Nico Huber Reviewed-on: https://review.coreboot.org/c/coreboot/+/34596 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons Reviewed-by: Kyösti Mälkki --- src/southbridge/intel/common/spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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) -- cgit v1.2.3