spi: Change spi_xfer to work in units of bytes instead of bits.

Whenever spi_xfer is called and whenver it's implemented, the natural unit for
the amount of data being transfered is bytes. The API expected things to be
expressed in bits, however, which led to a lot of multiplying and dividing by
eight, and checkes to make sure things were multiples of eight. All of that
can now be removed.

BUG=None
TEST=Built and booted on link, falco, peach_pit and nyan and looked for SPI
errors in the firmware log. Built for rambi.
BRANCH=None

Change-Id: I02365bdb6960a35def7be7a0cd1aa0a2cc09392f
Signed-off-by: Gabe Black <gabeblack@google.com>
Reviewed-on: https://chromium-review.googlesource.com/192049
Reviewed-by: Gabe Black <gabeblack@chromium.org>
Tested-by: Gabe Black <gabeblack@chromium.org>
Commit-Queue: Gabe Black <gabeblack@chromium.org>
[km: cherry-pick from chromium]
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: http://review.coreboot.org/6175
Tested-by: build bot (Jenkins)
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>

2 files changed, 10 insertions, 20 deletions

diff --git a/src/soc/intel/baytrail/spi.c b/src/soc/intel/baytrail/spi.c
index d75caca7af..b649f53411 100644
--- a/src/soc/intel/baytrail/spi.c
+++ b/src/soc/intel/baytrail/spi.c
@@ -500,7 +500,7 @@ static int ich_status_poll(u16 bitmask, int wait_til_set)
 }
 
 int spi_xfer(struct spi_slave *slave, const void *dout,
-		unsigned int bitsout, void *din, unsigned int bitsin)
+		unsigned int bytesout, void *din, unsigned int bytesin)
 {
 	uint16_t control;
 	int16_t opcode_index;
@@ -508,26 +508,21 @@ int spi_xfer(struct spi_slave *slave, const void *dout,
 	int status;
 
 	spi_transaction trans = {
-		dout, bitsout / 8,
-		din, bitsin / 8,
+		dout, bytesout,
+		din, bytesin,
 		0xff, 0xff, 0
 	};
 
 	/* There has to always at least be an opcode. */
-	if (!bitsout || !dout) {
+	if (!bytesout || !dout) {
 		printk(BIOS_DEBUG, "ICH SPI: No opcode for transfer\n");
 		return -1;
 	}
 	/* Make sure if we read something we have a place to put it. */
-	if (bitsin != 0 && !din) {
+	if (bytesin != 0 && !din) {
 		printk(BIOS_DEBUG, "ICH SPI: Read but no target buffer\n");
 		return -1;
 	}
-	/* Right now we don't support writing partial bytes. */
-	if (bitsout % 8 || bitsin % 8) {
-		printk(BIOS_DEBUG, "ICH SPI: Accessing partial bytes not supported\n");
-		return -1;
-	}
 
 	if (ich_status_poll(SPIS_SCIP, 0) == -1)
 		return -1;
diff --git a/src/soc/intel/fsp_baytrail/spi.c b/src/soc/intel/fsp_baytrail/spi.c
index 5e8da95d3e..50f11231da 100644
--- a/src/soc/intel/fsp_baytrail/spi.c
+++ b/src/soc/intel/fsp_baytrail/spi.c
@@ -498,7 +498,7 @@ static int ich_status_poll(u16 bitmask, int wait_til_set)
 }
 
 int spi_xfer(struct spi_slave *slave, const void *dout,
-		unsigned int bitsout, void *din, unsigned int bitsin)
+		unsigned int bytesout, void *din, unsigned int bytesin)
 {
 	uint16_t control;
 	int16_t opcode_index;
@@ -506,26 +506,21 @@ int spi_xfer(struct spi_slave *slave, const void *dout,
 	int status;
 
 	spi_transaction trans = {
-		dout, bitsout / 8,
-		din, bitsin / 8,
+		dout, bytesout,
+		din, bytesin,
 		0xff, 0xff, 0
 	};
 
 	/* There has to always at least be an opcode. */
-	if (!bitsout || !dout) {
+	if (!bytesout || !dout) {
 		printk(BIOS_DEBUG, "ICH SPI: No opcode for transfer\n");
 		return -1;
 	}
 	/* Make sure if we read something we have a place to put it. */
-	if (bitsin != 0 && !din) {
+	if (bytesin != 0 && !din) {
 		printk(BIOS_DEBUG, "ICH SPI: Read but no target buffer\n");
 		return -1;
 	}
-	/* Right now we don't support writing partial bytes. */
-	if (bitsout % 8 || bitsin % 8) {
-		printk(BIOS_DEBUG, "ICH SPI: Accessing partial bytes not supported\n");
-		return -1;
-	}
 
 	if (ich_status_poll(SPIS_SCIP, 0) == -1)
 		return -1;