On Fri, Aug 21, 2015 at 10:25:59AM +0200, Takashi Iwai wrote:
> On Fri, 21 Aug 2015 09:48:33 +0200,
> Lars-Peter Clausen wrote:
> > 
> > Convert the ac97_bus from legacy suspend/resume callbacks to dev_pm_ops.
> > 
> > Since there isn't anything special to do at the bus level the bus driver
> > does not have to implement any callbacks. The device driver core will
> > automatically pick up and execute the device's PM ops.
> > 
> > As there is only a single AC'97 driver implementing suspend and resume,
> > update both the core and driver at the same time to avoid unnecessary code
> > churn.
> > 
> > While we are at it also drop the ifdefs around the suspend/resume functions
> > to increase compile test coverage.
> > 
> > Signed-off-by: Lars-Peter Clausen <l...@metafoo.de>
> > ---
> > This patch touches code in both ALSA and the input subsystem. Ideally this
> > patch will be merged via the ALSA tree. Given that the wm97xx touchscreen
> > driver is not seeing too many changes these days the risk of conflicts
> > should be low.
> 
> If Dmitry is happy with this, I'll take this.

Sounds good to me.

Acked-by: Dmitry Torokhov <dmitry.torok...@gmail.com>

Thanks!

> 
> 
> thanks,
> 
> Takashi
> 
> > 
> > Changes since v1:
> >     * Drop ifdefs around suspend/resum functions
> > ---
> >  drivers/input/touchscreen/wm97xx-core.c | 13 ++++---------
> >  sound/ac97_bus.c                        | 26 --------------------------
> >  2 files changed, 4 insertions(+), 35 deletions(-)
> > 
> > diff --git a/drivers/input/touchscreen/wm97xx-core.c 
> > b/drivers/input/touchscreen/wm97xx-core.c
> > index b1ae779..1534e9b 100644
> > --- a/drivers/input/touchscreen/wm97xx-core.c
> > +++ b/drivers/input/touchscreen/wm97xx-core.c
> > @@ -732,8 +732,7 @@ static int wm97xx_remove(struct device *dev)
> >     return 0;
> >  }
> >  
> > -#ifdef CONFIG_PM
> > -static int wm97xx_suspend(struct device *dev, pm_message_t state)
> > +static int __maybe_unused wm97xx_suspend(struct device *dev)
> >  {
> >     struct wm97xx *wm = dev_get_drvdata(dev);
> >     u16 reg;
> > @@ -765,7 +764,7 @@ static int wm97xx_suspend(struct device *dev, 
> > pm_message_t state)
> >     return 0;
> >  }
> >  
> > -static int wm97xx_resume(struct device *dev)
> > +static int __maybe_unused wm97xx_resume(struct device *dev)
> >  {
> >     struct wm97xx *wm = dev_get_drvdata(dev);
> >  
> > @@ -799,10 +798,7 @@ static int wm97xx_resume(struct device *dev)
> >     return 0;
> >  }
> >  
> > -#else
> > -#define wm97xx_suspend             NULL
> > -#define wm97xx_resume              NULL
> > -#endif
> > +static SIMPLE_DEV_PM_OPS(wm97xx_pm_ops, wm97xx_suspend, wm97xx_resume);
> >  
> >  /*
> >   * Machine specific operations
> > @@ -836,8 +832,7 @@ static struct device_driver wm97xx_driver = {
> >     .owner =        THIS_MODULE,
> >     .probe =        wm97xx_probe,
> >     .remove =       wm97xx_remove,
> > -   .suspend =      wm97xx_suspend,
> > -   .resume =       wm97xx_resume,
> > +   .pm =           &wm97xx_pm_ops,
> >  };
> >  
> >  static int __init wm97xx_init(void)
> > diff --git a/sound/ac97_bus.c b/sound/ac97_bus.c
> > index 2b50cbe..57a6dfc 100644
> > --- a/sound/ac97_bus.c
> > +++ b/sound/ac97_bus.c
> > @@ -27,35 +27,9 @@ static int ac97_bus_match(struct device *dev, struct 
> > device_driver *drv)
> >     return 1;
> >  }
> >  
> > -#ifdef CONFIG_PM
> > -static int ac97_bus_suspend(struct device *dev, pm_message_t state)
> > -{
> > -   int ret = 0;
> > -
> > -   if (dev->driver && dev->driver->suspend)
> > -           ret = dev->driver->suspend(dev, state);
> > -
> > -   return ret;
> > -}
> > -
> > -static int ac97_bus_resume(struct device *dev)
> > -{
> > -   int ret = 0;
> > -
> > -   if (dev->driver && dev->driver->resume)
> > -           ret = dev->driver->resume(dev);
> > -
> > -   return ret;
> > -}
> > -#endif /* CONFIG_PM */
> > -
> >  struct bus_type ac97_bus_type = {
> >     .name           = "ac97",
> >     .match          = ac97_bus_match,
> > -#ifdef CONFIG_PM
> > -   .suspend        = ac97_bus_suspend,
> > -   .resume         = ac97_bus_resume,
> > -#endif /* CONFIG_PM */
> >  };
> >  
> >  static int __init ac97_bus_init(void)
> > -- 
> > 2.1.4
> > 
> > 

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to