On Saturday, January 04, 2014 6:02 PM, Dmitry Torokhov wrote: > > Signed-off-by: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> > --- > > Compile-tested only. > > > drivers/input/keyboard/twl4030_keypad.c | 70 +++++++++++---------------------- > 1 file changed, 22 insertions(+), 48 deletions(-) > > diff --git a/drivers/input/keyboard/twl4030_keypad.c b/drivers/input/keyboard/twl4030_keypad.c > index f663c19..71eb041 100644 > --- a/drivers/input/keyboard/twl4030_keypad.c > +++ b/drivers/input/keyboard/twl4030_keypad.c [.....] > -static int twl4030_kp_remove(struct platform_device *pdev) > -{ > - struct twl4030_keypad *kp = platform_get_drvdata(pdev); > - > - free_irq(kp->irq, kp); > - input_unregister_device(kp->input); Hi Dmitry Torokhov, IMHO, input_unregister_device() seems to be necessary, because input_register_device() is still used. If I am wrong, please let me know kindly. Sebastian Reichel, Would you repeat insmod & rmmod 'twl4030_keypad.ko'? Happy New Year! :-) Best regards, Jingoo Han -- 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