diff options
author | 2019-08-05 19:55:13 -0500 | |
---|---|---|
committer | 2019-08-21 14:36:01 +0530 | |
commit | e0279b6b5522f86680a9e16a12dbb2e40b1e2698 (patch) | |
tree | 2cde080fab9e72e307eec88c1ee39f506493f410 | |
parent | soundwire: include mod_devicetable.h to avoid compiling warnings (diff) | |
download | linux-e0279b6b5522f86680a9e16a12dbb2e40b1e2698.tar.gz linux-e0279b6b5522f86680a9e16a12dbb2e40b1e2698.tar.bz2 linux-e0279b6b5522f86680a9e16a12dbb2e40b1e2698.zip |
soundwire: stream: fix disable sequence
When we disable the stream and then call hw_free, two bank switches
will be handled and as a result we re-enable the stream on hw_free.
Make sure the stream is disabled on both banks.
TODO: we need to completely revisit all this and make sure we have a
mirroring mechanism between current and alternate banks.
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20190806005522.22642-9-pierre-louis.bossart@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
-rw-r--r-- | drivers/soundwire/stream.c | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/drivers/soundwire/stream.c b/drivers/soundwire/stream.c index 53f5e790fcd7..75b9ad1fb1a6 100644 --- a/drivers/soundwire/stream.c +++ b/drivers/soundwire/stream.c @@ -1637,7 +1637,24 @@ static int _sdw_disable_stream(struct sdw_stream_runtime *stream) } } - return do_bank_switch(stream); + ret = do_bank_switch(stream); + if (ret < 0) { + dev_err(bus->dev, "Bank switch failed: %d\n", ret); + return ret; + } + + /* make sure alternate bank (previous current) is also disabled */ + list_for_each_entry(m_rt, &stream->master_list, stream_node) { + bus = m_rt->bus; + /* Disable port(s) */ + ret = sdw_enable_disable_ports(m_rt, false); + if (ret < 0) { + dev_err(bus->dev, "Disable port(s) failed: %d\n", ret); + return ret; + } + } + + return 0; } /** |