hwrng: xgene - Improve error reporting for problems during .remove()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tue, 14 Feb 2023 16:28:29 +0000 (17:28 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Tue, 14 Mar 2023 09:06:41 +0000 (17:06 +0800)
Returning an error value in a platform driver's remove callback results in
a generic error message being emitted by the driver core, but otherwise it
doesn't make a difference. The device goes away anyhow.

As the driver already emits a better error message than the core, suppress
the generic error message by returning zero unconditionally.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/char/hw_random/xgene-rng.c

index 3166261..c67d318 100644 (file)
@@ -366,7 +366,7 @@ static int xgene_rng_remove(struct platform_device *pdev)
        if (rc)
                dev_err(&pdev->dev, "RNG init wakeup failed error %d\n", rc);
 
-       return rc;
+       return 0;
 }
 
 static const struct of_device_id xgene_rng_of_match[] = {