Commit e65f2fce authored by Minghao Chi's avatar Minghao Chi Committed by Mark Brown

ASoC: codecs: wm8962: using pm_runtime_resume_and_get instead of pm_runtime_get_sync

Using pm_runtime_resume_and_get is more appropriate
for simplifing code
Reported-by: default avatarZeal Robot <zealci@zte.com.cn>
Signed-off-by: default avatarMinghao Chi <chi.minghao@zte.com.cn>
Acked-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220418110259.2559144-1-chi.minghao@zte.com.cnSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 0a8ff26d
...@@ -2896,9 +2896,8 @@ static int wm8962_set_fll(struct snd_soc_component *component, int fll_id, int s ...@@ -2896,9 +2896,8 @@ static int wm8962_set_fll(struct snd_soc_component *component, int fll_id, int s
reinit_completion(&wm8962->fll_lock); reinit_completion(&wm8962->fll_lock);
ret = pm_runtime_get_sync(component->dev); ret = pm_runtime_resume_and_get(component->dev);
if (ret < 0) { if (ret < 0) {
pm_runtime_put_noidle(component->dev);
dev_err(component->dev, "Failed to resume device: %d\n", ret); dev_err(component->dev, "Failed to resume device: %d\n", ret);
return ret; return ret;
} }
...@@ -3030,9 +3029,8 @@ static irqreturn_t wm8962_irq(int irq, void *data) ...@@ -3030,9 +3029,8 @@ static irqreturn_t wm8962_irq(int irq, void *data)
unsigned int active; unsigned int active;
int reg, ret; int reg, ret;
ret = pm_runtime_get_sync(dev); ret = pm_runtime_resume_and_get(dev);
if (ret < 0) { if (ret < 0) {
pm_runtime_put_noidle(dev);
dev_err(dev, "Failed to resume: %d\n", ret); dev_err(dev, "Failed to resume: %d\n", ret);
return IRQ_NONE; return IRQ_NONE;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment