On Thu, 2025-06-26 at 11:20 +0300, Imre Deak wrote:
> From: Imre Deak <imre.d...@gmail.com>
>
> So far the CONNECTED_OFF_ENTRY_REQUESTED request was accepted only
> implicitly, by acking all the IRQs raised by the sink. Make this
> explicit by printing a debug message.
>
> Signed-off-by: Imre Deak <imre.d...@gmail.com>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 2e6ed7d2a64a6..90f6fe20e4c0d 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -5474,6 +5474,8 @@ static void intel_dp_handle_device_service_irq(struct
> intel_dp *intel_dp, u8 irq
> static bool intel_dp_handle_link_service_irq(struct intel_dp *intel_dp, u8
> irq_mask)
> {
> struct intel_display *display = to_intel_display(intel_dp);
> + struct intel_connector *connector = intel_dp->attached_connector;
> + struct intel_encoder *encoder = &dp_to_dig_port(intel_dp)->base;
> bool reprobe_needed = false;
>
> if (irq_mask & RX_CAP_CHANGED)
> @@ -5482,6 +5484,12 @@ static bool intel_dp_handle_link_service_irq(struct
> intel_dp *intel_dp, u8 irq_m
> if (irq_mask & HDMI_LINK_STATUS_CHANGED)
> intel_dp_handle_hdmi_link_status_change(intel_dp);
>
> + if (irq_mask & CONNECTED_OFF_ENTRY_REQUESTED)
> + drm_dbg_kms(display->drm,
> + "[CONNECTOR:%d:%s][ENCODER:%d:%s] Allowing
> connected off request\n",
> + connector->base.base.id, connector->base.name,
> + encoder->base.base.id, encoder->base.name);
> +
> if ((irq_mask & DP_TUNNELING_IRQ) &&
> drm_dp_tunnel_handle_irq(display->dp_tunnel_mgr,
> &intel_dp->aux))
Reviewed-by: Luca Coelho <luciano.coe...@intel.com>
--
Cheers,
Luca.
