Commit d1b4c755 authored by Guenter Roeck's avatar Guenter Roeck

hwmon: (spd5118) Add suspend/resume support

Add suspend/resume support to ensure that limit and configuration
registers are updated and synchronized after a suspend/resume cycle.

Cc: Armin Wolf <W_Armin@gmx.de>
Cc: Stephen Horvath <s.horvath@outlook.com.au>
Reviewed-by: default avatarThomas Weißschuh <linux@weissschuh.net>
Tested-by: default avatarStephen Horvath <s.horvath@outlook.com.au>
Tested-by: default avatarArmin Wolf <W_Armin@gmx.de>
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
parent 09262e98
...@@ -20,6 +20,7 @@ ...@@ -20,6 +20,7 @@
#include <linux/i2c.h> #include <linux/i2c.h>
#include <linux/hwmon.h> #include <linux/hwmon.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/pm.h>
#include <linux/regmap.h> #include <linux/regmap.h>
#include <linux/units.h> #include <linux/units.h>
...@@ -432,6 +433,8 @@ static int spd5118_probe(struct i2c_client *client) ...@@ -432,6 +433,8 @@ static int spd5118_probe(struct i2c_client *client)
if (!spd5118_vendor_valid(bank, vendor)) if (!spd5118_vendor_valid(bank, vendor))
return -ENODEV; return -ENODEV;
dev_set_drvdata(dev, regmap);
hwmon_dev = devm_hwmon_device_register_with_info(dev, "spd5118", hwmon_dev = devm_hwmon_device_register_with_info(dev, "spd5118",
regmap, &spd5118_chip_info, regmap, &spd5118_chip_info,
NULL); NULL);
...@@ -449,6 +452,41 @@ static int spd5118_probe(struct i2c_client *client) ...@@ -449,6 +452,41 @@ static int spd5118_probe(struct i2c_client *client)
return 0; return 0;
} }
static int spd5118_suspend(struct device *dev)
{
struct regmap *regmap = dev_get_drvdata(dev);
u32 regval;
int err;
/*
* Make sure the configuration register in the regmap cache is current
* before bypassing it.
*/
err = regmap_read(regmap, SPD5118_REG_TEMP_CONFIG, &regval);
if (err < 0)
return err;
regcache_cache_bypass(regmap, true);
regmap_update_bits(regmap, SPD5118_REG_TEMP_CONFIG, SPD5118_TS_DISABLE,
SPD5118_TS_DISABLE);
regcache_cache_bypass(regmap, false);
regcache_cache_only(regmap, true);
regcache_mark_dirty(regmap);
return 0;
}
static int spd5118_resume(struct device *dev)
{
struct regmap *regmap = dev_get_drvdata(dev);
regcache_cache_only(regmap, false);
return regcache_sync(regmap);
}
static DEFINE_SIMPLE_DEV_PM_OPS(spd5118_pm_ops, spd5118_suspend, spd5118_resume);
static const struct i2c_device_id spd5118_id[] = { static const struct i2c_device_id spd5118_id[] = {
{ "spd5118", 0 }, { "spd5118", 0 },
{ } { }
...@@ -466,6 +504,7 @@ static struct i2c_driver spd5118_driver = { ...@@ -466,6 +504,7 @@ static struct i2c_driver spd5118_driver = {
.driver = { .driver = {
.name = "spd5118", .name = "spd5118",
.of_match_table = spd5118_of_ids, .of_match_table = spd5118_of_ids,
.pm = pm_sleep_ptr(&spd5118_pm_ops),
}, },
.probe = spd5118_probe, .probe = spd5118_probe,
.id_table = spd5118_id, .id_table = spd5118_id,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment