From: tarun-tarun143 <[email protected]>

Validate the maximum link rate against the source-supported rates
before using the maximum link limits to calculate allowed
configurations.

This prevents an unsupported link rate from being accepted as a
maximum link limit.

Signed-off-by: tarun-tarun143 <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_dp_link_caps.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

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 7b6cc6055da8..0934488153bb 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_link_caps.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_link_caps.c
@@ -710,7 +710,10 @@ static bool max_link_limits_valid(struct 
intel_dp_link_caps *link_caps,
            !is_power_of_2(max_link_limits->lane_count))
                return false;
 
-       /* TODO: Validate max_link_limits->rate against the source supported 
rates. */
+       if (intel_dp_rate_index(link_caps->dp->source_rates,
+                               link_caps->dp->num_source_rates,
+                               max_link_limits->rate) < 0)
+               return false;
 
        intel_dp_link_caps_get_forced_params(link_caps, &forced_params);
        allowed_configs = calc_allowed_config_filter(link_caps, 
link_caps->enabled_configs,

base-commit: 73e3f0710014fe6d4ed98cfc02292f6121db7558
-- 
2.43.0

Reply via email to