hwspinlock: omap: Emit only one error message for errors in .remove()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tue, 14 Mar 2023 18:00:59 +0000 (19:00 +0100)
committerBjorn Andersson <andersson@kernel.org>
Sat, 15 Jul 2023 20:04:00 +0000 (13:04 -0700)
If a remove callback of a platform driver returns a non-zero value, the
driver core emits an error message, otherwise ignores the value and
completes unbinding the device.

As omap_hwspinlock_remove() already emits an error message, suppress the
core's error message by returning zero.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Baolin Wang <baolin.wang@linux.alibaba.com>
Link: https://lore.kernel.org/r/20230314180100.2865801-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/hwspinlock/omap_hwspinlock.c

index dfe8295..a2538c6 100644 (file)
@@ -153,7 +153,7 @@ static int omap_hwspinlock_remove(struct platform_device *pdev)
        ret = hwspin_lock_unregister(bank);
        if (ret) {
                dev_err(&pdev->dev, "%s failed: %d\n", __func__, ret);
-               return ret;
+               return 0;
        }
 
        pm_runtime_disable(&pdev->dev);