Commit 0e836c3b authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Mark Brown

spi: sh-msiof: Restrict bits per word to 8/16/24/32 on R-Car Gen2/3

While the MSIOF variants in older SuperH and SH/R-Mobile SoCs support
bits-per-word values in the full range 8..32, the variants present in
R-Car Gen2 and Gen3 SoCs are restricted to 8, 16, 24, or 32.

Obtain the value from family-specific sh_msiof_chipdata to fix this.
Reported-by: default avatarYoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: default avatarSimon Horman <horms+renesas@verge.net.au>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent daad4d2a
...@@ -32,6 +32,7 @@ ...@@ -32,6 +32,7 @@
#include <asm/unaligned.h> #include <asm/unaligned.h>
struct sh_msiof_chipdata { struct sh_msiof_chipdata {
u32 bits_per_word_mask;
u16 tx_fifo_size; u16 tx_fifo_size;
u16 rx_fifo_size; u16 rx_fifo_size;
u16 ctlr_flags; u16 ctlr_flags;
...@@ -1048,6 +1049,7 @@ static int sh_msiof_transfer_one(struct spi_controller *ctlr, ...@@ -1048,6 +1049,7 @@ static int sh_msiof_transfer_one(struct spi_controller *ctlr,
} }
static const struct sh_msiof_chipdata sh_data = { static const struct sh_msiof_chipdata sh_data = {
.bits_per_word_mask = SPI_BPW_RANGE_MASK(8, 32),
.tx_fifo_size = 64, .tx_fifo_size = 64,
.rx_fifo_size = 64, .rx_fifo_size = 64,
.ctlr_flags = 0, .ctlr_flags = 0,
...@@ -1055,6 +1057,8 @@ static const struct sh_msiof_chipdata sh_data = { ...@@ -1055,6 +1057,8 @@ static const struct sh_msiof_chipdata sh_data = {
}; };
static const struct sh_msiof_chipdata rcar_gen2_data = { static const struct sh_msiof_chipdata rcar_gen2_data = {
.bits_per_word_mask = SPI_BPW_MASK(8) | SPI_BPW_MASK(16) |
SPI_BPW_MASK(24) | SPI_BPW_MASK(32),
.tx_fifo_size = 64, .tx_fifo_size = 64,
.rx_fifo_size = 64, .rx_fifo_size = 64,
.ctlr_flags = SPI_CONTROLLER_MUST_TX, .ctlr_flags = SPI_CONTROLLER_MUST_TX,
...@@ -1062,6 +1066,8 @@ static const struct sh_msiof_chipdata rcar_gen2_data = { ...@@ -1062,6 +1066,8 @@ static const struct sh_msiof_chipdata rcar_gen2_data = {
}; };
static const struct sh_msiof_chipdata rcar_gen3_data = { static const struct sh_msiof_chipdata rcar_gen3_data = {
.bits_per_word_mask = SPI_BPW_MASK(8) | SPI_BPW_MASK(16) |
SPI_BPW_MASK(24) | SPI_BPW_MASK(32),
.tx_fifo_size = 64, .tx_fifo_size = 64,
.rx_fifo_size = 64, .rx_fifo_size = 64,
.ctlr_flags = SPI_CONTROLLER_MUST_TX, .ctlr_flags = SPI_CONTROLLER_MUST_TX,
...@@ -1386,7 +1392,7 @@ static int sh_msiof_spi_probe(struct platform_device *pdev) ...@@ -1386,7 +1392,7 @@ static int sh_msiof_spi_probe(struct platform_device *pdev)
ctlr->setup = sh_msiof_spi_setup; ctlr->setup = sh_msiof_spi_setup;
ctlr->prepare_message = sh_msiof_prepare_message; ctlr->prepare_message = sh_msiof_prepare_message;
ctlr->slave_abort = sh_msiof_slave_abort; ctlr->slave_abort = sh_msiof_slave_abort;
ctlr->bits_per_word_mask = SPI_BPW_RANGE_MASK(8, 32); ctlr->bits_per_word_mask = chipdata->bits_per_word_mask;
ctlr->auto_runtime_pm = true; ctlr->auto_runtime_pm = true;
ctlr->transfer_one = sh_msiof_transfer_one; ctlr->transfer_one = sh_msiof_transfer_one;
......
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