On Mon, Aug 20, 2012 at 04:02:05AM +0000, Kim, Milo wrote: [...] > diff --git a/drivers/leds/leds-lm3530.c b/drivers/leds/leds-lm3530.c [...] > @@ -111,6 +114,8 @@ struct lm3530_data { > struct regulator *regulator; > enum led_brightness brightness; > bool enable; > + struct pwm_device *pwm; > + int period_ns; I forgot: maybe this should be an "unsigned int"? Thierry