[PATCH 16/20] drm/i915/dp: Print debug message for a sink connected off request

Luca Coelho luca at coelho.fi
Thu Jul 3 13:03:52 UTC 2025


On Thu, 2025-06-26 at 11:20 +0300, Imre Deak wrote:
> From: Imre Deak <imre.deak at 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.deak at 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.coelho at intel.com>

--
Cheers,
Luca.


More information about the Intel-gfx mailing list