On 12/12/2017 06:02 AM, Mark Brown wrote: > On Mon, Dec 11, 2017 at 01:01:57PM -0600, Andrew F. Davis wrote: > >> + /* Configure TDM slot width. This is only applicable to TAS5722. */ >> + if (tas5720->devtype == TAS5722) { >> + ret = snd_soc_update_bits(codec, TAS5722_DIGITAL_CTRL2_REG, >> + TAS5722_TDM_SLOT_16B, >> + slot_width == 16 ? >> + TAS5722_TDM_SLOT_16B : 0); >> + if (ret < 0) >> + goto error_snd_soc_update_bits; >> + } > > Use a switch statement, that way additional variants can be handled more > sensibly. > Will fix. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel