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