Quoting Dmitry Baryshkov (2022-09-01 02:15:26) > There is little point in using irq_of_parse_and_map(). Switch to plain > and usual platform_get_irq() for parsing the DP IRQ line. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > drivers/gpu/drm/msm/dp/dp_display.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c > index 3173e6962a78..40c7f91abec9 100644 > --- a/drivers/gpu/drm/msm/dp/dp_display.c > +++ b/drivers/gpu/drm/msm/dp/dp_display.c > @@ -1243,8 +1243,8 @@ int dp_display_request_irq(struct msm_dp *dp_display) > > dp = container_of(dp_display, struct dp_display_private, dp_display); > > - dp->irq = irq_of_parse_and_map(dp->pdev->dev.of_node, 0); > - if (!dp->irq) { > + dp->irq = platform_get_irq(dp->pdev, 0); > + if (dp->irq < 0) { > DRM_ERROR("failed to get irq\n"); Can drop the error as well because platform_get_irq() prints an error on failure.