On Fri, Jan 28, 2011 at 02:26:37PM -0700, Stephen Warren wrote: > + gpio_set_value_cansleep(pdata->gpio_spkr_en, > + !!SND_SOC_DAPM_EVENT_ON(event)); GPIOs should accept C logical values - if you need the !! there's a bug in the GPIO driver.