Skip to content

Commit

Permalink
power: supply: rk817: stop updating info in suspend
Browse files Browse the repository at this point in the history
The driver has a thread that checks the battery every 8 seconds. Stop
this thread during device suspend as while the device is suspended not
all values seem to be read correctly (such as battery voltage). The
resume function triggers the thread to start again.

Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
Link: https://lore.kernel.org/r/20240926144346.94630-2-macroalpha82@gmail.com
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
  • Loading branch information
Chris Morgan authored and Sebastian Reichel committed Oct 16, 2024
1 parent c46a9ee commit bded860
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion drivers/power/supply/rk817_charger.c
Original file line number Diff line number Diff line change
Expand Up @@ -1202,6 +1202,15 @@ static int rk817_charger_probe(struct platform_device *pdev)
return 0;
}

static int __maybe_unused rk817_suspend(struct device *dev)
{
struct rk817_charger *charger = dev_get_drvdata(dev);

cancel_delayed_work_sync(&charger->work);

return 0;
}

static int __maybe_unused rk817_resume(struct device *dev)
{

Expand All @@ -1213,7 +1222,7 @@ static int __maybe_unused rk817_resume(struct device *dev)
return 0;
}

static SIMPLE_DEV_PM_OPS(rk817_charger_pm, NULL, rk817_resume);
static SIMPLE_DEV_PM_OPS(rk817_charger_pm, rk817_suspend, rk817_resume);

static struct platform_driver rk817_charger_driver = {
.probe = rk817_charger_probe,
Expand Down

0 comments on commit bded860

Please sign in to comment.