diff options
| author |   Lothar Waßmann <LW@KARO-electronics.de> | 2014-01-06 14:17:58 +0100 |
|---|---|---|
| committer |   Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-01-17 10:57:41 +0200 |
| commit | 838bdf723bdc8582acf96fdb4fa783b52139fb1e (patch) | |
| tree | f0cfa21658db4968c6de497eaff81000face75bc /include/linux/platform_data | |
| parent | video: mxsfb: convert pr_debug()/dev_dbg() to pr_err()/dev_err() for error messages (diff) | |
| download | linux-dev-838bdf723bdc8582acf96fdb4fa783b52139fb1e.tar.xz linux-dev-838bdf723bdc8582acf96fdb4fa783b52139fb1e.zip | |
video: mxsfb: fix broken videomode selection
Currently the driver re-implements the code found in of_get_videomode()
except for the fact that the latter honors the 'native-mode' property
to select a spcific video timing from the list of possible timings.
The driver builds up a list of all video timings, but uses only the
last mode from the list anyway. While building the list it incorrectly
OR's the 'pixelclk-active' and 'de-active' flags of all modes into
single flags, possibly leading to a wrong pixelclock or data-enable
polarity setting.
Fix this by using the of_get_videomode() directly with the
OF_USE_NATIVE_MODE flag.
Since all current dts files only have one entry in their
display-timings node, this bug was not apparent and the fix does not
change the driver's behaviour for the current users.
Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'include/linux/platform_data')
0 files changed, 0 insertions, 0 deletions