spi: gpio: Simplify SPI_MASTER_NO_TX check in spi_gpio_probe()
authorAndrey Smirnov <andrew.smirnov@gmail.com>
Wed, 3 Apr 2019 04:01:25 +0000 (21:01 -0700)
committerMark Brown <broonie@kernel.org>
Wed, 3 Apr 2019 04:32:24 +0000 (11:32 +0700)
Swap branches of the if statement in order to simplify it's logical
condition being checked. No functional change intended.

Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Cc: Chris Healy <cphealy@gmail.com>
Cc: linux-spi@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-gpio.c

index ca97b861c625c84b713c9e3b2e54bc4b7ebec9e1..137f47e271dcd2a32c6d40838585ffbaad0161ce 100644 (file)
@@ -440,16 +440,16 @@ static int spi_gpio_probe(struct platform_device *pdev)
        bb->chipselect = spi_gpio_chipselect;
        bb->set_line_direction = spi_gpio_set_direction;
 
-       if ((master_flags & SPI_MASTER_NO_TX) == 0) {
-               bb->txrx_word[SPI_MODE_0] = spi_gpio_txrx_word_mode0;
-               bb->txrx_word[SPI_MODE_1] = spi_gpio_txrx_word_mode1;
-               bb->txrx_word[SPI_MODE_2] = spi_gpio_txrx_word_mode2;
-               bb->txrx_word[SPI_MODE_3] = spi_gpio_txrx_word_mode3;
-       } else {
+       if (master_flags & SPI_MASTER_NO_TX) {
                bb->txrx_word[SPI_MODE_0] = spi_gpio_spec_txrx_word_mode0;
                bb->txrx_word[SPI_MODE_1] = spi_gpio_spec_txrx_word_mode1;
                bb->txrx_word[SPI_MODE_2] = spi_gpio_spec_txrx_word_mode2;
                bb->txrx_word[SPI_MODE_3] = spi_gpio_spec_txrx_word_mode3;
+       } else {
+               bb->txrx_word[SPI_MODE_0] = spi_gpio_txrx_word_mode0;
+               bb->txrx_word[SPI_MODE_1] = spi_gpio_txrx_word_mode1;
+               bb->txrx_word[SPI_MODE_2] = spi_gpio_txrx_word_mode2;
+               bb->txrx_word[SPI_MODE_3] = spi_gpio_txrx_word_mode3;
        }
        bb->setup_transfer = spi_bitbang_setup_transfer;