]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commit
drm/i915: Tightly scope intel_encoder to prevent invalid use
authorChris Wilson <chris@chris-wilson.co.uk>
Sun, 22 Aug 2010 09:54:23 +0000 (10:54 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Tue, 7 Sep 2010 10:14:19 +0000 (11:14 +0100)
commit8e647a279ca30029f19eca646de08a6338eab924
treec3d6704c962d50174705abfb4426cadd38d164f6
parenta25c25c2a2aa55e609099a9f74453c518aec29a6
drm/i915: Tightly scope intel_encoder to prevent invalid use

We reset intel_encoder for every matching encoder whilst iterating over
the encoders attached to this crtc when changing mode. As such in a
cloned configuration intel_encoder may not correspond to the correct
is_edp encoder.

By scoping intel_encoder to the loop, not only is the compiler able to
spot this mistake, we also improve readiability for ourselves.
[It might not be a mistake, within this function it is unclear as to
whether it is permissable for eDP to be cloned...]

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/intel_display.c