staging: xgifb: delete all references to mode "none"
authorAaro Koskinen <aaro.koskinen@iki.fi>
Mon, 29 Aug 2011 20:14:32 +0000 (23:14 +0300)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 6 Sep 2011 18:41:25 +0000 (11:41 -0700)
Delete all references to mode "none". It should be handled like any
other invalid mode.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/xgifb/XGI_main.h
drivers/staging/xgifb/XGI_main_26.c

index d3d58c0d56d66366d3d5240db695d3ac169936f9..467df83860ad2a5a13afcb42c0cc24043197e422 100644 (file)
@@ -211,9 +211,6 @@ static struct _XGIbios_mode {
        u16 rows;
        u8  chipset;
 } XGIbios_mode[] = {
-#define MODE_INDEX_NONE           0  /* TW: index for mode=none */
-       {"none",         0xFF, 0x0000, 0x0000,    0,    0,  0, 0,   0,  0,
-        MD_XGI300|MD_XGI315},  /* TW: for mode "none" */
        {"320x240x16",   0x56, 0x0000, 0x0000,  320,  240, 16, 1,  40, 15,
         MD_XGI315},
        {"320x480x8",    0x5A, 0x0000, 0x0000,  320,  480,  8, 1,  40, 30,
@@ -255,7 +252,7 @@ static struct _XGIbios_mode {
         MD_XGI300|MD_XGI315},
        {"800x600x8",    0x30, 0x0103, 0x0103,  800,  600,  8, 1, 100, 37,
         MD_XGI300|MD_XGI315},
-#define DEFAULT_MODE              21 /* index for 800x600x16 */
+#define DEFAULT_MODE              20 /* index for 800x600x16 */
        {"800x600x16",   0x47, 0x0114, 0x0114,  800,  600, 16, 1, 100, 37,
         MD_XGI300|MD_XGI315},
        {"800x600x24",   0x63, 0x013b, 0x0115,  800,  600, 32, 1, 100, 37,
index ff0392bb029743c2db6d79bc40114f5ce213063c..d903e8e7990d552c4729dd6e3cf346c3621fd4e5 100644 (file)
@@ -49,7 +49,7 @@
 #define XGIFB_ROM_SIZE 65536
 
 static char *mode;
-static int vesa;
+static int vesa = -1;
 
 /* -------------------- Macro definitions ---------------------------- */
 
@@ -458,11 +458,6 @@ static void XGIfb_search_mode(const char *name)
 {
        int i = 0, j = 0, l;
 
-       if (!strcmp(name, XGIbios_mode[MODE_INDEX_NONE].name)) {
-               printk(KERN_ERR "XGIfb: Mode 'none' not supported anymore. Using default.\n");
-               return;
-       }
-
        while (XGIbios_mode[i].mode_no != 0) {
                l = min(strlen(name), strlen(XGIbios_mode[i].name));
                if (!strncmp(name, XGIbios_mode[i].name, l)) {
@@ -480,11 +475,8 @@ static void XGIfb_search_vesamode(unsigned int vesamode)
 {
        int i = 0, j = 0;
 
-       if (vesamode == 0) {
-
-               printk(KERN_ERR "XGIfb: Mode 'none' not supported anymore. Using default.\n");
-               return;
-       }
+       if (vesamode == 0)
+               goto invalid;
 
        vesamode &= 0x1dff; /* Clean VESA mode number from other flags */
 
@@ -497,6 +489,8 @@ static void XGIfb_search_vesamode(unsigned int vesamode)
                }
                i++;
        }
+
+invalid:
        if (!j)
                printk(KERN_INFO "XGIfb: Invalid VESA mode 0x%x'\n", vesamode);
 }