Input: iqs62x-keys - suppress duplicated error message in .remove()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Mon, 20 Mar 2023 04:23:25 +0000 (21:23 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 20 Mar 2023 04:34:24 +0000 (21:34 -0700)
If a platform driver's remove callback returns non-zero the driver core
emits an error message. In such a case however iqs62x_keys_remove()
already issued a (better) message. So return zero to suppress the
generic message.

This patch has no other side effects as platform_remove() ignores the
return value of .remove() after the warning.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20230318225110.261439-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/keyboard/iqs62x-keys.c

index db793a550c258c35f8795b53e87f1589dd46a765..02ceebad7bdaaddc3e24436d4f0ac97ae06ceb6e 100644 (file)
@@ -320,7 +320,7 @@ static int iqs62x_keys_remove(struct platform_device *pdev)
        if (ret)
                dev_err(&pdev->dev, "Failed to unregister notifier: %d\n", ret);
 
-       return ret;
+       return 0;
 }
 
 static struct platform_driver iqs62x_keys_platform_driver = {