spi: tegra20-sflash: fix reference leak in tegra_sflash_resume
authorZhang Qilong <zhangqilong3@huawei.com>
Tue, 3 Nov 2020 14:13:23 +0000 (22:13 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 9 Nov 2020 19:44:28 +0000 (19:44 +0000)
pm_runtime_get_sync will increment pm usage counter even it
failed. Forgetting to pm_runtime_put_noidle will result in
reference leak in tegra_sflash_resume, so we should fix it.

Fixes: 8528547bcc336 ("spi: tegra: add spi driver for sflash controller")
Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com>
Link: https://lore.kernel.org/r/20201103141323.5841-1-zhangqilong3@huawei.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-tegra20-sflash.c

index b59015c7c8a80405f523efb1299b337bb1a3839a..cfb7de73793763023ca42645c25c79359820af5d 100644 (file)
@@ -552,6 +552,7 @@ static int tegra_sflash_resume(struct device *dev)
 
        ret = pm_runtime_get_sync(dev);
        if (ret < 0) {
+               pm_runtime_put_noidle(dev);
                dev_err(dev, "pm runtime failed, e = %d\n", ret);
                return ret;
        }