can: m_can: Always acknowledge all interrupts
authorMarkus Schneider-Pargmann <msp@baylibre.com>
Wed, 15 Mar 2023 11:05:32 +0000 (12:05 +0100)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Fri, 24 Mar 2023 18:13:54 +0000 (19:13 +0100)
The code already exits the function on !ir before this condition. No
need to check again if anything is set as IR_ALL_INT is 0xffffffff.

Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Link: https://lore.kernel.org/all/20230315110546.2518305-3-msp@baylibre.com
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/m_can/m_can.c

index 563625a701fca9773e218e1ff45c067b4978dc12..8eb327ae3bdf0c9976d5b008d0020b045fdd33ec 100644 (file)
@@ -1083,8 +1083,7 @@ static irqreturn_t m_can_isr(int irq, void *dev_id)
                return IRQ_NONE;
 
        /* ACK all irqs */
-       if (ir & IR_ALL_INT)
-               m_can_write(cdev, M_CAN_IR, ir);
+       m_can_write(cdev, M_CAN_IR, ir);
 
        if (cdev->ops->clear_interrupts)
                cdev->ops->clear_interrupts(cdev);