On Wed, Feb 17, 2016 at 05:28:49PM -0800, anish kumar wrote: > + if (pclk_bclk_ratio == 2) > + bclk_value = MAX9867_IFC1B_PCLK_2; > + else if (pclk_bclk_ratio == 4) > + bclk_value = MAX9867_IFC1B_PCLK_4; > + else if (pclk_bclk_ratio == 8) > + bclk_value = MAX9867_IFC1B_PCLK_8; > + else if (pclk_bclk_ratio == 16) > + bclk_value = MAX9867_IFC1B_PCLK_16; > + else { > + dev_err(codec->dev, "unsupported sampling rate\n"); > + return -EINVAL; > + } > + break; This looks like a switch statement, please write it as a switch statement. Please send a followup patch implementing this.
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel