Re: [PATCH 0/2] iio: Convert some drivers from deprecated UNIVERSAL_DEV_PM_OPS()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Aug 7, 2022 at 8:59 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
>
> During discussions of the new PM macro definitions, it because clear that
> UNIVERSAL_DEV_PM_OPS() generally doesn't make much sense as it can lead
> to unnecessary work.
>
> For the drivers in this set I've taken a look at how the ops are used
> and believe we can just switch to DEFINE_RUNTIME_PM_OPS() which will only
> do the suspend if the device is not already runtime suspended.
>
> I'd like some review of these (more than for the straight forward NOP conversions).

Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>

> Thanks,
>
> Jonathan
>
>
> Jonathan Cameron (2):
>   iio: pressure: icp10100: Switch from UNIVERSAL to
>     DEFINE_RUNTIME_DEV_PM_OPS().
>   iio: temp: mlx90632: Switch form UNVIVERSAL to
>     DEFINE_RUNTIME_DEV_PM_OPS()
>
>  drivers/iio/pressure/icp10100.c    | 10 +++++-----
>  drivers/iio/temperature/mlx90632.c | 10 +++++-----
>  2 files changed, 10 insertions(+), 10 deletions(-)
>
> --
> 2.37.1
>


-- 
With Best Regards,
Andy Shevchenko



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux