From: Imre Deak Date: Mon, 24 Oct 2016 16:33:25 +0000 (+0300) Subject: drm/i915/dp: Remove debug dependency of DPCD SW/HW revision read X-Git-Tag: v4.10-rc1~154^2~37^2~133 X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=commitdiff_plain;h=6f172a43a6d679e4a6563376f749868a9bf29752;p=linux.git drm/i915/dp: Remove debug dependency of DPCD SW/HW revision read Performing DPCD AUX reads based on debug settings may introduce obscure bugs in other places that depend on the read being done (or being not done). To reduce the uncertainty perform the reads unconditionally. Cc: Mika Kahola Suggested-by: Jani Nikula Signed-off-by: Imre Deak Reviewed-by: Jani Nikula Link: http://patchwork.freedesktop.org/patch/msgid/1477326811-30431-3-git-send-email-imre.deak@intel.com --- diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 951a0bbc03c5..5af3907cf672 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -1456,9 +1456,6 @@ static void intel_dp_print_hw_revision(struct intel_dp *intel_dp) uint8_t rev; int len; - if ((drm_debug & DRM_UT_KMS) == 0) - return; - if (!drm_dp_is_branch(intel_dp->dpcd)) return; @@ -1474,9 +1471,6 @@ static void intel_dp_print_sw_revision(struct intel_dp *intel_dp) uint8_t rev[2]; int len; - if ((drm_debug & DRM_UT_KMS) == 0) - return; - if (!drm_dp_is_branch(intel_dp->dpcd)) return;