Commit 85a84a61 authored by William Zhang's avatar William Zhang Committed by Mark Brown

spi: bcm63xx-hsspi: Endianness fix for ARM based SoC

HSSPI controller uses big endian for the opcode in the message to the
controller ping pong buffer. Use cpu_to_be16 to properly handle the
endianness for both big and little endian host.

Fixes: 142168eb ("spi: bcm63xx-hsspi: add bcm63xx HSSPI driver")
Signed-off-by: default avatarKursad Oney <kursad.oney@broadcom.com>
Signed-off-by: default avatarWilliam Zhang <william.zhang@broadcom.com>
Acked-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>

Link: https://lore.kernel.org/r/20230207065826.285013-7-william.zhang@broadcom.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 99d7428a
...@@ -194,7 +194,7 @@ static int bcm63xx_hsspi_do_txrx(struct spi_device *spi, struct spi_transfer *t) ...@@ -194,7 +194,7 @@ static int bcm63xx_hsspi_do_txrx(struct spi_device *spi, struct spi_transfer *t)
tx += curr_step; tx += curr_step;
} }
__raw_writew(opcode | curr_step, bs->fifo); __raw_writew((u16)cpu_to_be16(opcode | curr_step), bs->fifo);
/* enable interrupt */ /* enable interrupt */
__raw_writel(HSSPI_PINGx_CMD_DONE(0), __raw_writel(HSSPI_PINGx_CMD_DONE(0),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment