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.

Attachment: signature.asc
Description: PGP signature

Reply via email to