mt76: mt7615: do not update pm stats in case of error
authorLorenzo Bianconi <lorenzo@kernel.org>
Mon, 13 Jun 2022 12:03:28 +0000 (14:03 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 17 Aug 2022 12:23:29 +0000 (14:23 +0200)
[ Upstream commit 79717c4eeeae9dec894794fbe8af72f08f03ebdd ]

Do not update pm stats if mt7615_mcu_fw_pmctrl returns an error.

Fixes: abe912ae3cd42 ("mt76: mt7663: add awake and doze time accounting")
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/wireless/mediatek/mt76/mt7615/mcu.c

index 4fed3af..bde65af 100644 (file)
@@ -385,10 +385,11 @@ static int mt7615_mcu_fw_pmctrl(struct mt7615_dev *dev)
        }
 
        mt7622_trigger_hif_int(dev, false);
-
-       pm->stats.last_doze_event = jiffies;
-       pm->stats.awake_time += pm->stats.last_doze_event -
-                               pm->stats.last_wake_event;
+       if (!err) {
+               pm->stats.last_doze_event = jiffies;
+               pm->stats.awake_time += pm->stats.last_doze_event -
+                                       pm->stats.last_wake_event;
+       }
 out:
        mutex_unlock(&pm->mutex);