Commit eb39ad25 authored by Jingoo Han's avatar Jingoo Han Committed by Linus Torvalds

backlight: vgg2432a4: convert vgg2432a4_driver to dev_pm_ops

Instead of using legacy suspend/resume methods, using newer dev_pm_ops
structure allows better control over power management. Also, use of
pm_message_t is deprecated. Thus, it is removed.
Signed-off-by: default avatarJingoo Han <jg1.han@samsung.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 1dde37d4
...@@ -270,27 +270,21 @@ int ili9320_remove(struct ili9320 *ili) ...@@ -270,27 +270,21 @@ int ili9320_remove(struct ili9320 *ili)
} }
EXPORT_SYMBOL_GPL(ili9320_remove); EXPORT_SYMBOL_GPL(ili9320_remove);
#ifdef CONFIG_PM #ifdef CONFIG_PM_SLEEP
int ili9320_suspend(struct ili9320 *lcd, pm_message_t state) int ili9320_suspend(struct ili9320 *lcd)
{ {
int ret; int ret;
dev_dbg(lcd->dev, "%s: event %d\n", __func__, state.event); ret = ili9320_power(lcd, FB_BLANK_POWERDOWN);
if (state.event == PM_EVENT_SUSPEND) { if (lcd->platdata->suspend == ILI9320_SUSPEND_DEEP) {
ret = ili9320_power(lcd, FB_BLANK_POWERDOWN); ili9320_write(lcd, ILI9320_POWER1, lcd->power1 |
ILI9320_POWER1_SLP |
if (lcd->platdata->suspend == ILI9320_SUSPEND_DEEP) { ILI9320_POWER1_DSTB);
ili9320_write(lcd, ILI9320_POWER1, lcd->power1 | lcd->initialised = 0;
ILI9320_POWER1_SLP |
ILI9320_POWER1_DSTB);
lcd->initialised = 0;
}
return ret;
} }
return 0; return ret;
} }
EXPORT_SYMBOL_GPL(ili9320_suspend); EXPORT_SYMBOL_GPL(ili9320_suspend);
......
...@@ -76,5 +76,5 @@ extern void ili9320_shutdown(struct ili9320 *lcd); ...@@ -76,5 +76,5 @@ extern void ili9320_shutdown(struct ili9320 *lcd);
/* PM */ /* PM */
extern int ili9320_suspend(struct ili9320 *lcd, pm_message_t state); extern int ili9320_suspend(struct ili9320 *lcd);
extern int ili9320_resume(struct ili9320 *lcd); extern int ili9320_resume(struct ili9320 *lcd);
...@@ -205,18 +205,15 @@ static int vgg2432a4_lcd_init(struct ili9320 *lcd, ...@@ -205,18 +205,15 @@ static int vgg2432a4_lcd_init(struct ili9320 *lcd,
return ret; return ret;
} }
#ifdef CONFIG_PM #ifdef CONFIG_PM_SLEEP
static int vgg2432a4_suspend(struct spi_device *spi, pm_message_t state) static int vgg2432a4_suspend(struct device *dev)
{ {
return ili9320_suspend(spi_get_drvdata(spi), state); return ili9320_suspend(dev_get_drvdata(dev));
} }
static int vgg2432a4_resume(struct spi_device *spi) static int vgg2432a4_resume(struct device *dev)
{ {
return ili9320_resume(spi_get_drvdata(spi)); return ili9320_resume(dev_get_drvdata(dev));
} }
#else
#define vgg2432a4_suspend NULL
#define vgg2432a4_resume NULL
#endif #endif
static struct ili9320_client vgg2432a4_client = { static struct ili9320_client vgg2432a4_client = {
...@@ -249,16 +246,17 @@ static void vgg2432a4_shutdown(struct spi_device *spi) ...@@ -249,16 +246,17 @@ static void vgg2432a4_shutdown(struct spi_device *spi)
ili9320_shutdown(spi_get_drvdata(spi)); ili9320_shutdown(spi_get_drvdata(spi));
} }
static SIMPLE_DEV_PM_OPS(vgg2432a4_pm_ops, vgg2432a4_suspend, vgg2432a4_resume);
static struct spi_driver vgg2432a4_driver = { static struct spi_driver vgg2432a4_driver = {
.driver = { .driver = {
.name = "VGG2432A4", .name = "VGG2432A4",
.owner = THIS_MODULE, .owner = THIS_MODULE,
.pm = &vgg2432a4_pm_ops,
}, },
.probe = vgg2432a4_probe, .probe = vgg2432a4_probe,
.remove = vgg2432a4_remove, .remove = vgg2432a4_remove,
.shutdown = vgg2432a4_shutdown, .shutdown = vgg2432a4_shutdown,
.suspend = vgg2432a4_suspend,
.resume = vgg2432a4_resume,
}; };
module_spi_driver(vgg2432a4_driver); module_spi_driver(vgg2432a4_driver);
......
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