ASoC: rt5670: Remove unbalanced pm_runtime_put()
authorHans de Goede <hdegoede@redhat.com>
Tue, 13 Dec 2022 12:33:19 +0000 (13:33 +0100)
committerMark Brown <broonie@kernel.org>
Tue, 13 Dec 2022 18:16:31 +0000 (18:16 +0000)
For some reason rt5670_i2c_probe() does a pm_runtime_put() at the end
of a successful probe. But it has never done a pm_runtime_get() leading
to the following error being logged into dmesg:

 rt5670 i2c-10EC5640:00: Runtime PM usage count underflow!

Fix this by removing the unnecessary pm_runtime_put().

Fixes: 64e89e5f5548 ("ASoC: rt5670: Add runtime PM support")
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20221213123319.11285-1-hdegoede@redhat.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/rt5670.c

index ebac6ca..a230f44 100644 (file)
@@ -3311,8 +3311,6 @@ static int rt5670_i2c_probe(struct i2c_client *i2c)
        if (ret < 0)
                goto err;
 
-       pm_runtime_put(&i2c->dev);
-
        return 0;
 err:
        pm_runtime_disable(&i2c->dev);