Hi,
Am Dienstag, 3. Januar 2023, 11:55:34 CET schrieb Xavier Roumegue (OSS):
> From: Xavier Roumegue <xavier.roumegue@xxxxxxxxxxx>
>
> In case the IRQ allocation returns an error in dw100_probe(), the pm
> runtime is not disabled before to return.
>
> Add the missing unwind goto on the error handling path of the IRQ
> allocation request.
>
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Reported-by: Dan Carpenter <error27@xxxxxxxxx>
> Signed-off-by: Xavier Roumegue <xavier.roumegue@xxxxxxxxxxx>
> ---
> drivers/media/platform/nxp/dw100/dw100.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/nxp/dw100/dw100.c
> b/drivers/media/platform/nxp/dw100/dw100.c index f6d48c36f386..189d60cd5ed1
> 100644
> --- a/drivers/media/platform/nxp/dw100/dw100.c
> +++ b/drivers/media/platform/nxp/dw100/dw100.c
> @@ -1571,7 +1571,7 @@ static int dw100_probe(struct platform_device *pdev)
> dev_name(&pdev->dev), dw_dev);
> if (ret < 0) {
> dev_err(&pdev->dev, "Failed to request irq: %d\n", ret);
> - return ret;
> + goto err_pm;
> }
>
> ret = v4l2_device_register(&pdev->dev, &dw_dev->v4l2_dev);
Doesn't it make more sense to request/allocate the IRQ (and other resources)
before enabling runtime PM?
Best regards,
Alexander
