| Message ID | 20231120151121.60942-1-jiajie.ho@starfivetech.com |
|---|---|
| State | Accepted |
| Commit | 2d37b3649c412b3bcecfea932cb677f7a5775b15 |
| Headers | show |
| Series | hwrng: starfive - Fix dev_err_probe return error | expand |
On Mon, Nov 20, 2023 at 11:11:21PM +0800, Jia Jie Ho wrote: > Current dev_err_probe will return 0 instead of proper error code if > driver failed to get irq number. Fix the return err code. > > Signed-off-by: Jia Jie Ho <jiajie.ho@starfivetech.com> > Reported-by: kernel test robot <lkp@intel.com> > Reported-by: Dan Carpenter <error27@gmail.com> > Closes: https://lore.kernel.org/r/202311160649.3GhKCfhd-lkp@intel.com/ > --- > drivers/char/hw_random/jh7110-trng.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Patch applied. Thanks.
diff --git a/drivers/char/hw_random/jh7110-trng.c b/drivers/char/hw_random/jh7110-trng.c index 38474d48a25e..b1f94e3c0c6a 100644 --- a/drivers/char/hw_random/jh7110-trng.c +++ b/drivers/char/hw_random/jh7110-trng.c @@ -300,7 +300,7 @@ static int starfive_trng_probe(struct platform_device *pdev) ret = devm_request_irq(&pdev->dev, irq, starfive_trng_irq, 0, pdev->name, (void *)trng); if (ret) - return dev_err_probe(&pdev->dev, irq, + return dev_err_probe(&pdev->dev, ret, "Failed to register interrupt handler\n"); trng->hclk = devm_clk_get(&pdev->dev, "hclk");
Current dev_err_probe will return 0 instead of proper error code if driver failed to get irq number. Fix the return err code. Signed-off-by: Jia Jie Ho <jiajie.ho@starfivetech.com> Reported-by: kernel test robot <lkp@intel.com> Reported-by: Dan Carpenter <error27@gmail.com> Closes: https://lore.kernel.org/r/202311160649.3GhKCfhd-lkp@intel.com/ --- drivers/char/hw_random/jh7110-trng.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)