drm/i915: Don't add panel_fixed_mode to the probed modes list at LVDS init.
authorSteve Aarnio <steve.j.aarnio@linux.intel.com>
Thu, 12 Feb 2009 19:34:02 +0000 (11:34 -0800)
committerDave Airlie <airlied@redhat.com>
Fri, 20 Feb 2009 02:21:12 +0000 (12:21 +1000)
commit67eabc0553a32c491fdb392ff2358a0384562050
tree3110a71cedec4f54b8fe6fb60bb34ce7da4235b9
parentea39f835168f60b01e59d0f348da25d297e7cf94
drm/i915: Don't add panel_fixed_mode to the probed modes list at LVDS init.

In the case where no EDID data is read from the device, adding the
panel_fixed_mode pointer to the probed modes list causes data corruption.

If the panel_fixed_mode pointer is added to the probed modes list at
init time, a copy of the mode is added again at drm_get_modes() request
time.  Then, the panel_fixed_mode pointer is freed because it is seen as
a duplicate mode.  Unfortunately, this pointer is still stored and used
in mode_fixup().

Because the panel_fixed_mode data is copied and returned at
drm_get_modes() time, it is unnecessary to add this information at init
time.

Signed-off-by: Steve Aarnio <steve.j.aarnio@intel.com>
Signed-off-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Dave Airlie <airlied@linux.ie>
drivers/gpu/drm/i915/intel_lvds.c