diff options
| author | Hans de Goede <hdegoede@redhat.com> | 2022-12-13 13:33:19 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-01-07 12:07:34 +0100 |
| commit | 525f1c8ada75acb0de84c7aa8c0c031edb8c0105 (patch) | |
| tree | 70762bbc587334d9634909b7179a613de74b5cf6 | |
| parent | 593f0c94dcb4471556d1aecf457c1215b2e7490d (diff) | |
| download | linux-stable-525f1c8ada75acb0de84c7aa8c0c031edb8c0105.tar.gz linux-stable-525f1c8ada75acb0de84c7aa8c0c031edb8c0105.tar.bz2 linux-stable-525f1c8ada75acb0de84c7aa8c0c031edb8c0105.zip | |
ASoC: rt5670: Remove unbalanced pm_runtime_put()
[ Upstream commit 6c900dcc3f7331a67ed29739d74524e428d137fb ]
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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | sound/soc/codecs/rt5670.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sound/soc/codecs/rt5670.c b/sound/soc/codecs/rt5670.c index fdc14e50d3b9..13b944bcde71 100644 --- a/sound/soc/codecs/rt5670.c +++ b/sound/soc/codecs/rt5670.c @@ -3028,8 +3028,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); |