Re: [alsa-devel] [PATCH v5] ASoC: max98090: remove 24-bit format support if RJ is 0

On Mon, Jun 17, 2019 at 11:55:26AM +0800, Yu-Hsuan Hsu wrote:

• /*

• * When TDM = 0, remove 24-bit format support if it is not in right
  

• * justified mode.
  

• */
  

• if (!max98090->tdm_slots &&

• (fmt & SND_SOC_DAIFMT_FORMAT_MASK) != SND_SOC_DAIFMT_RIGHT_J) {
  

• substream->runtime->hw.formats = SNDRV_PCM_FMTBIT_S16_LE;
  

• snd_pcm_hw_constraint_msbits(substream->runtime, 0, 16, 16);
  

• }

Do you need both the addition of constraints and the one way modification of the formats here? Also the indentation is messed up which makes things hard to read, the second line of the conditional is aligned with the contents of the if block.