hwrng: nomadik - two simple cleanups

[PATCH 0/2] hwrng: nomadik - two simple cleanups

Posted by Martin Kaiser 2 years, 7 months ago

Use devm_clk_get_enabled to fix a (theoretical) race condition during removal.
Call dev_error_probe to print an error message and exit.

Both changes were compile-tested only, I don't have this hardware.

Martin Kaiser (2):
  hwrng: nomadik - keep clock enabled while hwrng is registered
  hwrng: nomadik - use dev_err_probe

 drivers/char/hw_random/nomadik-rng.c | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

-- 
2.30.2

Re: [PATCH 0/2] hwrng: nomadik - two simple cleanups

Posted by Herbert Xu 2 years, 6 months ago

On Sun, Jul 02, 2023 at 07:35:01PM +0200, Martin Kaiser wrote:
> Use devm_clk_get_enabled to fix a (theoretical) race condition during removal.
> Call dev_error_probe to print an error message and exit.
> 
> Both changes were compile-tested only, I don't have this hardware.
> 
> Martin Kaiser (2):
>   hwrng: nomadik - keep clock enabled while hwrng is registered
>   hwrng: nomadik - use dev_err_probe
> 
>  drivers/char/hw_random/nomadik-rng.c | 19 +++++--------------
>  1 file changed, 5 insertions(+), 14 deletions(-)
> 
> -- 
> 2.30.2

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