viafb: some small cleanup for global variables

We do not need viafb_second{,_virtual}_{xres,yres} outside of
viafbdev.c so move them there and eliminate the virtual ones where
the only sane usage is done during initalization.

Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>

diff --git a/drivers/video/via/global.c b/drivers/video/via/global.c
index 1ee511b733071b1641ea5eabff3e4c9073c7f98c..e10d8249534c682543f0a43cfb9398240e357219 100644 (file)
--- a/drivers/video/via/global.c
+++ b/drivers/video/via/global.c
@@ -40,10 +40,6 @@ int viafb_hotplug_Yres = 480;
 int viafb_hotplug_bpp = 32;
 int viafb_hotplug_refresh = 60;
 int viafb_primary_dev = None_Device;
-unsigned int viafb_second_xres = 640;
-unsigned int viafb_second_yres = 480;
-unsigned int viafb_second_virtual_xres;
-unsigned int viafb_second_virtual_yres;
 int viafb_lcd_panel_id = LCD_PANEL_ID_MAXIMUM + 1;
 struct fb_info *viafbinfo;
 struct fb_info *viafbinfo1;

diff --git a/drivers/video/via/global.h b/drivers/video/via/global.h
index 38ef5ac669539dc69191f65eea8f6d7923c5ebac..ff969dc3459352bbfea5df7dec88e111576584c3 100644 (file)
--- a/drivers/video/via/global.h
+++ b/drivers/video/via/global.h
@@ -73,8 +73,6 @@ extern int viafb_hotplug_bpp;
 extern int viafb_hotplug_refresh;
 extern int viafb_primary_dev;
 
-extern unsigned int viafb_second_xres;
-extern unsigned int viafb_second_yres;
 extern int viafb_lcd_panel_id;
 
 #endif /* __GLOBAL_H__ */

diff --git a/drivers/video/via/hw.h b/drivers/video/via/hw.h
index 8858593405aab122049c5d4a1365741287e7b98b..090d167863a4a124365fffcced1010beb45087cb 100644 (file)
--- a/drivers/video/via/hw.h
+++ b/drivers/video/via/hw.h
@@ -910,7 +910,6 @@ struct via_device_mapping {
        const char *name;
 };
 
-extern unsigned int viafb_second_virtual_xres;
 extern int viafb_SAMM_ON;
 extern int viafb_dual_fb;
 extern int viafb_LCD2_ON;

diff --git a/drivers/video/via/viafbdev.c b/drivers/video/via/viafbdev.c
index ed9bd7978cc0d93dc83c64a6eec4094bf8f387a7..eace9a4257f48fc68ba4a9a0fe7a8d702c72c589 100644 (file)
--- a/drivers/video/via/viafbdev.c
+++ b/drivers/video/via/viafbdev.c
@@ -37,6 +37,8 @@ static char *viafb_mode1;
 static int viafb_bpp = 32;
 static int viafb_bpp1 = 32;
 
+static unsigned int viafb_second_xres = 640;
+static unsigned int viafb_second_yres = 480;
 static unsigned int viafb_second_offset;
 static int viafb_second_size;
 
@@ -440,8 +442,8 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
                if (viafb_SAMM_ON == 1) {
                        u.viamode.xres_sec = viafb_second_xres;
                        u.viamode.yres_sec = viafb_second_yres;
-                       u.viamode.virtual_xres_sec = viafb_second_virtual_xres;
-                       u.viamode.virtual_yres_sec = viafb_second_virtual_yres;
+                       u.viamode.virtual_xres_sec = viafb_dual_fb ? viafbinfo1->var.xres_virtual : viafbinfo->var.xres_virtual;
+                       u.viamode.virtual_yres_sec = viafb_dual_fb ? viafbinfo1->var.yres_virtual : viafbinfo->var.yres_virtual;
                        u.viamode.refresh_sec = viafb_refresh1;
                        u.viamode.bpp_sec = viafb_bpp1;
                } else {
@@ -1790,14 +1792,10 @@ int __devinit via_fb_pci_probe(struct viafb_dev *vdev)
 
        parse_mode(viafb_mode, &default_xres, &default_yres);
        vmode_entry = viafb_get_mode(default_xres, default_yres);
-       if (viafb_SAMM_ON == 1) {
+       if (viafb_SAMM_ON == 1)
                parse_mode(viafb_mode1, &viafb_second_xres,
                        &viafb_second_yres);
 
-               viafb_second_virtual_xres = viafb_second_xres;
-               viafb_second_virtual_yres = viafb_second_yres;
-       }
-
        default_var.xres = default_xres;
        default_var.yres = default_yres;
        default_var.xres_virtual = default_xres;
@@ -1841,8 +1839,8 @@ int __devinit via_fb_pci_probe(struct viafb_dev *vdev)
 
                default_var.xres = viafb_second_xres;
                default_var.yres = viafb_second_yres;
-               default_var.xres_virtual = viafb_second_virtual_xres;
-               default_var.yres_virtual = viafb_second_virtual_yres;
+               default_var.xres_virtual = viafb_second_xres;
+               default_var.yres_virtual = viafb_second_yres;
                default_var.bits_per_pixel = viafb_bpp1;
                viafb_fill_var_timing_info(&default_var, viafb_get_refresh(
                        default_var.xres, default_var.yres, viafb_refresh1),

diff --git a/drivers/video/via/viafbdev.h b/drivers/video/via/viafbdev.h
index ff60e1ddcffc9b75805545833a5b4c3c1ea6da0f..59e40d1bd61495154d59f089045ca83fd1f8ea79 100644 (file)
--- a/drivers/video/via/viafbdev.h
+++ b/drivers/video/via/viafbdev.h
@@ -83,8 +83,6 @@ struct viafb_par {
        struct chip_information *chip_info;
 };
 
-extern unsigned int viafb_second_virtual_yres;
-extern unsigned int viafb_second_virtual_xres;
 extern int viafb_SAMM_ON;
 extern int viafb_dual_fb;
 extern int viafb_LCD2_ON;