Commit 128ebb89 authored by Amelie Delaunay's avatar Amelie Delaunay Committed by Mark Brown

spi: stm32: use normal conditional statements instead of ternary operator

This patch replace ternary operator use by normal condition statements
to ease code reading.
It also removes redundant !!.
Signed-off-by: default avatarAmelie Delaunay <amelie.delaunay@st.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 042c1c60
...@@ -267,7 +267,10 @@ static int stm32_spi_prepare_mbr(struct stm32_spi *spi, u32 speed_hz) ...@@ -267,7 +267,10 @@ static int stm32_spi_prepare_mbr(struct stm32_spi *spi, u32 speed_hz)
return -EINVAL; return -EINVAL;
/* Determine the first power of 2 greater than or equal to div */ /* Determine the first power of 2 greater than or equal to div */
mbrdiv = (div & (div - 1)) ? fls(div) : fls(div) - 1; if (div & (div - 1))
mbrdiv = fls(div);
else
mbrdiv = fls(div) - 1;
spi->cur_speed = spi->clk_rate / (1 << mbrdiv); spi->cur_speed = spi->clk_rate / (1 << mbrdiv);
...@@ -285,9 +288,12 @@ static u32 stm32_spi_prepare_fthlv(struct stm32_spi *spi) ...@@ -285,9 +288,12 @@ static u32 stm32_spi_prepare_fthlv(struct stm32_spi *spi)
/* data packet should not exceed 1/2 of fifo space */ /* data packet should not exceed 1/2 of fifo space */
half_fifo = (spi->fifo_size / 2); half_fifo = (spi->fifo_size / 2);
fthlv = (spi->cur_bpw <= 8) ? half_fifo : if (spi->cur_bpw <= 8)
(spi->cur_bpw <= 16) ? (half_fifo / 2) : fthlv = half_fifo;
(half_fifo / 4); else if (spi->cur_bpw <= 16)
fthlv = half_fifo / 2;
else
fthlv = half_fifo / 4;
/* align packet size with data registers access */ /* align packet size with data registers access */
if (spi->cur_bpw > 8) if (spi->cur_bpw > 8)
...@@ -462,9 +468,9 @@ static bool stm32_spi_can_dma(struct spi_master *master, ...@@ -462,9 +468,9 @@ static bool stm32_spi_can_dma(struct spi_master *master,
struct stm32_spi *spi = spi_master_get_devdata(master); struct stm32_spi *spi = spi_master_get_devdata(master);
dev_dbg(spi->dev, "%s: %s\n", __func__, dev_dbg(spi->dev, "%s: %s\n", __func__,
(!!(transfer->len > spi->fifo_size)) ? "true" : "false"); (transfer->len > spi->fifo_size) ? "true" : "false");
return !!(transfer->len > spi->fifo_size); return (transfer->len > spi->fifo_size);
} }
/** /**
...@@ -493,7 +499,8 @@ static irqreturn_t stm32_spi_irq(int irq, void *dev_id) ...@@ -493,7 +499,8 @@ static irqreturn_t stm32_spi_irq(int irq, void *dev_id)
* Full-Duplex, need to poll RXP event to know if there are remaining * Full-Duplex, need to poll RXP event to know if there are remaining
* data, before disabling SPI. * data, before disabling SPI.
*/ */
mask |= ((spi->rx_buf && !spi->cur_usedma) ? SPI_SR_RXP : 0); if (spi->rx_buf && !spi->cur_usedma)
mask |= SPI_SR_RXP;
if (!(sr & mask)) { if (!(sr & mask)) {
dev_dbg(spi->dev, "spurious IT (sr=0x%08x, ier=0x%08x)\n", dev_dbg(spi->dev, "spurious IT (sr=0x%08x, ier=0x%08x)\n",
...@@ -656,12 +663,18 @@ static void stm32_spi_dma_config(struct stm32_spi *spi, ...@@ -656,12 +663,18 @@ static void stm32_spi_dma_config(struct stm32_spi *spi,
enum dma_slave_buswidth buswidth; enum dma_slave_buswidth buswidth;
u32 maxburst; u32 maxburst;
buswidth = (spi->cur_bpw <= 8) ? DMA_SLAVE_BUSWIDTH_1_BYTE : if (spi->cur_bpw <= 8)
(spi->cur_bpw <= 16) ? DMA_SLAVE_BUSWIDTH_2_BYTES : buswidth = DMA_SLAVE_BUSWIDTH_1_BYTE;
DMA_SLAVE_BUSWIDTH_4_BYTES; else if (spi->cur_bpw <= 16)
buswidth = DMA_SLAVE_BUSWIDTH_2_BYTES;
else
buswidth = DMA_SLAVE_BUSWIDTH_4_BYTES;
/* Valid for DMA Half or Full Fifo threshold */ /* Valid for DMA Half or Full Fifo threshold */
maxburst = (spi->cur_fthlv == 2) ? 1 : spi->cur_fthlv; if (spi->cur_fthlv == 2)
maxburst = 1;
else
maxburst = spi->cur_fthlv;
memset(dma_conf, 0, sizeof(struct dma_slave_config)); memset(dma_conf, 0, sizeof(struct dma_slave_config));
dma_conf->direction = dir; dma_conf->direction = dir;
...@@ -920,9 +933,12 @@ static int stm32_spi_transfer_one_setup(struct stm32_spi *spi, ...@@ -920,9 +933,12 @@ static int stm32_spi_transfer_one_setup(struct stm32_spi *spi,
~cfg2_clrb) | cfg2_setb, ~cfg2_clrb) | cfg2_setb,
spi->base + STM32_SPI_CFG2); spi->base + STM32_SPI_CFG2);
nb_words = DIV_ROUND_UP(transfer->len * 8, if (spi->cur_bpw <= 8)
(spi->cur_bpw <= 8) ? 8 : nb_words = transfer->len;
(spi->cur_bpw <= 16) ? 16 : 32); else if (spi->cur_bpw <= 16)
nb_words = DIV_ROUND_UP(transfer->len * 8, 16);
else
nb_words = DIV_ROUND_UP(transfer->len * 8, 32);
nb_words <<= SPI_CR2_TSIZE_SHIFT; nb_words <<= SPI_CR2_TSIZE_SHIFT;
if (nb_words <= SPI_CR2_TSIZE) { if (nb_words <= SPI_CR2_TSIZE) {
......
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