HI, > > On Mon, Nov 23, 2020 at 06:20:00PM +0800, Zhang Qilong wrote: > > pm_runtime_get_sync will increment pm usage counter even it failed. > > Forgetting to putting operation will result in reference leak here. We > > fix it by replacing it with pm_runtime_resume_and_get to keep usage > > counter balanced. > > > Turns out this function doesn't exist in the mainline kernel, meaning it can not > be used to fix a patch in the mainline kernel. > I dropped this patch from my queue. > I am sorry ,I forgot to add the dependency, the commit is as flowing: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v5.10-rc6&id=dd8088d5a8969dc2b42f71d7bc01c25c61a78066 I will update the description next version. Best regards, Zhang Qilong > Guenter > > > Fixes: 323aeb0eb5d9a ("hwmon: (ina3221) Add PM runtime support") > > Signed-off-by: Zhang Qilong <zhangqilong3@xxxxxxxxxx> > > --- > > drivers/hwmon/ina3221.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/hwmon/ina3221.c b/drivers/hwmon/ina3221.c index > > 72cfdc007e60..d80bd3efcd6d 100644 > > --- a/drivers/hwmon/ina3221.c > > +++ b/drivers/hwmon/ina3221.c > > @@ -489,7 +489,7 @@ static int ina3221_write_enable(struct device > > *dev, int channel, bool enable) > > > > /* For enabling routine, increase refcount and resume() at first */ > > if (enable) { > > - ret = pm_runtime_get_sync(ina->pm_dev); > > + ret = pm_runtime_resume_and_get(ina->pm_dev); > > if (ret < 0) { > > dev_err(dev, "Failed to get PM runtime\n"); > > return ret;
