wilc1000: use correct write command sequence in wilc_spi_sync_ext()
authorAjay Singh <ajay.kathat@microchip.com>
Thu, 16 Sep 2021 16:49:22 +0000 (16:49 +0000)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 21 Sep 2021 15:08:17 +0000 (18:08 +0300)
Instead of using double read for the same register, use the write register
command after the read command.
The correct sequence is to use the read value in write command instead of
reading the same register again.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20210916164902.74629-11-ajay.kathat@microchip.com
drivers/net/wireless/microchip/wilc1000/spi.c

index 4f0bc24d2a7854e542e270c2cbe68dc38dcada9b..640850f989dd956f0732b64cf6ec26a2da5fdda7 100644 (file)
@@ -1167,7 +1167,7 @@ static int wilc_spi_sync_ext(struct wilc *wilc, int nint)
                for (i = 0; (i < 3) && (nint > 0); i++, nint--)
                        reg |= BIT(i);
 
-               ret = wilc_spi_read_reg(wilc, WILC_INTR2_ENABLE, &reg);
+               ret = wilc_spi_write_reg(wilc, WILC_INTR2_ENABLE, reg);
                if (ret) {
                        dev_err(&spi->dev, "Failed write reg (%08x)...\n",
                                WILC_INTR2_ENABLE);