[U-Boot] [PATCH] mmc_spi: Big-endian support

Pantelis Antoniou panto at antoniou-consulting.com
Wed Aug 12 09:21:31 CEST 2015


Hi Yoshinori,

> On Jun 1, 2015, at 09:22 , Yoshinori Sato <ysato at users.sourceforge.jp> wrote:
> 
> Currently implement always swap for big-endian value.
> So doesn't work big-endian environment.
> 
> Signed-off-by: Yoshinori Sato <ysato at users.sourceforge.jp>
> ---
> drivers/mmc/mmc_spi.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/mmc/mmc_spi.c b/drivers/mmc/mmc_spi.c
> index 5b5b33a..9032a73 100644
> --- a/drivers/mmc/mmc_spi.c
> +++ b/drivers/mmc/mmc_spi.c
> @@ -11,7 +11,7 @@
> #include <spi.h>
> #include <crc.h>
> #include <linux/crc7.h>
> -#include <linux/byteorder/swab.h>
> +#include <asm/byteorder.h>
> 
> /* MMC/SD in SPI mode reports R1 status always */
> #define R1_SPI_IDLE		(1 << 0)
> @@ -91,7 +91,7 @@ static uint mmc_spi_readdata(struct mmc *mmc, void *xbuf,
> 			spi_xfer(spi, bsize * 8, NULL, buf, 0);
> 			spi_xfer(spi, 2 * 8, NULL, &crc, 0);
> #ifdef CONFIG_MMC_SPI_CRC_ON
> -			if (swab16(cyg_crc16(buf, bsize)) != crc) {
> +			if (be_to_cpu16(cyg_crc16(buf, bsize)) != crc) {
> 				debug("%s: CRC error\n", mmc->cfg->name);
> 				r1 = R1_SPI_COM_CRC;
> 				break;
> @@ -120,7 +120,7 @@ static uint mmc_spi_writedata(struct mmc *mmc, const void *xbuf,
> 	tok[1] = multi ? SPI_TOKEN_MULTI_WRITE : SPI_TOKEN_SINGLE;
> 	while (bcnt--) {
> #ifdef CONFIG_MMC_SPI_CRC_ON
> -		crc = swab16(cyg_crc16((u8 *)buf, bsize));
> +		crc = cpu_to_be16(cyg_crc16((u8 *)buf, bsize));
> #endif
> 		spi_xfer(spi, 2 * 8, tok, NULL, 0);
> 		spi_xfer(spi, bsize * 8, buf, NULL, 0);
> @@ -193,7 +193,7 @@ static int mmc_spi_request(struct mmc *mmc, struct mmc_cmd *cmd,
> 	} else if (cmd->resp_type == MMC_RSP_R2) {
> 		r1 = mmc_spi_readdata(mmc, cmd->response, 1, 16);
> 		for (i = 0; i < 4; i++)
> -			cmd->response[i] = swab32(cmd->response[i]);
> +			cmd->response[i] = be32_to_cpu(cmd->response[i]);
> 		debug("r128 %x %x %x %x\n", cmd->response[0], cmd->response[1],
> 		      cmd->response[2], cmd->response[3]);
> 	} else if (!data) {
> @@ -205,7 +205,7 @@ static int mmc_spi_request(struct mmc *mmc, struct mmc_cmd *cmd,
> 		case SD_CMD_SEND_IF_COND:
> 		case MMC_CMD_SPI_READ_OCR:
> 			spi_xfer(spi, 4 * 8, NULL, cmd->response, 0);
> -			cmd->response[0] = swab32(cmd->response[0]);
> +			cmd->response[0] = be32_to_cpu(cmd->response[0]);
> 			debug("r32 %x\n", cmd->response[0]);
> 			break;
> 		case MMC_CMD_SEND_STATUS:
> -- 
> 2.1.4
> 

Applied, thanks.

— Pantelis




More information about the U-Boot mailing list