media: venus: Warn only once about problems in .remove()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Sun, 26 Mar 2023 14:31:10 +0000 (16:31 +0200)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Tue, 11 Apr 2023 14:59:16 +0000 (16:59 +0200)
The only effect of returning an error code in a remove callback is that
the driver core emits a warning. The device is unbound anyhow.

As the remove callback already emits a (quite verbose) warning when ret
is non-zero, return zero to suppress the additional warning.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/platform/qcom/venus/core.c

index 990a151..403ffb9 100644 (file)
@@ -448,7 +448,7 @@ static int venus_remove(struct platform_device *pdev)
        mutex_destroy(&core->lock);
        venus_dbgfs_deinit(core);
 
-       return ret;
+       return 0;
 }
 
 static void venus_core_shutdown(struct platform_device *pdev)