Use existing macro GENMASK to get the bitmask value. Moved the code to
get the bitmask value outside the loop, as its only required one time.

Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
---
 drivers/staging/wilc1000/wilc_spi.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/wilc1000/wilc_spi.c 
b/drivers/staging/wilc1000/wilc_spi.c
index 131d2b7..c63f534 100644
--- a/drivers/staging/wilc1000/wilc_spi.c
+++ b/drivers/staging/wilc1000/wilc_spi.c
@@ -955,6 +955,7 @@ static int wilc_spi_read_int(struct wilc *wilc, u32 
*int_status)
        tmp = (byte_cnt >> 2) & IRQ_DMA_WD_CNT_MASK;
 
        j = 0;
+       unknown_mask = GENMASK(g_spi.nint - 1, 0);
        do {
                wilc_spi_read_reg(wilc, 0x1a90, &irq_flags);
                tmp |= ((irq_flags >> 27) << IRG_FLAGS_OFFSET);
@@ -964,8 +965,6 @@ static int wilc_spi_read_int(struct wilc *wilc, u32 
*int_status)
                        tmp |= (((irq_flags >> 0) & 0x7) << k);
                }
 
-               unknown_mask = ~((1ul << g_spi.nint) - 1);
-
                if ((tmp >> IRG_FLAGS_OFFSET) & unknown_mask) {
                        dev_err(&spi->dev,
                                "Unexpected interrupt(2):j=%d,tmp=%x,mask=%x\n",
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to