diff options
| author | Pierre-Louis Bossart <[email protected]> | 2020-05-15 16:15:31 -0500 |
|---|---|---|
| committer | Mark Brown <[email protected]> | 2020-05-18 12:38:30 +0100 |
| commit | ee5866222ab58531c988492ea54931c1346d4fd4 (patch) | |
| tree | d86bfb3e6eaba110dc017b0f535ebcbdcdb75074 | |
| parent | 07b542fe831cbefce163ad1b3aa7292c8a6332b8 (diff) | |
ASoC: codecs: rt1308-sdw: remove duplicate allocation
The .read_prop callback is supposed to be called by the SoundWire core
only. Calling it again from this driver results in an additional
memory allocation for no good reason.
Signed-off-by: Pierre-Louis Bossart <[email protected]>
Reviewed-by: Guennadi Liakhovetski <[email protected]>
Reviewed-by: Rander Wang <[email protected]>
Cc: Oder Chiou <[email protected]>
Cc: Shuming Fan <[email protected]>
Cc: Jack Yu <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Mark Brown <[email protected]>
| -rw-r--r-- | sound/soc/codecs/rt1308-sdw.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/sound/soc/codecs/rt1308-sdw.c b/sound/soc/codecs/rt1308-sdw.c index 91cc5a15c211..b0ba0d2acbdd 100644 --- a/sound/soc/codecs/rt1308-sdw.c +++ b/sound/soc/codecs/rt1308-sdw.c @@ -178,10 +178,6 @@ static int rt1308_io_init(struct device *dev, struct sdw_slave *slave) if (rt1308->hw_init) return 0; - ret = rt1308_read_prop(slave); - if (ret < 0) - goto _io_init_err_; - if (rt1308->first_hw_init) { regcache_cache_only(rt1308->regmap, false); regcache_cache_bypass(rt1308->regmap, true); @@ -282,7 +278,6 @@ static int rt1308_io_init(struct device *dev, struct sdw_slave *slave) dev_dbg(&slave->dev, "%s hw_init complete\n", __func__); -_io_init_err_: return ret; } |