Commit bfcef5d2 authored by Jani Nikula's avatar Jani Nikula

drm/i915/dp: reduce missing TPS3 support errors to debug logging

Per spec, TPS3 support is mandatory for downstream devices that support
HBR2. We've therefore logged errors on HBR2 without TPS3 since

commit 1da7d713
Author: Jani Nikula <jani.nikula@intel.com>
Date:   Thu Sep 3 11:16:08 2015 +0300

    drm/i915: ignore link rate in TPS3 selection

However, it seems there are real world devices out there that just
aren't spec compliant, and still work at HBR2 using TPS2. So reduce the
error message to debug logging.

Cc: Ander Conselvan de Oliveira <conselvan2@gmail.com>
Cc: Sivakumar Thulasimani <sivakumar.thulasimani@intel.com>
Reviewed-by: default avatarSivakumar Thulasimani <sivakumar.thulasimani@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92932
Fixes: 1da7d713 ("drm/i915: ignore link rate in TPS3 selection")
Cc: drm-intel-fixes@lists.freedesktop.org
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1454667370-8001-2-git-send-email-jani.nikula@intel.com
parent 23a5110d
...@@ -222,19 +222,27 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) ...@@ -222,19 +222,27 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp)
static u32 intel_dp_training_pattern(struct intel_dp *intel_dp) static u32 intel_dp_training_pattern(struct intel_dp *intel_dp)
{ {
u32 training_pattern = DP_TRAINING_PATTERN_2; u32 training_pattern = DP_TRAINING_PATTERN_2;
bool source_tps3, sink_tps3;
/* /*
* Intel platforms that support HBR2 also support TPS3. TPS3 support is * Intel platforms that support HBR2 also support TPS3. TPS3 support is
* also mandatory for downstream devices that support HBR2. * also mandatory for downstream devices that support HBR2. However, not
* all sinks follow the spec.
* *
* Due to WaDisableHBR2 SKL < B0 is the only exception where TPS3 is * Due to WaDisableHBR2 SKL < B0 is the only exception where TPS3 is
* supported but still not enabled. * supported in source but still not enabled.
*/ */
if (intel_dp_source_supports_hbr2(intel_dp) && source_tps3 = intel_dp_source_supports_hbr2(intel_dp);
drm_dp_tps3_supported(intel_dp->dpcd)) sink_tps3 = drm_dp_tps3_supported(intel_dp->dpcd);
if (source_tps3 && sink_tps3) {
training_pattern = DP_TRAINING_PATTERN_3; training_pattern = DP_TRAINING_PATTERN_3;
else if (intel_dp->link_rate == 540000) } else if (intel_dp->link_rate == 540000) {
DRM_ERROR("5.4 Gbps link rate without HBR2/TPS3 support\n"); if (!source_tps3)
DRM_DEBUG_KMS("5.4 Gbps link rate without source HBR2/TPS3 support\n");
if (!sink_tps3)
DRM_DEBUG_KMS("5.4 Gbps link rate without sink TPS3 support\n");
}
return training_pattern; return training_pattern;
} }
......
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