On Tue, Apr 03, 2012 at 10:52:26AM +0530, Sourav Poddar wrote: > +static int kbd_write_irqstatus(struct omap4_keypad *keypad_data, > + u32 offset, u32 value) > +{ > + return __raw_writel(value, keypad_data->base + offset); > +} > + > +static int kbd_write_irqenable(struct omap4_keypad *keypad_data, > + u32 offset, u32 value) > +{ > + return __raw_writel(value, keypad_data->base + offset); > +} if this only writes to irqenable, why do you pass offset as an argument? likewise for irqstatus. -- balbi
Attachment:
signature.asc
Description: Digital signature