lkml.org 
[lkml]   [2019]   [Jun]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2019-06-17 16:57    [W:0.027 / U:0.492 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site