]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: asserts for lvds pre_enable
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Sun, 16 Jun 2013 19:42:39 +0000 (21:42 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 1 Jul 2013 09:27:51 +0000 (11:27 +0200)
commitef97bddde2d79f5a4528b87832e2dec331fc9821
tree921de06207a5a36cd2e1067b143514c8c3ef460c
parent3d0a3f49bf9d11f5a993802f1fc1395b64111724
drm/i915: asserts for lvds pre_enable

Lots of bangin my head against the wall^UExperiments have shown that
we really need to enable the lvds port before we enable plls. Strangely
that seems to include the fdi rx pll on the pch.

Note that the pch pll assert can fire since the lvds port has it's own
special clock source settings in the DPLL register, which means it
will never have a shared dpll (since there's only one LVDS port).

Anyway, encode this new evidence with a few nice WARNs.

v2: Incorporate review comments from Imre.
- Explain why lvds can't have a shared dpll.
- Update the WARN output.

Cc: Imre Deak <imre.deak@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/i915/intel_drv.h
drivers/gpu/drm/i915/intel_lvds.c