Naidu, On Fri, Nov 21, 2014 at 12:13 PM, <Naidu.Tellapati@xxxxxxxxx> wrote: > From: Naidu Tellapati <Naidu.Tellapati@xxxxxxxxxx> > > The Pistachio SOC from Imagination Technologies includes a Pulse Width > Modulation DAC which produces 1 to 4 digital bit-outputs which represent > digital waveforms. These PWM outputs are primarily in charge of controlling > backlight LED devices. > > Signed-off-by: Naidu Tellapati <Naidu.Tellapati@xxxxxxxxxx> > Signed-off-by: Sai Masarapu <Sai.Masarapu@xxxxxxxxxx> One tiny comment below, otherwise: Reviewed-by: Andrew Bresticker <abrestic@xxxxxxxxxxxx> > +static int img_pwm_remove(struct platform_device *pdev) > +{ > + unsigned int i; > + unsigned int val; > + No need for this newline. > + struct img_pwm_chip *pwm_chip = platform_get_drvdata(pdev); -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html