drm/tilcdc: Fix irq free on unload
authorTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Tue, 19 Sep 2023 07:12:50 +0000 (10:12 +0300)
committerTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Tue, 21 Nov 2023 09:09:33 +0000 (11:09 +0200)
The driver only frees the reserved irq if priv->irq_enabled is set to
true. However, the driver mistakenly sets priv->irq_enabled to false,
instead of true, in tilcdc_irq_install(), and thus the driver never
frees the irq, causing issues on loading the driver a second time.

Fixes: b6366814fa77 ("drm/tilcdc: Convert to Linux IRQ interfaces")
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Aradhya Bhatia <a-bhatia1@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230919-lcdc-v1-1-ba60da7421e1@ideasonboard.com
drivers/gpu/drm/tilcdc/tilcdc_drv.c

index 8ebd7134ee21bdc2a574ac12015a4da1ee7b208d..2f6eaac7f659b4e4369e4e9d9dac5d7194b7a4f0 100644 (file)
@@ -138,7 +138,7 @@ static int tilcdc_irq_install(struct drm_device *dev, unsigned int irq)
        if (ret)
                return ret;
 
-       priv->irq_enabled = false;
+       priv->irq_enabled = true;
 
        return 0;
 }