Re: [PATCH] iio: proximity: srf04: fix runtime PM autosuspend cleanup
Johan Hovold <[email protected]>
| Newsgroups | org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Aug 07, 2026 at 10:19:37PM +0800, Guangshuo Li wrote: > srf04_probe() calls pm_runtime_use_autosuspend() when the optional > power GPIO is present, but srf04_remove() does not call the matching > pm_runtime_dont_use_autosuspend() before disabling runtime PM. > > The runtime PM documentation requires pm_runtime_use_autosuspend() to > be balanced with pm_runtime_dont_use_autosuspend() when the driver is > removed. Failing to do so can leave the autosuspend usage state > unbalanced and may result in a usage_count leak when the autosuspend > delay is negative. I believe the term "leak" here is misleading as the usage count would be dropped again when the user re-enables autosuspend through sysfs, right? That said, drivers should clean up after themselves so disabling autosuspend is still the right to do. But it's more of a cleanup than a fix (and does not need to be backported). Johan