From d9d351d2512af0804f00f0561b66ab18ba879dce Mon Sep 17 00:00:00 2001 From: Martin Kepplinger Date: Fri, 30 Jun 2017 14:12:33 +0200 Subject: soc/intel/quark/spi.c: Explain a read in order to flush buffers In order for this (seemingly unnecessary) status assignment to stay, let's explain it in a comment. Change-Id: I0a364539c37005cfd637b75c8cc23b84e274294d Signed-off-by: Martin Kepplinger Reviewed-on: https://review.coreboot.org/20411 Tested-by: build bot (Jenkins) Reviewed-by: Lee Leahy Reviewed-by: Paul Menzel --- src/soc/intel/quark/spi.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/soc/intel/quark') diff --git a/src/soc/intel/quark/spi.c b/src/soc/intel/quark/spi.c index 1d257a563e..c943567e1d 100644 --- a/src/soc/intel/quark/spi.c +++ b/src/soc/intel/quark/spi.c @@ -166,7 +166,10 @@ static int xfer(const struct spi_slave *slave, const void *dout, ctrlr->address = (data[0] << 16) | (data[1] << 8) | data[2]; + + /* read in order to flush the write buffer */ status = ctrlr->address; + data += 3; bytesout -= 3; } -- cgit v1.2.3