From: Imre Deak <imre.deak@intel.com>
To: <intel-gfx@lists.freedesktop.org>, <intel-xe@lists.freedesktop.org>
Cc: Imre Deak <imre.deak@gmail.com>
Subject: [PATCH v2 18/20] drm/i915/dp_mst: Reuse intel_dp_handle_link_service_irq()
Date: Wed, 25 Feb 2026 18:27:49 +0200 [thread overview]
Message-ID: <20260225162751.1255913-19-imre.deak@intel.com> (raw)
In-Reply-To: <20260225162751.1255913-1-imre.deak@intel.com>
From: Imre Deak <imre.deak@gmail.com>
Use intel_dp_handle_link_service_irq() while handling an MST HPD IRQ,
instead of open-coding this.
Signed-off-by: Imre Deak <imre.deak@gmail.com>
---
drivers/gpu/drm/i915/display/intel_dp.c | 13 ++++---------
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 826d3d63c43ff..27eea2cc0c301 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -5508,6 +5508,8 @@ intel_dp_mst_hpd_irq(struct intel_dp *intel_dp, u8 *esi, u8 *ack)
}
}
+static bool intel_dp_handle_link_service_irq(struct intel_dp *intel_dp, u8 irq_mask);
+
/**
* intel_dp_check_mst_status - service any pending MST interrupts, check link status
* @intel_dp: Intel DP struct
@@ -5555,14 +5557,7 @@ intel_dp_check_mst_status(struct intel_dp *intel_dp)
if (ack[1] & (DP_DOWN_REP_MSG_RDY | DP_UP_REQ_MSG_RDY))
drm_dp_mst_hpd_irq_send_new_request(&intel_dp->mst.mgr);
- if (ack[3] & RX_CAP_CHANGED)
- reprobe_needed = true;
-
- if ((ack[3] & LINK_STATUS_CHANGED) || intel_dp->link.force_retrain)
- intel_dp_check_link_state(intel_dp);
-
- if ((ack[3] & DP_TUNNELING_IRQ) &&
- drm_dp_tunnel_handle_irq(display->dp_tunnel_mgr, &intel_dp->aux))
+ if (intel_dp_handle_link_service_irq(intel_dp, ack[3]))
reprobe_needed = true;
}
@@ -5860,7 +5855,7 @@ static bool intel_dp_handle_link_service_irq(struct intel_dp *intel_dp, u8 irq_m
if (irq_mask & RX_CAP_CHANGED)
reprobe_needed = true;
- if (irq_mask & LINK_STATUS_CHANGED)
+ if ((irq_mask & LINK_STATUS_CHANGED) || intel_dp->link.force_retrain)
intel_dp_check_link_state(intel_dp);
if (irq_mask & HDMI_LINK_STATUS_CHANGED)
--
2.49.1
next prev parent reply other threads:[~2026-02-25 16:29 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-25 16:27 [PATCH v2 00/20] drm/i915/dp: Fix few SST HPD IRQ handling issues Imre Deak
2026-02-25 16:27 ` [PATCH v2 01/20] drm/i915/dp_mst: Reprobe connector if the IRQ ESI read failed Imre Deak
2026-02-25 16:27 ` [PATCH v2 02/20] drm/i915/dp_mst: Verify the link status always the same way Imre Deak
2026-02-25 16:27 ` [PATCH v2 03/20] drm/i915/dp_mst: Reuse intel_dp_check_link_state() in the HPD IRQ handler Imre Deak
2026-02-25 16:27 ` [PATCH v2 04/20] drm/i915/dp: Handle a tunneling IRQ after acking it Imre Deak
2026-02-25 16:27 ` [PATCH v2 05/20] drm/i915/dp: Handle the RX_CAP_CHANGED HPD IRQ Imre Deak
2026-02-25 16:27 ` [PATCH v2 06/20] drm/i915/dp: Handle the DOWNSTREAM_PORT_STATUS_CHANGED event Imre Deak
2026-02-25 16:27 ` [PATCH v2 07/20] drm/i915/dp: Don't clobber the encoder state in the HPD IRQ handler Imre Deak
2026-02-25 16:27 ` [PATCH v2 08/20] drm/i915/dp: Remove the device service IRQ handling from connector detect Imre Deak
2026-02-25 16:27 ` [PATCH v2 09/20] drm/i915/dp: Fix the device service IRQ DPCD_REV check Imre Deak
2026-02-25 16:27 ` [PATCH v2 10/20] drm/i915/dp: Fix the link " Imre Deak
2026-02-25 16:27 ` [PATCH v2 11/20] drm/i915/dp: Reprobe connector if getting/acking device IRQs fails Imre Deak
2026-02-25 16:27 ` [PATCH v2 12/20] drm/i915/dp: Reprobe connector if getting/acking link service " Imre Deak
2026-02-25 16:27 ` [PATCH v2 13/20] drm/i915/dp: Return early if getting/acking device " Imre Deak
2026-02-25 16:27 ` [PATCH v2 14/20] drm/i915/dp: Return early if getting/acking link " Imre Deak
2026-02-25 16:27 ` [PATCH v2 15/20] drm/i915/dp: Read/ack sink count and sink IRQs for SST as it's done for MST Imre Deak
2026-02-25 16:27 ` [PATCH v2 16/20] drm/i915/dp: Print debug message for a sink connected off request Imre Deak
2026-02-25 16:27 ` [PATCH v2 17/20] drm/i915/dp: Check SST link status while handling link service IRQs Imre Deak
2026-02-25 16:27 ` Imre Deak [this message]
2026-02-25 16:27 ` [PATCH v2 19/20] drm/i915/dp: Ack only the handled device " Imre Deak
2026-02-25 16:27 ` [PATCH v2 20/20] drm/i915/dp: Ack only the handled link " Imre Deak
2026-02-25 16:34 ` [PATCH v2 00/20] drm/i915/dp: Fix few SST HPD IRQ handling issues Imre Deak
2026-02-25 16:52 ` ✓ CI.KUnit: success for drm/i915/dp: Fix few SST HPD IRQ handling issues (rev2) Patchwork
2026-02-25 17:26 ` ✓ Xe.CI.BAT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260225162751.1255913-19-imre.deak@intel.com \
--to=imre.deak@intel.com \
--cc=imre.deak@gmail.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox