viafb: clean up duoview
authorFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>
Tue, 22 Sep 2009 23:47:18 +0000 (16:47 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 23 Sep 2009 14:39:52 +0000 (07:39 -0700)
Clean the duoview handling up by replacing the varible with the funtion in
the only place where it is used.  This is a code cleanup only, no runtime
change expected.

Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
Cc: Scott Fang <ScottFang@viatech.com.cn>
Cc: Joseph Chan <JosephChan@via.com.tw>
Cc: Harald Welte <laforge@gnumonks.org>
Cc: Jonathan Corbet <corbet@lwn.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/video/via/viafbdev.c
drivers/video/via/viafbdev.h

index 50e9300d0de0a0894afe422191630fc0b0c21679..0cd112a3690185d6a09cab82a9b636442a30ed77 100644 (file)
@@ -1009,7 +1009,8 @@ static int viafb_cursor(struct fb_info *info, struct fb_cursor *cursor)
                return -ENODEV;
 
        /* When duoview and using lcd , use soft cursor */
-       if (viafb_LCD_ON || ((struct viafb_par *)(info->par))->duoview)
+       if (viafb_LCD_ON || (!viafb_SAMM_ON &&
+               viafb_LCD2_ON + viafb_DVI_ON + viafb_CRT_ON == 2))
                return -ENODEV;
 
        viafb_show_hw_cursor(info, HW_Cursor_OFF);
@@ -1379,18 +1380,6 @@ static int get_primary_device(void)
        return primary_device;
 }
 
-static u8 is_duoview(void)
-{
-       if (0 == viafb_SAMM_ON) {
-               if (viafb_LCD_ON + viafb_LCD2_ON +
-                       viafb_DVI_ON + viafb_CRT_ON == 2)
-                       return true;
-               return false;
-       } else {
-               return false;
-       }
-}
-
 static void apply_second_mode_setting(struct fb_var_screeninfo
        *sec_var)
 {
@@ -1499,8 +1488,6 @@ static int apply_device_setting(struct viafb_ioctl_setting setting_info,
                need_set_mode = 1;
        }
 
-       viaparinfo->duoview = is_duoview();
-
        if (!need_set_mode) {
                ;
        } else {
@@ -1621,7 +1608,6 @@ static void parse_active_dev(void)
                viafb_CRT_ON = STATE_ON;
                viafb_SAMM_ON = STATE_OFF;
        }
-       viaparinfo->duoview = is_duoview();
 }
 
 static void parse_video_dev(void)
index cd3dff7cc1709a95b386e357d9e99f70adb3677f..92318770107eb6eefc89fa90c5e8f0e1e8935a41 100644 (file)
@@ -52,7 +52,6 @@ struct viafb_par {
        u32 VQ_end;             /* Virtual Queue End Address */
        u32 iga_path;
        struct proc_dir_entry *proc_entry;      /*viafb proc entry */
-       u8 duoview;             /*Is working in duoview mode? */
 
        /* I2C stuff */
        struct via_i2c_stuff i2c_stuff;