diff options
author | Takashi Iwai <tiwai@suse.de> | 2018-07-25 23:17:19 +0200 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2018-07-26 17:09:06 +0100 |
commit | b5453e8ca311fdb6003c6583ad101d2b9131b994 (patch) | |
tree | 3a48b22ee89e5ee3bc7870ca4a89decd6e5d7ff6 /sound/soc/intel/boards/kbl_rt5663_max98927.c | |
parent | 533a9274850b041b32fbe6d1df58a5c5b0b9e652 (diff) | |
download | linux-b5453e8ca311fdb6003c6583ad101d2b9131b994.tar.gz linux-b5453e8ca311fdb6003c6583ad101d2b9131b994.tar.bz2 linux-b5453e8ca311fdb6003c6583ad101d2b9131b994.zip |
ASoC: intel: Fix snd_pcm_format_t handling
As sparse warns, the PCM format type can't be dealt as integer as
found in Intel SST driver codes.
Fix them in the following two ways:
- The open code with snd_mask_set() and params->masks reference is
replaced with params_set_format()
- The rest codes with snd_mask_set(fmt, SNDRV_PCM_FORMAT_XXX) are
replaced with the new helper, snd_mask_set_format().
Reported-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/intel/boards/kbl_rt5663_max98927.c')
-rw-r--r-- | sound/soc/intel/boards/kbl_rt5663_max98927.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/soc/intel/boards/kbl_rt5663_max98927.c b/sound/soc/intel/boards/kbl_rt5663_max98927.c index 3a61252fe450..21a6490746a6 100644 --- a/sound/soc/intel/boards/kbl_rt5663_max98927.c +++ b/sound/soc/intel/boards/kbl_rt5663_max98927.c @@ -434,14 +434,14 @@ static int kabylake_ssp_fixup(struct snd_soc_pcm_runtime *rtd, rate->min = rate->max = 48000; channels->min = channels->max = 2; snd_mask_none(fmt); - snd_mask_set(fmt, SNDRV_PCM_FORMAT_S24_LE); + snd_mask_set_format(fmt, SNDRV_PCM_FORMAT_S24_LE); } /* * The speaker on the SSP0 supports S16_LE and not S24_LE. * thus changing the mask here */ if (!strcmp(be_dai_link->name, "SSP0-Codec")) - snd_mask_set(fmt, SNDRV_PCM_FORMAT_S16_LE); + snd_mask_set_format(fmt, SNDRV_PCM_FORMAT_S16_LE); return 0; } |