> -----Original Message-----
> From: Intel-xe <intel-xe-boun...@lists.freedesktop.org> On Behalf Of Imre Deak
> Sent: Tuesday, 5 August 2025 10.37
> To: intel-gfx@lists.freedesktop.org; intel...@lists.freedesktop.org
> Subject: [PATCH 19/19] drm/i915/tc: Debug print the pin assignment and max 
> lane count
> 
> Debug print the TypeC pin assignment and max lane count value during HW 
> readout and after resetting the TypeC mode.
> 

Reviewed-by: Mika Kahola <mika.kah...@intel.com>

> Signed-off-by: Imre Deak <imre.d...@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_tc.c | 21 +++++++++++++++++----
>  1 file changed, 17 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_tc.c 
> b/drivers/gpu/drm/i915/display/intel_tc.c
> index f6bc253bec559..0330b428eb398 100644
> --- a/drivers/gpu/drm/i915/display/intel_tc.c
> +++ b/drivers/gpu/drm/i915/display/intel_tc.c
> @@ -277,6 +277,14 @@ static u32 get_lane_mask(struct intel_tc_port *tc)
>       return lane_mask >> DP_LANE_ASSIGNMENT_SHIFT(tc->phy_fia_idx);
>  }
> 
> +static char pin_assignment_name(enum intel_tc_pin_assignment
> +pin_assignment) {
> +     if (pin_assignment == INTEL_TC_PIN_ASSIGNMENT_NONE)
> +             return '-';
> +
> +     return 'A' + pin_assignment - INTEL_TC_PIN_ASSIGNMENT_A; }
> +
>  static enum intel_tc_pin_assignment
>  get_pin_assignment(struct intel_tc_port *tc)  { @@ -1506,10 +1514,13 @@ 
> static void intel_tc_port_reset_mode(struct
> intel_tc_port *tc,
>       if (!force_disconnect)
>               tc_phy_connect(tc, required_lanes);
> 
> -     drm_dbg_kms(display->drm, "Port %s: TC port mode reset (%s -> %s)\n",
> +     drm_dbg_kms(display->drm,
> +                 "Port %s: TC port mode reset (%s -> %s) pin assignment: %c 
> max
> +lanes: %d\n",
>                   tc->port_name,
>                   tc_port_mode_name(old_tc_mode),
> -                 tc_port_mode_name(tc->mode));
> +                 tc_port_mode_name(tc->mode),
> +                 pin_assignment_name(tc->pin_assignment),
> +                 tc->max_lane_count);
>  }
> 
>  static bool intel_tc_port_needs_reset(struct intel_tc_port *tc) @@ -1664,9 
> +1675,11 @@ void
> intel_tc_port_sanitize_mode(struct intel_digital_port *dig_port,
>               __intel_tc_port_put_link(tc);
>       }
> 
> -     drm_dbg_kms(display->drm, "Port %s: sanitize mode (%s)\n",
> +     drm_dbg_kms(display->drm, "Port %s: sanitize mode (%s) pin assignment:
> +%c max lanes: %d\n",
>                   tc->port_name,
> -                 tc_port_mode_name(tc->mode));
> +                 tc_port_mode_name(tc->mode),
> +                 pin_assignment_name(tc->pin_assignment),
> +                 tc->max_lane_count);
> 
>       mutex_unlock(&tc->lock);
>  }
> --
> 2.49.1

Reply via email to