Thread (7 messages) 7 messages, 3 authors, 2011-01-21

RE: [PATCH 2/3] Input: ad7879 - Convert SPI to dev_pm_ops

From: "Hennerich, Michael" <Michael.Hennerich@analog.com>
Date: 2011-01-20 21:19:43

Mark Brown wrote on 2011-01-20:
There is a move to deprecate bus-specific PM operations and move to
using dev_pm_ops instead in order to reduce the amount of boilerplate
code in buses and facilitiate updates to the PM core. Do this move for
the ad7979 SPI driver.

Signed-off-by: Mark Brown <redacted>
Acked-by: "Hennerich, Michael" <Michael.Hennerich@analog.com>
quoted hunk ↗ jump to hunk
---
 drivers/input/touchscreen/ad7879-spi.c |   17 +++++++++--------
 1 files changed, 9 insertions(+), 8 deletions(-)
diff --git a/drivers/input/touchscreen/ad7879-spi.c
b/drivers/input/touchscreen/ad7879-spi.c index 59c6e68..ddf732f 100644
--- a/drivers/input/touchscreen/ad7879-spi.c +++
b/drivers/input/touchscreen/ad7879-spi.c @@ -7,6 +7,7 @@
  */
 #include <linux/input.h>     /* BUS_SPI */ +#include <linux/pm.h> #include
 <linux/spi/spi.h>

 #include "ad7879.h" @@ -20,9 +21,10 @@ #define AD7879_WRITECMD(reg)
 (AD7879_CMD(reg))  #define
AD7879_READCMD(reg)  (AD7879_CMD(reg) | AD7879_CMD_READ)

-#ifdef CONFIG_PM -static int ad7879_spi_suspend(struct spi_device *spi,
pm_message_t message) +#ifdef CONFIG_PM_SLEEP +static int
ad7879_spi_suspend(struct device *dev)
 { +  struct spi_device *spi = to_spi_device(dev);    struct ad7879 *ts =
 spi_get_drvdata(spi);

      ad7879_suspend(ts);
@@ -30,19 +32,19 @@ static int ad7879_spi_suspend(struct spi_device
*spi, pm_message_t message)
      return 0;
 }
-static int ad7879_spi_resume(struct spi_device *spi)
+static int ad7879_spi_resume(struct device *dev)
 { +  struct spi_device *spi = to_spi_device(dev);    struct ad7879 *ts =
 spi_get_drvdata(spi);

      ad7879_resume(ts);

      return 0;
 }
-#else
-# define ad7879_spi_suspend NULL
-# define ad7879_spi_resume  NULL
 #endif
+static SIMPLE_DEV_PM_OPS(ad7879_spi_pm, ad7879_spi_suspend,
+ad7879_spi_resume); +
 /*
  * ad7879_read/write are only used for initial setup and for sysfs
  controls. * The main traffic is done in ad7879_collect().
@@ -173,11 +175,10 @@ static struct spi_driver ad7879_spi_driver = {
              .name   = "ad7879",             .bus    = &spi_bus_type,                .owner  = THIS_MODULE,
 +            .pm     = &ad7879_spi_pm,       },      .probe          = ad7879_spi_probe,     .remove         =
 __devexit_p(ad7879_spi_remove),
-     .suspend        = ad7879_spi_suspend,
-     .resume         = ad7879_spi_resume,
 };

 static int __init ad7879_spi_init(void)
Greetings,
Michael

--
Analog Devices GmbH      Wilhelm-Wagenfeld-Str. 6      80807 Muenchen
Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, Margaret Seif

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help