diff options
author | Krzysztof Helt <krzysztof.h1@wp.pl> | 2008-06-06 01:45:56 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-06-06 14:29:09 -0400 |
commit | e2d4ecafd24d6eee4ae6bdbede0cfd0e78423a33 (patch) | |
tree | 156d2e0df8b8888dff9e7f885c40f4cc338d1626 /drivers/video | |
parent | b8c141e8fd80fa64d80c6a74492053f25a28e0ea (diff) |
modedb: fix incorrect sync and vmode flags for CVT modes
The temporary structure for calculated CVT mode is not initialized. Few
fields have only bits or-ed or and-ed so they may be left in incorrect
(random) state.
Testing of the tridentfb seems like a good exercise for the fbdev layer.
Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/modedb.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/video/modedb.c b/drivers/video/modedb.c index 473562191586..d1bbef930dfa 100644 --- a/drivers/video/modedb.c +++ b/drivers/video/modedb.c | |||
@@ -590,6 +590,7 @@ done: | |||
590 | "", (margins) ? " with margins" : "", (interlace) ? | 590 | "", (margins) ? " with margins" : "", (interlace) ? |
591 | " interlaced" : ""); | 591 | " interlaced" : ""); |
592 | 592 | ||
593 | memset(&cvt_mode, 0, sizeof(cvt_mode)); | ||
593 | cvt_mode.xres = xres; | 594 | cvt_mode.xres = xres; |
594 | cvt_mode.yres = yres; | 595 | cvt_mode.yres = yres; |
595 | cvt_mode.refresh = (refresh) ? refresh : 60; | 596 | cvt_mode.refresh = (refresh) ? refresh : 60; |