Unverified Commit 9e116f5a authored by tangmeng's avatar tangmeng Committed by Mark Brown
Browse files

ASoC: SOF: Remove redundant return statements 



After the free PCM action is executed, no matter what
the return result is, it will return directly in sof_pcm_trigger.
So the return statement here is redundant.

Signed-off-by: default avatartangmeng <tangmeng@uniontech.com>
Reviewed-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: default avatarDaniel Baluta <daniel.baluta@nxp.com>
Reviewed-by: default avatarPéter Ujfalusi <peter.ujfalusi@linux.intel.com>
Signed-off-by: default avatarRanjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/20220330201926.1330402-12-ranjani.sridharan@linux.intel.com


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

sound/soc/sof/pcm.c

+1 −4
Original line number Diff line number Diff line
@@ -347,12 +347,9 @@ static int sof_pcm_trigger(struct snd_soc_component *component, 
		snd_sof_pcm_platform_trigger(sdev, substream, cmd);



	/* free PCM if reset_hw_params is set and the STOP IPC is successful */

	if (!ret && reset_hw_params) {

	if (!ret && reset_hw_params)

		ret = sof_pcm_stream_free(sdev, substream, spcm, substream->stream,

					  free_widget_list);

		if (ret < 0)

			return ret;

	}



	return ret;

}