Re: [PATCH 01/13] hwrng: cctrng: Discard pm_runtime_put_autosuspend() return value

From: Herbert Xu

Date: Fri Nov 06 2026 - 06:29:13 EST


On Tue, Oct 06, 2026 at 04:13:29PM -0700, Brian Norris wrote:
> Printing an error message on pm_runtime_put_autosuspend() returning a
> non-zero value is not particularly useful.
>
> Returning an error code from pm_runtime_put_autosuspend() merely means
> that it has not queued up a timer or work item to check whether or not
> the device can be suspended and there are many perfectly valid
> situations in which that can happen, like after writing "on" to the
> device's runtime PM "control" attribute in sysfs for one example.
>
> Accordingly, update cc_trng_pm_put_suspend() to simply discard the
> return value of pm_runtime_put_autosuspend().
>
> This will facilitate a planned change of the
> pm_runtime_put_autosuspend() return type to void in the future, similar
> to commit 3afd8df02433 ("PM: runtime: Change pm_runtime_put() return
> type to void").
>
> Signed-off-by: Brian Norris <briannorris@xxxxxxxxxxxx>
> ---
> This patch is independent of the rest of the series, except for the end
> (changing the return type). I expect it can be applied by individual
> maintainers, and we pick up the end once the dust is settled.
>
> drivers/char/hw_random/cctrng.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)

Patch applied. Thanks.
--
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt