On Thu, Nov 10, 2011 at 12:21:13AM -0800, Kuninori Morimoto wrote: > + switch (level) { > + case SND_SOC_BIAS_OFF: > + snd_soc_write(codec, PW_MGMT1, 0x00); > + break; > + default: > + snd_soc_update_bits(codec, PW_MGMT1, PMVCM, PMVCM); > + } Should have a break; here for coding style.