Commit af7080f5 authored by Thulasimani,Sivakumar's avatar Thulasimani,Sivakumar Committed by Jani Nikula

drm/i915: fix link rates reported for SKL

This patch fixes the bug that SKL SKUs before B0 might return
HBR2 as supported even though it is not supposed to be enabled
on such platforms.

v2: optimize if else condition (Jani)
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: default avatarSivakumar Thulasimani <sivakumar.thulasimani@intel.com>
[Jani: minor whitespace fix.]
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
parent 6cc38aca
...@@ -1223,21 +1223,24 @@ static bool intel_dp_source_supports_hbr2(struct drm_device *dev) ...@@ -1223,21 +1223,24 @@ static bool intel_dp_source_supports_hbr2(struct drm_device *dev)
static int static int
intel_dp_source_rates(struct drm_device *dev, const int **source_rates) intel_dp_source_rates(struct drm_device *dev, const int **source_rates)
{ {
int size;
if (IS_BROXTON(dev)) { if (IS_BROXTON(dev)) {
*source_rates = bxt_rates; *source_rates = bxt_rates;
return ARRAY_SIZE(bxt_rates); size = ARRAY_SIZE(bxt_rates);
} else if (IS_SKYLAKE(dev)) { } else if (IS_SKYLAKE(dev)) {
*source_rates = skl_rates; *source_rates = skl_rates;
return ARRAY_SIZE(skl_rates); size = ARRAY_SIZE(skl_rates);
} else {
*source_rates = default_rates;
size = ARRAY_SIZE(default_rates);
} }
*source_rates = default_rates;
/* This depends on the fact that 5.4 is last value in the array */ /* This depends on the fact that 5.4 is last value in the array */
if (intel_dp_source_supports_hbr2(dev)) if (!intel_dp_source_supports_hbr2(dev))
return (DP_LINK_BW_5_4 >> 3) + 1; size--;
else
return (DP_LINK_BW_2_7 >> 3) + 1; return size;
} }
static void static void
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment