backlight: adp8870: convert adp8870 to dev_pm_ops

Instead of using legacy suspend/resume methods, using newer dev_pm_ops
structure allows better control over power management.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
index 7930ab3ff4deacb96092c1bf0887d46eddc79bef..302c8009a7dc44783a705a332ffa69bcc3534a9a 100644 (file)
--- a/drivers/video/backlight/adp8870_bl.c
+++ b/drivers/video/backlight/adp8870_bl.c
@@ -947,25 +947,29 @@ static int adp8870_remove(struct i2c_client *client)
        return 0;
 }
 
-#ifdef CONFIG_PM
-static int adp8870_i2c_suspend(struct i2c_client *client, pm_message_t message)
+#ifdef CONFIG_PM_SLEEP
+static int adp8870_i2c_suspend(struct device *dev)
 {
+       struct i2c_client *client = to_i2c_client(dev);
+
        adp8870_clr_bits(client, ADP8870_MDCR, NSTBY);
 
        return 0;
 }
 
-static int adp8870_i2c_resume(struct i2c_client *client)
+static int adp8870_i2c_resume(struct device *dev)
 {
+       struct i2c_client *client = to_i2c_client(dev);
+
        adp8870_set_bits(client, ADP8870_MDCR, NSTBY | BLEN);
 
        return 0;
 }
-#else
-#define adp8870_i2c_suspend NULL
-#define adp8870_i2c_resume NULL
 #endif
 
+static SIMPLE_DEV_PM_OPS(adp8870_i2c_pm_ops, adp8870_i2c_suspend,
+                       adp8870_i2c_resume);
+
 static const struct i2c_device_id adp8870_id[] = {
        { "adp8870", 0 },
        { }
@@ -974,12 +978,11 @@ MODULE_DEVICE_TABLE(i2c, adp8870_id);
 
 static struct i2c_driver adp8870_driver = {
        .driver = {
-               .name = KBUILD_MODNAME,
+               .name   = KBUILD_MODNAME,
+               .pm     = &adp8870_i2c_pm_ops,
        },
        .probe    = adp8870_probe,
        .remove   = adp8870_remove,
-       .suspend = adp8870_i2c_suspend,
-       .resume  = adp8870_i2c_resume,
        .id_table = adp8870_id,
 };