ASoC: fsl_ssi: Remove FSLSSI_I2S_RATES definition (58055677) · Commits · 戴 / test

The comment for the FSLSSI_I2S_RATES definition states that the driver currently only supports I2S slave mode, which is no longer correct. As FSLSSI_I2S_RATES is the same as the standard SNDRV_PCM_RATE_CONTINUOUS, just remove its definition and its comments to make the code simpler. Signed-off-by:

Fabio Estevam <fabio.estevam@nxp.com> Acked-by:

Timur Tabi <timur@tabi.org> Signed-off-by:

Mark Brown <broonie@kernel.org>

sound/soc/fsl/fsl_ssi.c

+2 −12

Original line number	Diff line number	Diff line
		@@ -54,16 +54,6 @@
		#include "fsl_ssi.h"
		#include "imx-pcm.h"

		/**
		* FSLSSI_I2S_RATES: sample rates supported by the I2S
		*
		* This driver currently only supports the SSI running in I2S slave mode,
		* which means the codec determines the sample rate. Therefore, we tell
		* ALSA that we support all rates and let the codec driver decide what rates
		* are really supported.
		*/
		#define FSLSSI_I2S_RATES SNDRV_PCM_RATE_CONTINUOUS

		/**
		* FSLSSI_I2S_FORMATS: audio formats supported by the SSI
		*
		@@ -1217,14 +1207,14 @@ static struct snd_soc_dai_driver fsl_ssi_dai_template = {
		.stream_name = "CPU-Playback",
		.channels_min = 1,
		.channels_max = 32,
		.rates = FSLSSI_I2S_RATES,
		.rates = SNDRV_PCM_RATE_CONTINUOUS,
		.formats = FSLSSI_I2S_FORMATS,
		},
		.capture = {
		.stream_name = "CPU-Capture",
		.channels_min = 1,
		.channels_max = 32,
		.rates = FSLSSI_I2S_RATES,
		.rates = SNDRV_PCM_RATE_CONTINUOUS,
		.formats = FSLSSI_I2S_FORMATS,
		},
		.ops = &fsl_ssi_dai_ops,

Admin message