On Wed, Jan 19, 2011 at 01:50:05PM -0700, Stephen Warren wrote: > + if (SND_SOC_DAPM_EVENT_ON(event)) > + gpio_set_value_cansleep(pdata->gpio_spkr_en, 1); > + else > + gpio_set_value_cansleep(pdata->gpio_spkr_en, 0); Very minor stylistic thing but you could say: gpio_set_value_cansleep(pdata->gpio_spkr_en, SND_SOC_DAPM_EVENT_ON(event)); instead. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html