Use pm_sleep_ptr/DEFINE_SIMPLE_DEV_PM_OPS to simplify the code.
Note: Apparently conditional compiling based on CONFIG_PM
wasn't optimal, it should have been CONFIG_PM_SLEEP.
Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx>
---
drivers/hwmon/jc42.c | 14 ++------------
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/hwmon/jc42.c b/drivers/hwmon/jc42.c
index f958e830b..5cd0b8cc6 100644
--- a/drivers/hwmon/jc42.c
+++ b/drivers/hwmon/jc42.c
@@ -570,8 +570,6 @@ static void jc42_remove(struct i2c_client *client)
}
}
-#ifdef CONFIG_PM
-
static int jc42_suspend(struct device *dev)
{
struct jc42_data *data = dev_get_drvdata(dev);
@@ -598,15 +596,7 @@ static int jc42_resume(struct device *dev)
return regcache_sync(data->regmap);
}
-static const struct dev_pm_ops jc42_dev_pm_ops = {
- .suspend = jc42_suspend,
- .resume = jc42_resume,
-};
-
-#define JC42_DEV_PM_OPS (&jc42_dev_pm_ops)
-#else
-#define JC42_DEV_PM_OPS NULL
-#endif /* CONFIG_PM */
+static DEFINE_SIMPLE_DEV_PM_OPS(jc42_pm_ops, jc42_suspend, jc42_resume);
static const struct i2c_device_id jc42_id[] = {
{ "jc42", 0 },
@@ -626,7 +616,7 @@ static struct i2c_driver jc42_driver = {
.class = I2C_CLASS_SPD | I2C_CLASS_HWMON,
.driver = {
.name = "jc42",
- .pm = JC42_DEV_PM_OPS,
+ .pm = pm_sleep_ptr(&jc42_pm_ops),
.of_match_table = of_match_ptr(jc42_of_ids),
},
.probe = jc42_probe,
--
2.42.1
