summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFabrizio Castro <fabrizio.castro.jz@renesas.com>2023-07-15 02:04:02 +0100
committerMark Brown <broonie@kernel.org>2023-07-17 12:41:44 +0100
commit9f5ac599801c0f7c0969fa94c638265ed988b9bc (patch)
treeb9e4e5aea6c220363db7c668b4587df365b989f2
parent2ed2699f58891c72fcd462129345d09424f986c5 (diff)
downloadlinux-stable-9f5ac599801c0f7c0969fa94c638265ed988b9bc.tar.gz
linux-stable-9f5ac599801c0f7c0969fa94c638265ed988b9bc.tar.bz2
linux-stable-9f5ac599801c0f7c0969fa94c638265ed988b9bc.zip
spi: rzv2m-csi: Replace unnecessary ternary operators
The ternary operators used to initialize tx_completed and rx_completed are not necessary, replace them with a better implementation. Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com> Link: https://lore.kernel.org/r/20230715010407.1751715-6-fabrizio.castro.jz@renesas.com Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r--drivers/spi/spi-rzv2m-csi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/spi/spi-rzv2m-csi.c b/drivers/spi/spi-rzv2m-csi.c
index 1c65cd5f2039..038f1486b7d7 100644
--- a/drivers/spi/spi-rzv2m-csi.c
+++ b/drivers/spi/spi-rzv2m-csi.c
@@ -439,8 +439,8 @@ static int rzv2m_csi_setup(struct spi_device *spi)
static int rzv2m_csi_pio_transfer(struct rzv2m_csi_priv *csi)
{
- bool tx_completed = csi->txbuf ? false : true;
- bool rx_completed = csi->rxbuf ? false : true;
+ bool tx_completed = !csi->txbuf;
+ bool rx_completed = !csi->rxbuf;
int ret = 0;
/* Make sure the TX FIFO is empty */