Commit d3d71c99 authored by Dan Murphy's avatar Dan Murphy Committed by Mark Brown

ASoC: tas2770: Fix unbalanced calls to pm_runtime

Fix the unbalanced call to the pm_runtime_disable when removing the
module.  pm_runtime_enable is not called nor is the pm_runtime setup in
the code.  Remove the i2c_remove function and the pm_runtime_disable.

Fixes: 1a476abc ("tas2770: add tas2770 smart PA kernel driver")
Signed-off-by: default avatarDan Murphy <dmurphy@ti.com>
Link: https://lore.kernel.org/r/20200918190548.12598-5-dmurphy@ti.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent b23d9eb8
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
#include <linux/i2c.h> #include <linux/i2c.h>
#include <linux/gpio.h> #include <linux/gpio.h>
#include <linux/gpio/consumer.h> #include <linux/gpio/consumer.h>
#include <linux/pm_runtime.h>
#include <linux/regulator/consumer.h> #include <linux/regulator/consumer.h>
#include <linux/firmware.h> #include <linux/firmware.h>
#include <linux/regmap.h> #include <linux/regmap.h>
...@@ -781,13 +780,6 @@ static int tas2770_i2c_probe(struct i2c_client *client, ...@@ -781,13 +780,6 @@ static int tas2770_i2c_probe(struct i2c_client *client,
return result; return result;
} }
static int tas2770_i2c_remove(struct i2c_client *client)
{
pm_runtime_disable(&client->dev);
return 0;
}
static const struct i2c_device_id tas2770_i2c_id[] = { static const struct i2c_device_id tas2770_i2c_id[] = {
{ "tas2770", 0}, { "tas2770", 0},
{ } { }
...@@ -808,7 +800,6 @@ static struct i2c_driver tas2770_i2c_driver = { ...@@ -808,7 +800,6 @@ static struct i2c_driver tas2770_i2c_driver = {
.of_match_table = of_match_ptr(tas2770_of_match), .of_match_table = of_match_ptr(tas2770_of_match),
}, },
.probe = tas2770_i2c_probe, .probe = tas2770_i2c_probe,
.remove = tas2770_i2c_remove,
.id_table = tas2770_i2c_id, .id_table = tas2770_i2c_id,
}; };
......
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