On Tue, Feb 10, 2015 at 05:26:03PM +0530, Sudip Mukherjee wrote: > initialiaze lcd parameters only if lcd is enabled. > > Signed-off-by: Sudip Mukherjee <su...@vectorindia.org>
One minor comment below (cosmetic), but after this it's OK. Acked-by: Willy Tarreau <w...@1wt.eu> > + if (lcd.enabled) { > + /* > + * Init lcd struct with load-time values to preserve exact > + * current functionality (at least for now). > + */ > + lcd.height = lcd_height; > + lcd.width = lcd_width; > + lcd.bwidth = lcd_bwidth; > + lcd.hwidth = lcd_hwidth; > + lcd.charset = lcd_charset; > + lcd.proto = lcd_proto; > + lcd.pins.e = lcd_e_pin; > + lcd.pins.rs = lcd_rs_pin; > + lcd.pins.rw = lcd_rw_pin; > + lcd.pins.cl = lcd_cl_pin; > + lcd.pins.da = lcd_da_pin; > + lcd.pins.bl = lcd_bl_pin; > + > + /* Leave it for now, just in case */ ^^^^^^^^^^^^^^^^^^^ Please fix indenting the comment here. > + lcd.esc_seq.len = -1; > + } > + Willy _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel