On Tue, Jul 24, 2012 at 13:56:24, Thierry Reding wrote: > On Tue, Jul 24, 2012 at 10:25:17AM +0200, Lars-Peter Clausen wrote: > > On 07/24/2012 08:51 AM, Thierry Reding wrote: > > > > > > How about the following? > > > > > > /** > > > * enum pwm_polarity - polarity of a PWM signal > > > * @PWM_POLARITY_NORMAL: a high signal for the duration of the duty- > > > * cycle, followed by a low signal for the remainder of the pulse > > > * period > > > * @PWM_POLARITY_INVERSED: a low signal for the duration of the duty- > > > * cycle, followed by a high signal for the remainder of the pulse > > > * period > > > */ > > > enum pwm_polarity { > > > PWM_POLARITY_NORMAL, > > > PWM_POLARITY_INVERSED, > > > }; > > > > > > > Looks fine :) > > Philip: can you fold this into your patch? Sure. I will add. Thanks Avinash > > Thierry > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html