On 2021-12-17 12:11 AM, Pierre-Louis Bossart wrote:
snd_hdac_ext_stop_streams() has really nothing to do with the extension, it just loops over the bus streams.
Move it to the hdac_stream layer and rename to remove the 'ext' prefix and add the precision that the chip will also be stopped.
Signed-off-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com Reviewed-by: Kai Vehmanen kai.vehmanen@intel.com Reviewed-by: Péter Ujfalusi peter.ujfalusi@linux.intel.com Reviewed-by: Ranjani Sridharan ranjani.sridharan@linux.intel.com
Change looks good. I wonder though, if there is a better 'name' for the process (here, stop streams + stop chip as a tear down process) than 'stop_streams_and_chip'.
Reviewed-by: Cezary Rojewski cezary.rojewski@intel.com