ASoC: codecs: rtq9128: Fix PM_RUNTIME usage
authorChiYuan Huang <cy_huang@richtek.com>
Fri, 29 Dec 2023 01:46:01 +0000 (09:46 +0800)
committerMark Brown <broonie@kernel.org>
Fri, 5 Jan 2024 15:20:46 +0000 (15:20 +0000)
If 'pm_runtime_resume_and_get' is used, must check the return value to
prevent the active count not matched problem.

Signed-off-by: ChiYuan Huang <cy_huang@richtek.com>
Link: https://msgid.link/r/bebd9e2bed9e0528a7fd9c528d785da02caf4f1a.1703813842.git.cy_huang@richtek.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/rtq9128.c

index c22b047115cc47217d6455697014503d5a6aa4a1..bda64f9eeb624c745f8dd473f2d5307989f22d35 100644 (file)
@@ -391,7 +391,11 @@ static int rtq9128_component_probe(struct snd_soc_component *comp)
        unsigned int val;
        int i, ret;
 
-       pm_runtime_resume_and_get(comp->dev);
+       ret = pm_runtime_resume_and_get(comp->dev);
+       if (ret < 0) {
+               dev_err(comp->dev, "Failed to resume device (%d)\n", ret);
+               return ret;
+       }
 
        val = snd_soc_component_read(comp, RTQ9128_REG_EFUSE_DATA);