diff options
| author | Dinghao Liu <dinghao.liu@zju.edu.cn> | 2021-04-07 13:11:49 +0800 |
|---|---|---|
| committer | Lee Jones <lee.jones@linaro.org> | 2021-04-14 16:06:40 +0100 |
| commit | fe6df2b48043bbe1e852b2320501d3b169363c35 (patch) | |
| tree | 526f248d5efbfde60680ece98fa9bb57ff0ad8cd /drivers/mfd/arizona-irq.c | |
| parent | 0b79c53e8a1c5c26c5a364c8f041ca6890a29f08 (diff) | |
| download | lwn-fe6df2b48043bbe1e852b2320501d3b169363c35.tar.gz lwn-fe6df2b48043bbe1e852b2320501d3b169363c35.zip | |
mfd: arizona: Fix rumtime PM imbalance on error
pm_runtime_get_sync() will increase the rumtime PM counter
even it returns an error. Thus a pairing decrement is needed
to prevent refcount leak. Fix this by replacing this API with
pm_runtime_resume_and_get(), which will not change the runtime
PM counter on error.
Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd/arizona-irq.c')
| -rw-r--r-- | drivers/mfd/arizona-irq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/arizona-irq.c b/drivers/mfd/arizona-irq.c index 077d9ab112b7..d919ae9691e2 100644 --- a/drivers/mfd/arizona-irq.c +++ b/drivers/mfd/arizona-irq.c @@ -100,7 +100,7 @@ static irqreturn_t arizona_irq_thread(int irq, void *data) unsigned int val; int ret; - ret = pm_runtime_get_sync(arizona->dev); + ret = pm_runtime_resume_and_get(arizona->dev); if (ret < 0) { dev_err(arizona->dev, "Failed to resume device: %d\n", ret); return IRQ_NONE; |