[PATCH] iio: light: apds9960: add system-wide suspend

[PATCH] iio: light: apds9960: add system-wide suspend

[Matt Ranostay]

APDS9960 can safefully force runtime suspend if the system wants
to enter system-wide suspend

Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
---
 drivers/iio/light/apds9960.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
index a4304edc3e0f..67da1287e987 100644
--- a/drivers/iio/light/apds9960.c
+++ b/drivers/iio/light/apds9960.c
@@ -1112,6 +1112,8 @@ static int apds9960_runtime_resume(struct device *dev)
 #endif
 
 static const struct dev_pm_ops apds9960_pm_ops = {
+	SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
+				pm_runtime_force_resume)
 	SET_RUNTIME_PM_OPS(apds9960_runtime_suspend,
 			   apds9960_runtime_resume, NULL)
 };
-- 
2.11.0

Re: [PATCH] iio: light: apds9960: add system-wide suspend

[Matt Ranostay]

On Wed, Apr 12, 2017 at 7:16 PM, Matt Ranostay
<matt.ranostay@konsulko.com> wrote:
> APDS9960 can safefully force runtime suspend if the system wants

Yikes! Typo should be -> safely

> to enter system-wide suspend
>
> Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
> ---
>  drivers/iio/light/apds9960.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
> index a4304edc3e0f..67da1287e987 100644
> --- a/drivers/iio/light/apds9960.c
> +++ b/drivers/iio/light/apds9960.c
> @@ -1112,6 +1112,8 @@ static int apds9960_runtime_resume(struct device *dev)
>  #endif
>
>  static const struct dev_pm_ops apds9960_pm_ops = {
> +       SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> +                               pm_runtime_force_resume)
>         SET_RUNTIME_PM_OPS(apds9960_runtime_suspend,
>                            apds9960_runtime_resume, NULL)
>  };
> --
> 2.11.0
>

Re: [PATCH] iio: light: apds9960: add system-wide suspend

[Jonathan Cameron]

On 13/04/17 19:21, Matt Ranostay wrote:
> On Wed, Apr 12, 2017 at 7:16 PM, Matt Ranostay
> <matt.ranostay@konsulko.com> wrote:
>> APDS9960 can safefully force runtime suspend if the system wants
> 
> Yikes! Typo should be -> safely
Applied and fixed up typo.

Jonathan
> 
>> to enter system-wide suspend
>>
>> Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
>> ---
>>  drivers/iio/light/apds9960.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
>> index a4304edc3e0f..67da1287e987 100644
>> --- a/drivers/iio/light/apds9960.c
>> +++ b/drivers/iio/light/apds9960.c
>> @@ -1112,6 +1112,8 @@ static int apds9960_runtime_resume(struct device *dev)
>>  #endif
>>
>>  static const struct dev_pm_ops apds9960_pm_ops = {
>> +       SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
>> +                               pm_runtime_force_resume)
>>         SET_RUNTIME_PM_OPS(apds9960_runtime_suspend,
>>                            apds9960_runtime_resume, NULL)
>>  };
>> --
>> 2.11.0
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>