> -----Original Message-----
> From: Intel-gfx <[email protected]> On Behalf Of Imre 
> Deak
> Sent: Tuesday, 16 June 2026 23.09
> To: [email protected]; [email protected]
> Subject: [PATCH v2 16/28] drm/i915/dp_link_caps: Add helper to get the number 
> of supported link rates
> 
> Add intel_dp_link_caps_num_common_rates() to return the number of
> supported link rates tracked by the link_caps module. This prepares for
> tracking these capabilities internally within the link caps module.
> 

Reviewed-by: Mika Kahola <[email protected]>

> Signed-off-by: Imre Deak <[email protected]>
> ---
>  drivers/gpu/drm/i915/display/intel_dp.c           | 4 ++--
>  drivers/gpu/drm/i915/display/intel_dp_link_caps.c | 5 +++++
>  drivers/gpu/drm/i915/display/intel_dp_link_caps.h | 1 +
>  3 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c 
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index f9c3d3561c417..84640c8394534 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -1766,7 +1766,7 @@ intel_dp_compute_link_config_wide(struct intel_dp 
> *intel_dp,
>               int link_bpp_x16 =
>                       
> intel_dp_output_format_link_bpp_x16(pipe_config->output_format, bpp);
> 
> -             for (i = 0; i < intel_dp->num_common_rates; i++) {
> +             for (i = 0; i < 
> intel_dp_link_caps_num_common_rates(intel_dp->link.caps); i++) {
>                       link_rate = intel_dp_common_rate(intel_dp, i);
>                       if (link_rate < limits->min_rate ||
>                           link_rate > limits->max_rate)
> @@ -1995,7 +1995,7 @@ static int dsc_compute_link_config(struct intel_dp 
> *intel_dp,
>       int link_rate, lane_count;
>       int i;
> 
> -     for (i = 0; i < intel_dp->num_common_rates; i++) {
> +     for (i = 0; i < 
> intel_dp_link_caps_num_common_rates(intel_dp->link.caps); i++) {
>               link_rate = intel_dp_common_rate(intel_dp, i);
>               if (link_rate < limits->min_rate || link_rate > 
> limits->max_rate)
>                       continue;
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_caps.c 
> b/drivers/gpu/drm/i915/display/intel_dp_link_caps.c
> index 13f9bfd5d7bad..09b60a0cd6fbb 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_link_caps.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_link_caps.c
> @@ -67,6 +67,11 @@ int intel_dp_max_common_rate(struct intel_dp *intel_dp)
>       return intel_dp_common_rate(intel_dp, intel_dp->num_common_rates - 1);
>  }
> 
> +int intel_dp_link_caps_num_common_rates(struct intel_dp_link_caps *link_caps)
> +{
> +     return link_caps->dp->num_common_rates;
> +}
> +
>  void intel_dp_link_caps_print_common_rates(struct intel_dp_link_caps 
> *link_caps)
>  {
>       struct intel_dp *intel_dp = link_caps->dp;
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_caps.h 
> b/drivers/gpu/drm/i915/display/intel_dp_link_caps.h
> index 7333df6b82f97..3413f6f760453 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_link_caps.h
> +++ b/drivers/gpu/drm/i915/display/intel_dp_link_caps.h
> @@ -15,6 +15,7 @@ int intel_dp_common_len_rate_limit(const struct intel_dp 
> *intel_dp,
>                                  int max_rate);
>  int intel_dp_common_rate(struct intel_dp *intel_dp, int index);
>  int intel_dp_max_common_rate(struct intel_dp *intel_dp);
> +int intel_dp_link_caps_num_common_rates(struct intel_dp_link_caps 
> *link_caps);
> 
>  void intel_dp_link_caps_print_common_rates(struct intel_dp_link_caps 
> *link_caps);
> 
> --
> 2.49.1

Reply via email to