0e4f336f50
The pm_runtime_put_noidle() call in remove isn't balanced with any get, so drop it. Note this isn't a bug as the runtime pm core will not allow the reference count to go negative, making this a noop. However, it is confusing to the reader so let's drop it. pm_runtime_resume_and_get() replacement found using the coccicheck script under review at: https://lore.kernel.org/lkml/20210427141946.2478411-1-Julia.Lawall@inria.fr/ As pm_runtime_resume_and_get() returns <= 0 take advantage of that to change the error checking to if (ret) which is more in keeping with the rest of this driver. This is a prequel to taking a closer look at the runtime pm in IIO drivers in general. Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Cc: Matt Ranostay <matt.ranostay@konsulko.com> Acked-by: Matt Ranostay <matt.ranostay@konsulko.com> Reviewed-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> Link: https://lore.kernel.org/r/20210516162103.1332291-4-jic23@kernel.org |
||
|---|---|---|
| .. | ||
| Kconfig | ||
| Makefile | ||
| ams-iaq-core.c | ||
| atlas-ezo-sensor.c | ||
| atlas-sensor.c | ||
| bme680.h | ||
| bme680_core.c | ||
| bme680_i2c.c | ||
| bme680_spi.c | ||
| ccs811.c | ||
| pms7003.c | ||
| scd30.h | ||
| scd30_core.c | ||
| scd30_i2c.c | ||
| scd30_serial.c | ||
| sgp30.c | ||
| sps30.c | ||
| sps30.h | ||
| sps30_i2c.c | ||
| sps30_serial.c | ||
| vz89x.c | ||