]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: Drop now misleading DDI comment from dp_link_down
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 20 May 2014 20:46:50 +0000 (22:46 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 20 May 2014 20:50:05 +0000 (22:50 +0200)
commit23217f5eba1875cf3d7664fed911b3f3d420ef51
treec04d0cce3efa9dcf8f3eee90dbfc5a4724ffa5fe
parentd3e8f945e77889de5c3e678c54603be380458a0f
drm/i915: Drop now misleading DDI comment from dp_link_down

Since

commit e28cf7cf8f7733eb2402eb34286de2ed454d5852
Author: Imre Deak <imre.deak@intel.com>
Date:   Fri Jan 17 15:46:43 2014 +0200

    drm/i915: don't disable DP port after a failed link training

and

commit 014029f7d1f9d66280b367db107bf7b71bb7df65
Author: Imre Deak <imre.deak@intel.com>
Date:   Thu Jan 16 18:35:57 2014 +0200

    drm/i915: don't disable the DP port if the link is lost

we no longer call intel_dp_link_down from generic DP code, but only
from the !HAS_DDI dp encoder functions. hsw/bdw have their own encoder
disabling callback in intel_ddi.c.

Hence the early return is no longer needed and the big comment just
confusing, so let's rip it out. To ensure what we don't accidentally
use this again on ddi encoders add a WARN_ON instead.

Spotted while reading through intel_dp.c

Cc: Imre Deak <imre.deak@intel.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_dp.c