On Thu, Sep 01, 2022 at 12:15:24PM +0300, Dmitry Baryshkov wrote:
> Johan Hovold has reported that returning a probe deferral from the
> msm_dp_modeset_init() can cause issues because the IRQ is not freed
> properly. This (compile-tested only) series tries to fix the issue by
> moving devm_request_irq() to the probe callback.
For reference, here's an analysis of the underlying problem and a series
of fixes that addresses this:
https://lore.kernel.org/all/[email protected]/
Note that moving the irq request to probe also fixes the immediate
issue, but that can now be done as a follow-on cleanup (optimisation)
instead.
> Dmitry Baryshkov (3):
> drm/msm/dp: fold disable_irq into devm_request_irq
> drm/msm/dp: switch to using platform_get_irq()
> drm/msm/dp: move dp_request_irq() call to dp_display_probe()
>
> drivers/gpu/drm/msm/dp/dp_display.c | 19 +++++++++----------
> 1 file changed, 9 insertions(+), 10 deletions(-)
Johan
