From: Hans de Goede Date: Thu, 6 Jan 2022 11:06:07 +0000 (+0100) Subject: power: supply: axp288_fuel_gauge: Take lock before updating the valid flag X-Git-Tag: v6.1-rc5~1723^2~64 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=30abb3d07929137bf72327560e1595508a692c4e;p=platform%2Fkernel%2Flinux-starfive.git power: supply: axp288_fuel_gauge: Take lock before updating the valid flag The valid flag is protected by the mutex, so code clearing it should take the mutex before cleating the valid flag. Signed-off-by: Hans de Goede Signed-off-by: Sebastian Reichel --- diff --git a/drivers/power/supply/axp288_fuel_gauge.c b/drivers/power/supply/axp288_fuel_gauge.c index f7dce02..53d0e82 100644 --- a/drivers/power/supply/axp288_fuel_gauge.c +++ b/drivers/power/supply/axp288_fuel_gauge.c @@ -476,7 +476,9 @@ static irqreturn_t fuel_gauge_thread_handler(int irq, void *dev) dev_warn(info->dev, "Spurious Interrupt!!!\n"); } + mutex_lock(&info->lock); info->valid = 0; /* Force updating of the cached registers */ + mutex_unlock(&info->lock); power_supply_changed(info->bat); return IRQ_HANDLED; @@ -486,7 +488,9 @@ static void fuel_gauge_external_power_changed(struct power_supply *psy) { struct axp288_fg_info *info = power_supply_get_drvdata(psy); + mutex_lock(&info->lock); info->valid = 0; /* Force updating of the cached registers */ + mutex_unlock(&info->lock); power_supply_changed(info->bat); }