viafb: rename output devices
authorFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>
Sun, 19 Sep 2010 04:40:15 +0000 (04:40 +0000)
committerFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>
Fri, 24 Sep 2010 02:15:25 +0000 (02:15 +0000)
Now it looks like we finally know enough about the output devices to give
them proper names. As VIA_96 is often referred to as DVP0 rename it to
VIA_DVP0. As VIA_6C and VIA_93 seem to exist only on CLE266 and "replace"
DVP0 and DVP1 there rename them to VIA_LDVP0 and VIA_LDVP1 (L as legacy).
The proc names were changed accordingly which should be harmless as they
were just introduced and not beyond RFC state.
This patch should make things a bit more comfortable and less scary.

Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
Acked-by: Jonathan Corbet <corbet@lwn.net>
Cc: Joseph Chan <JosephChan@via.com.tw>
Cc: Andrew Morton <akpm@linux-foundation.org>
drivers/video/via/hw.c
drivers/video/via/hw.h
drivers/video/via/viafbdev.c

index 00fecb579acc067752e29c94e2336767a38e5509..7d7010acf1482576d69228e7ec7e2db42df666e8 100644 (file)
@@ -719,9 +719,9 @@ static struct rgbLUT palLUT_table[] = {
 };
 
 static struct via_device_mapping device_mapping[] = {
-       {VIA_6C, "6C"},
-       {VIA_93, "93"},
-       {VIA_96, "96"},
+       {VIA_LDVP0, "LDVP0"},
+       {VIA_LDVP1, "LDVP1"},
+       {VIA_DVP0, "DVP0"},
        {VIA_CRT, "CRT"},
        {VIA_DVP1, "DVP1"},
        {VIA_LVDS1, "LVDS1"},
@@ -763,11 +763,11 @@ static u32 get_dvi_devices(int output_interface)
 {
        switch (output_interface) {
        case INTERFACE_DVP0:
-               return VIA_96 | VIA_6C;
+               return VIA_DVP0 | VIA_LDVP0;
 
        case INTERFACE_DVP1:
                if (viaparinfo->chip_info->gfx_chip_name == UNICHROME_CLE266)
-                       return VIA_93;
+                       return VIA_LDVP1;
                else
                        return VIA_DVP1;
 
@@ -775,7 +775,7 @@ static u32 get_dvi_devices(int output_interface)
                if (viaparinfo->chip_info->gfx_chip_name == UNICHROME_CLE266)
                        return 0;
                else
-                       return VIA_LVDS2 | VIA_96;
+                       return VIA_LVDS2 | VIA_DVP0;
 
        case INTERFACE_DFP_LOW:
                if (viaparinfo->chip_info->gfx_chip_name == UNICHROME_CLE266)
@@ -794,13 +794,13 @@ static u32 get_lcd_devices(int output_interface)
 {
        switch (output_interface) {
        case INTERFACE_DVP0:
-               return VIA_96;
+               return VIA_DVP0;
 
        case INTERFACE_DVP1:
                return VIA_DVP1;
 
        case INTERFACE_DFP_HIGH:
-               return VIA_LVDS2 | VIA_96;
+               return VIA_LVDS2 | VIA_DVP0;
 
        case INTERFACE_DFP_LOW:
                return VIA_LVDS1 | VIA_DVP1;
@@ -988,17 +988,17 @@ static void set_crt_source(u8 iga)
        via_write_reg_mask(VIASR, 0x16, value, 0x40);
 }
 
-static inline void set_6C_source(u8 iga)
+static inline void set_ldvp0_source(u8 iga)
 {
        set_source_common(0x6C, 7, iga);
 }
 
-static inline void set_93_source(u8 iga)
+static inline void set_ldvp1_source(u8 iga)
 {
        set_source_common(0x93, 7, iga);
 }
 
-static inline void set_96_source(u8 iga)
+static inline void set_dvp0_source(u8 iga)
 {
        set_source_common(0x96, 4, iga);
 }
@@ -1020,12 +1020,12 @@ static inline void set_lvds2_source(u8 iga)
 
 void via_set_source(u32 devices, u8 iga)
 {
-       if (devices & VIA_6C)
-               set_6C_source(iga);
-       if (devices & VIA_93)
-               set_93_source(iga);
-       if (devices & VIA_96)
-               set_96_source(iga);
+       if (devices & VIA_LDVP0)
+               set_ldvp0_source(iga);
+       if (devices & VIA_LDVP1)
+               set_ldvp1_source(iga);
+       if (devices & VIA_DVP0)
+               set_dvp0_source(iga);
        if (devices & VIA_CRT)
                set_crt_source(iga);
        if (devices & VIA_DVP1)
@@ -1060,7 +1060,7 @@ static void set_crt_state(u8 state)
        via_write_reg_mask(VIACR, 0x36, value, 0x30);
 }
 
-static void set_96_state(u8 state)
+static void set_dvp0_state(u8 state)
 {
        u8 value;
 
@@ -1136,11 +1136,11 @@ void via_set_state(u32 devices, u8 state)
 {
        /*
        TODO: Can we enable/disable these devices? How?
-       if (devices & VIA_6C)
-       if (devices & VIA_93)
+       if (devices & VIA_LDVP0)
+       if (devices & VIA_LDVP1)
        */
-       if (devices & VIA_96)
-               set_96_state(state);
+       if (devices & VIA_DVP0)
+               set_dvp0_state(state);
        if (devices & VIA_CRT)
                set_crt_state(state);
        if (devices & VIA_DVP1)
index ad6f9b1cf40eb49242369f1246b6a9605f92f4a6..b764409467714f3aa8a176b923098acfc8632540 100644 (file)
@@ -33,9 +33,9 @@
 #define viafb_write_reg_mask(i, p, d, m)       via_write_reg_mask(p, i, d, m)
 
 /* VIA output devices */
-#define VIA_6C         0x00000001
-#define VIA_93         0x00000002
-#define VIA_96         0x00000004
+#define VIA_LDVP0      0x00000001
+#define VIA_LDVP1      0x00000002
+#define VIA_DVP0       0x00000004
 #define VIA_CRT                0x00000010
 #define VIA_DVP1       0x00000020
 #define VIA_LVDS1      0x00000040
index 50f486d9b087b0c00395d4c2a012bf807e2c677e..7cc057d7ef5e530ab50697641e999fc78bfa07d4 100644 (file)
@@ -64,14 +64,14 @@ static struct fb_ops viafb_ops;
  * K400 seems to support VIA_96, VIA_DVP1, VIA_LVDS{1,2} as in viamode.c
  */
 static const u32 supported_odev_map[] = {
-       [UNICHROME_CLE266]      = VIA_CRT | VIA_6C | VIA_93,
-       [UNICHROME_K400]        = VIA_CRT | VIA_96 | VIA_DVP1 | VIA_LVDS1
+       [UNICHROME_CLE266]      = VIA_CRT | VIA_LDVP0 | VIA_LDVP1,
+       [UNICHROME_K400]        = VIA_CRT | VIA_DVP0 | VIA_DVP1 | VIA_LVDS1
                                | VIA_LVDS2,
-       [UNICHROME_K800]        = VIA_CRT | VIA_96 | VIA_DVP1 | VIA_LVDS1
+       [UNICHROME_K800]        = VIA_CRT | VIA_DVP0 | VIA_DVP1 | VIA_LVDS1
                                | VIA_LVDS2,
-       [UNICHROME_PM800]       = VIA_CRT | VIA_96 | VIA_DVP1 | VIA_LVDS1
+       [UNICHROME_PM800]       = VIA_CRT | VIA_DVP0 | VIA_DVP1 | VIA_LVDS1
                                | VIA_LVDS2,
-       [UNICHROME_CN700]       = VIA_CRT | VIA_96 | VIA_DVP1 | VIA_LVDS1
+       [UNICHROME_CN700]       = VIA_CRT | VIA_DVP0 | VIA_DVP1 | VIA_LVDS1
                                | VIA_LVDS2,
        [UNICHROME_CX700]       = VIA_CRT | VIA_DVP1 | VIA_LVDS1 | VIA_LVDS2,
        [UNICHROME_CN750]       = VIA_CRT | VIA_DVP1 | VIA_LVDS1 | VIA_LVDS2,