diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2010-08-22 09:54:23 (GMT) |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-09-07 10:14:19 (GMT) |
commit | 8e647a279ca30029f19eca646de08a6338eab924 (patch) | |
tree | c3d6704c962d50174705abfb4426cadd38d164f6 /lib/klist.c | |
parent | a25c25c2a2aa55e609099a9f74453c518aec29a6 (diff) | |
download | linux-fsl-qoriq-8e647a279ca30029f19eca646de08a6338eab924.tar.xz |
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>
Diffstat (limited to 'lib/klist.c')
0 files changed, 0 insertions, 0 deletions