Hi,
I'm not sure that I follow why st1232_ts_pm_ops is needed
if CONFIG_PM_SLEEP isn't defined.
On Fri, Mar 30, 2012 at 11:18:02PM -0700, Dmitry Torokhov wrote:
> Signed-off-by: Dmitry Torokhov <dtor@xxxxxxx>
> ---
>
> Just compiled, not tested...
>
> drivers/input/touchscreen/st1232.c | 11 ++++-------
> 1 files changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/input/touchscreen/st1232.c b/drivers/input/touchscreen/st1232.c
> index e22e27e..a9be39f 100644
> --- a/drivers/input/touchscreen/st1232.c
> +++ b/drivers/input/touchscreen/st1232.c
> @@ -218,7 +218,7 @@ static int __devexit st1232_ts_remove(struct i2c_client *client)
> return 0;
> }
>
> -#ifdef CONFIG_PM
> +#ifdef CONFIG_PM_SLEEP
> static int st1232_ts_suspend(struct device *dev)
> {
> struct i2c_client *client = to_i2c_client(dev);
> @@ -243,12 +243,11 @@ static int st1232_ts_resume(struct device *dev)
> return 0;
> }
>
> -static const struct dev_pm_ops st1232_ts_pm_ops = {
> - .suspend = st1232_ts_suspend,
> - .resume = st1232_ts_resume,
> -};
> #endif
>
> +static SIMPLE_DEV_PM_OPS(st1232_ts_pm_ops,
> + st1232_ts_suspend, st1232_ts_resume);
> +
> static const struct i2c_device_id st1232_ts_id[] = {
> { ST1232_TS_NAME, 0 },
> { }
> @@ -269,9 +268,7 @@ static struct i2c_driver st1232_ts_driver = {
> .name = ST1232_TS_NAME,
> .owner = THIS_MODULE,
> .of_match_table = st1232_ts_dt_ids,
> -#ifdef CONFIG_PM
> .pm = &st1232_ts_pm_ops,
> -#endif
> },
> };
>
> --
> 1.7.7.6
>
>
> --
> Dmitry
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
