[coreboot-gerrit] New patch to review for coreboot: sb/amd/sb700: Do not reset fifo after skipping the sent bytes

Timothy Pearson (tpearson@raptorengineering.com) gerrit at coreboot.org
Wed Aug 3 21:54:02 CEST 2016


Timothy Pearson (tpearson at raptorengineering.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16054

-gerrit

commit cbd20fa76bcf722cf6d2a7826f0c17fcb3761482
Author: Timothy Pearson <tpearson at raptorengineering.com>
Date:   Wed Aug 3 14:49:01 2016 -0500

    sb/amd/sb700: Do not reset fifo after skipping the sent bytes
    
    Port commit e08493 to the SB700 platform
    
    Change-Id: Ie18c6cc0ccb31a0d16a80fcb4c2e147c19e228fe
    Signed-off-by: Timothy Pearson <tpearson at raptorengineering.com>
---
 src/southbridge/amd/sb700/spi.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/southbridge/amd/sb700/spi.c b/src/southbridge/amd/sb700/spi.c
index 0641ff9..fa316f1 100644
--- a/src/southbridge/amd/sb700/spi.c
+++ b/src/southbridge/amd/sb700/spi.c
@@ -134,8 +134,7 @@ int spi_xfer(struct spi_slave *slave, const void *dout,
 	for (count = 0; count < bytesout; count++) {
 		cmd = read8((void *)(spibar + 0x0C));
 	}
-
-	reset_internal_fifo_pointer();
+	/* read response bytes */
 	for (count = 0; count < bytesin; count++, din++) {
 		*(u8 *)din = read8((void *)(spibar + 0x0C));
 	}



More information about the coreboot-gerrit mailing list