Unverified Commit 0abfc84b authored by Cezary Rojewski's avatar Cezary Rojewski Committed by Mark Brown
Browse files

ASoC: Intel: avs: Split pcm pages freeing operation from hw_free() 



Prepare for introduction of PCM power management support. As freeing
pages during the suspend operation is not desired, separate
snd_pcm_lib_free_pages() from existing avs_dai_fe_hw_free() so that
majority of the code found within it can be reused for standard and PM
flows both.

Signed-off-by: default avatarCezary Rojewski <cezary.rojewski@intel.com>
Link: https://lore.kernel.org/r/20221027124702.1761002-2-cezary.rojewski@intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent ce992ff3

sound/soc/intel/avs/pcm.c

+10 −4
Original line number Diff line number Diff line
@@ -499,7 +499,7 @@ static int avs_dai_fe_hw_params(struct snd_pcm_substream *substream, 
	return ret;

}



static int avs_dai_fe_hw_free(struct snd_pcm_substream *substream, struct snd_soc_dai *dai)

static int __avs_dai_fe_hw_free(struct snd_pcm_substream *substream, struct snd_soc_dai *dai)

{

	struct avs_dma_data *data;

	struct hdac_ext_stream *host_stream;
@@ -523,9 +523,15 @@ static int avs_dai_fe_hw_free(struct snd_pcm_substream *substream, struct snd_so 
	snd_hdac_stream_cleanup(hdac_stream(host_stream));

	hdac_stream(host_stream)->prepared = false;



	ret = snd_pcm_lib_free_pages(substream);

	if (ret < 0)

		dev_dbg(dai->dev, "Failed to free pages!\n");

	return ret;

}



static int avs_dai_fe_hw_free(struct snd_pcm_substream *substream, struct snd_soc_dai *dai)

{

	int ret;



	ret = __avs_dai_fe_hw_free(substream, dai);

	snd_pcm_lib_free_pages(substream);



	return ret;

}