On Tuesday 12 June 2012 01:14 AM, Roland Stigge wrote: > + > +struct lpc32xx_kscan_drv { > + struct input_dev *input; > + struct lpc32xx_kscan_cfg *kscancfg; > + struct clk *clk; > + void __iomem *kscan_base; > + int irq; Do you really need this irq field? It is used in the probe and remove only. > + u8 lastkeystates[8]; > + u32 io_p_start; > + u32 io_p_size; > +}; -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html