summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCristian Ciocaltea <cristian.ciocaltea@collabora.com>2026-01-15 19:24:44 +0300
committerHeiko Stuebner <heiko@sntech.de>2026-01-20 13:09:35 +0300
commitaa156ad92523b21b12bfa5bb261d6affb4b1597f (patch)
tree191f12b10bea816395018c66aacb9cf5ff5621af
parent5f7be8afc40c5ccf1be0410514703e50a49532c0 (diff)
downloadlinux-aa156ad92523b21b12bfa5bb261d6affb4b1597f.tar.xz
drm/rockchip: dw_hdmi_qp: Optimize RK3588 HPD interrupt handling
The threaded interrupt handler on RK3588 checks HPD IRQ status before deciding to continue with interrupt clearing and unmasking. However, this is not really necessary, since the hard interrupt handler already performs the very same verification before waking the handler thread. Get rid of the redundant verification of the HPD interrupt status in the threaded interrupt handler. Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@collabora.com> Signed-off-by: Heiko Stuebner <heiko@sntech.de> Link: https://patch.msgid.link/20260115-dw-hdmi-qp-hpd-v1-2-e59c166eaa65@collabora.com
-rw-r--r--drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
index 831d221bfd0d..a8190a6e8978 100644
--- a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
+++ b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
@@ -315,11 +315,7 @@ static irqreturn_t dw_hdmi_qp_rk3588_hardirq(int irq, void *dev_id)
static irqreturn_t dw_hdmi_qp_rk3588_irq(int irq, void *dev_id)
{
struct rockchip_hdmi_qp *hdmi = dev_id;
- u32 intr_stat, val;
-
- regmap_read(hdmi->regmap, RK3588_GRF_SOC_STATUS1, &intr_stat);
- if (!intr_stat)
- return IRQ_NONE;
+ u32 val;
if (hdmi->port_id)
val = FIELD_PREP_WM16(RK3588_HDMI1_HPD_INT_CLR, 1);