ASoC: fsl_sai: Use snd_pcm_format_width()

Use common helper function snd_pcm_format_width() to make code neater.

Signed-off-by: Nicolin Chen <Guangyu.Chen@freescale.com>
Reviewed-by: Xiubo Li <Li.Xiubo@freescale.com>
Signed-off-by: Mark Brown <broonie@linaro.org>

diff --git a/sound/soc/fsl/fsl_sai.c b/sound/soc/fsl/fsl_sai.c
index 69a375f48efe45a953ebfb29d89b99849eb1ef5f..e68102e63521ee52c82be16be6ff8141f282cd35 100644 (file)
--- a/sound/soc/fsl/fsl_sai.c
+++ b/sound/soc/fsl/fsl_sai.c
@@ -244,9 +244,10 @@ static int fsl_sai_hw_params(struct snd_pcm_substream *substream,
                struct snd_pcm_hw_params *params,
                struct snd_soc_dai *cpu_dai)
 {
-       u32 val_cr4, val_cr5, val_mr, reg_cr4, reg_cr5, reg_mr, word_width;
+       u32 val_cr4, val_cr5, val_mr, reg_cr4, reg_cr5, reg_mr;
        unsigned int channels = params_channels(params);
        struct fsl_sai *sai = snd_soc_dai_get_drvdata(cpu_dai);
+       u32 word_width = snd_pcm_format_width(params_format(params));
 
        if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) {
                reg_cr4 = FSL_SAI_TCR4;
@@ -267,20 +268,6 @@ static int fsl_sai_hw_params(struct snd_pcm_substream *substream,
        val_cr5 &= ~FSL_SAI_CR5_W0W_MASK;
        val_cr5 &= ~FSL_SAI_CR5_FBT_MASK;
 
-       switch (params_format(params)) {
-       case SNDRV_PCM_FORMAT_S16_LE:
-               word_width = 16;
-               break;
-       case SNDRV_PCM_FORMAT_S20_3LE:
-               word_width = 20;
-               break;
-       case SNDRV_PCM_FORMAT_S24_LE:
-               word_width = 24;
-               break;
-       default:
-               return -EINVAL;
-       }
-
        val_cr4 |= FSL_SAI_CR4_SYWD(word_width);
        val_cr5 |= FSL_SAI_CR5_WNW(word_width);
        val_cr5 |= FSL_SAI_CR5_W0W(word_width);