Pavel Machek wrote: >> +static void >> +gpio_pwm_work (struct work_struct *work) >> +{ >> + struct gpio_pwm *gp = container_of(work, struct gpio_pwm, work); >> + >> + if (gp->active) >> + gpio_direction_output(gp->gpio, gp->polarity ? 1 : 0); >> + else >> + gpio_direction_output(gp->gpio, gp->polarity ? 0 : 1); >> +} >> > > ...polarity ^ active ? > ... except that if polarity and/or active are >1, I don't send the values 1 or 0 to gpio_direction_output. I don't know if the API is specifically intended to accept nonzero values that are greater than 1. b.g. -- Bill Gatliff Embedded systems training and consulting http://billgatliff.com bgat@xxxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-embedded" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html