From: Peng Fan <peng.fan@xxxxxxx> Without remove hook to clear wake irq, there will be kernel dump when doing module test. "bbnsm_pwrkey 44440000.bbnsm:pwrkey: wake irq already initialized" Add remove hook to clear wake irq and set wakeup to false. Fixes: 40e40fdfec3f ("Input: bbnsm_pwrkey - add bbnsm power key support") Cc: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> Signed-off-by: Peng Fan <peng.fan@xxxxxxx> --- V2: Per Christophe JAILLET Use remove, not remove_new Drop checking bbnsm pointer in remove drivers/input/misc/nxp-bbnsm-pwrkey.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/drivers/input/misc/nxp-bbnsm-pwrkey.c b/drivers/input/misc/nxp-bbnsm-pwrkey.c index eb4173f9c820..f0bf119309dd 100644 --- a/drivers/input/misc/nxp-bbnsm-pwrkey.c +++ b/drivers/input/misc/nxp-bbnsm-pwrkey.c @@ -187,6 +187,16 @@ static int bbnsm_pwrkey_probe(struct platform_device *pdev) return 0; } +static void bbnsm_pwrkey_remove(struct platform_device *pdev) +{ + struct bbnsm_pwrkey *bbnsm = platform_get_drvdata(pdev); + + dev_pm_clear_wake_irq(&pdev->dev); + device_init_wakeup(&pdev->dev, false); + + input_unregister_device(bbnsm->input); +} + static int __maybe_unused bbnsm_pwrkey_suspend(struct device *dev) { struct platform_device *pdev = to_platform_device(dev); @@ -223,6 +233,8 @@ static struct platform_driver bbnsm_pwrkey_driver = { .of_match_table = bbnsm_pwrkey_ids, }, .probe = bbnsm_pwrkey_probe, + .remove = bbnsm_pwrkey_remove, + }; module_platform_driver(bbnsm_pwrkey_driver); -- 2.37.1