We don't act on any errors reading registers while handling watchdog
interrupt. Since this is an interrupt handler, we cannot return such
errors. So just remove the variable.

Signed-off-by: Andrew Lunn <[email protected]>
---
 drivers/net/dsa/mv88e6xxx/global2.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/dsa/mv88e6xxx/global2.c 
b/drivers/net/dsa/mv88e6xxx/global2.c
index 8fd483020c5b..75b227d0f73b 100644
--- a/drivers/net/dsa/mv88e6xxx/global2.c
+++ b/drivers/net/dsa/mv88e6xxx/global2.c
@@ -876,19 +876,18 @@ static int mv88e6390_watchdog_setup(struct mv88e6xxx_chip 
*chip)
 
 static int mv88e6390_watchdog_action(struct mv88e6xxx_chip *chip, int irq)
 {
-       int err;
        u16 reg;
 
        mv88e6xxx_g2_write(chip, MV88E6390_G2_WDOG_CTL,
                           MV88E6390_G2_WDOG_CTL_PTR_EVENT);
-       err = mv88e6xxx_g2_read(chip, MV88E6390_G2_WDOG_CTL, &reg);
+       mv88e6xxx_g2_read(chip, MV88E6390_G2_WDOG_CTL, &reg);
 
        dev_info(chip->dev, "Watchdog event: 0x%04x",
                 reg & MV88E6390_G2_WDOG_CTL_DATA_MASK);
 
        mv88e6xxx_g2_write(chip, MV88E6390_G2_WDOG_CTL,
                           MV88E6390_G2_WDOG_CTL_PTR_HISTORY);
-       err = mv88e6xxx_g2_read(chip, MV88E6390_G2_WDOG_CTL, &reg);
+       mv88e6xxx_g2_read(chip, MV88E6390_G2_WDOG_CTL, &reg);
 
        dev_info(chip->dev, "Watchdog history: 0x%04x",
                 reg & MV88E6390_G2_WDOG_CTL_DATA_MASK);
-- 
2.27.0.rc2

Reply via email to