11 files changed, 63 insertions, 72 deletions

diff --git a/drivers/video/aty/radeon_base.c b/drivers/video/aty/radeon_base.c
index 2ce050193018..2349e71b0083 100644
--- a/drivers/video/aty/radeon_base.c
+++ b/drivers/video/aty/radeon_base.c
@@ -2102,7 +2102,9 @@ static ssize_t radeon_show_one_edid(char *buf, loff_t off, size_t count, const u
 }
 
 
-static ssize_t radeon_show_edid1(struct kobject *kobj, char *buf, loff_t off, size_t count)
+static ssize_t radeon_show_edid1(struct kobject *kobj,
+                                 struct bin_attribute *bin_attr,
+                                 char *buf, loff_t off, size_t count)
 {
         struct device *dev = container_of(kobj, struct device, kobj);
         struct pci_dev *pdev = to_pci_dev(dev);
@@ -2113,7 +2115,9 @@ static ssize_t radeon_show_edid1(struct kobject *kobj, char *buf, loff_t off, si
 }
 
 
-static ssize_t radeon_show_edid2(struct kobject *kobj, char *buf, loff_t off, size_t count)
+static ssize_t radeon_show_edid2(struct kobject *kobj,
+                                 struct bin_attribute *bin_attr,
+                                 char *buf, loff_t off, size_t count)
 {
         struct device *dev = container_of(kobj, struct device, kobj);
         struct pci_dev *pdev = to_pci_dev(dev);
@@ -2126,7 +2130,6 @@ static ssize_t radeon_show_edid2(struct kobject *kobj, char *buf, loff_t off, si
 static struct bin_attribute edid1_attr = {
         .attr   = {
                 .name   = "edid1",
-                .owner  = THIS_MODULE,
                 .mode   = 0444,
         },
         .size   = EDID_LENGTH,
@@ -2136,7 +2139,6 @@ static struct bin_attribute edid1_attr = {
 static struct bin_attribute edid2_attr = {
         .attr   = {
                 .name   = "edid2",
-                .owner  = THIS_MODULE,
                 .mode   = 0444,
         },
         .size   = EDID_LENGTH,
diff --git a/drivers/video/backlight/backlight.c b/drivers/video/backlight/backlight.c
index c65e81ff3578..7e06223bca94 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -172,7 +172,7 @@ static struct class backlight_class = {
 
 #define DECLARE_ATTR(_name,_mode,_show,_store)                  \
 {                                                               \
-        .attr   = { .name = __stringify(_name), .mode = _mode, .owner = THIS_MODULE },  \
+        .attr   = { .name = __stringify(_name), .mode = _mode }, \
         .show   = _show,                                        \
         .store  = _store,                                       \
 }
diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
index 6ef8f0a7a137..648b53c1fdea 100644
--- a/drivers/video/backlight/lcd.c
+++ b/drivers/video/backlight/lcd.c
@@ -157,7 +157,7 @@ static struct class lcd_class = {
 
 #define DECLARE_ATTR(_name,_mode,_show,_store)                  \
 {                                                               \
-        .attr   = { .name = __stringify(_name), .mode = _mode, .owner = THIS_MODULE },  \
+        .attr   = { .name = __stringify(_name), .mode = _mode }, \
         .show   = _show,                                        \
         .store  = _store,                                       \
 }
diff --git a/drivers/video/console/Kconfig b/drivers/video/console/Kconfig
index 63b85bf81a65..d3b8a6be2916 100644
--- a/drivers/video/console/Kconfig
+++ b/drivers/video/console/Kconfig
@@ -6,7 +6,7 @@ menu "Console display driver support"
 
 config VGA_CONSOLE
         bool "VGA text console" if EMBEDDED || !X86
-        depends on !ARCH_ACORN && !ARCH_EBSA110 && !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && !ARCH_VERSATILE && !SUPERH
+        depends on !ARCH_ACORN && !ARCH_EBSA110 && !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && !ARCH_VERSATILE && !SUPERH && !BFIN
         default y
         help
           Saying Y here will allow you to use Linux in text mode through a
diff --git a/drivers/video/kyro/STG4000InitDevice.c b/drivers/video/kyro/STG4000InitDevice.c
index ab5285a7f1d6..1d3f2080aa6f 100644
--- a/drivers/video/kyro/STG4000InitDevice.c
+++ b/drivers/video/kyro/STG4000InitDevice.c
@@ -247,7 +247,6 @@ int SetCoreClockPLL(volatile STG4000REG __iomem *pSTGReg, struct pci_dev *pDev)
         u32 ulCoreClock;
         u32 tmp;
         u32 ulChipSpeed;
-        u8 rev;
 
         STG_WRITE_REG(IntMask, 0xFFFF);
 
@@ -276,9 +275,9 @@ int SetCoreClockPLL(volatile STG4000REG __iomem *pSTGReg, struct pci_dev *pDev)
                       PMX2_SOFTRESET_ROM_RST);
 
         pci_read_config_word(pDev, PCI_CONFIG_SUBSYS_ID, &sub);
-        pci_read_config_byte(pDev, PCI_REVISION_ID, &rev);
 
-        ulChipSpeed = InitSDRAMRegisters(pSTGReg, (u32)sub, (u32)rev);
+        ulChipSpeed = InitSDRAMRegisters(pSTGReg, (u32)sub,
+                                         (u32)pDev->revision);
 
         if (ulChipSpeed == 0)
                 return -EINVAL;
diff --git a/drivers/video/matrox/matroxfb_base.c b/drivers/video/matrox/matroxfb_base.c
index c8559a756b75..886e475f22f2 100644
--- a/drivers/video/matrox/matroxfb_base.c
+++ b/drivers/video/matrox/matroxfb_base.c
@@ -1994,7 +1994,6 @@ static void matroxfb_unregister_device(struct matrox_fb_info* minfo) {
 
 static int matroxfb_probe(struct pci_dev* pdev, const struct pci_device_id* dummy) {
         struct board* b;
-        u_int8_t rev;
         u_int16_t svid;
         u_int16_t sid;
         struct matrox_fb_info* minfo;
@@ -2005,11 +2004,10 @@ static int matroxfb_probe(struct pci_dev* pdev, const struct pci_device_id* dumm
 #endif
         DBG(__FUNCTION__)
 
-        pci_read_config_byte(pdev, PCI_REVISION_ID, &rev);
         svid = pdev->subsystem_vendor;
         sid = pdev->subsystem_device;
         for (b = dev_list; b->vendor; b++) {
-                if ((b->vendor != pdev->vendor) || (b->device != pdev->device) || (b->rev < rev)) continue;
+                if ((b->vendor != pdev->vendor) || (b->device != pdev->device) || (b->rev < pdev->revision)) continue;
                 if (b->svid)
                         if ((b->svid != svid) || (b->sid != sid)) continue;
                 break;
diff --git a/drivers/video/matrox/matroxfb_crtc2.h b/drivers/video/matrox/matroxfb_crtc2.h
index 608e40bb20e9..177177609be7 100644
--- a/drivers/video/matrox/matroxfb_crtc2.h
+++ b/drivers/video/matrox/matroxfb_crtc2.h
@@ -2,8 +2,6 @@
 #define __MATROXFB_CRTC2_H__
 
 #include <linux/ioctl.h>
-#include <linux/i2c.h>
-#include <linux/i2c-algo-bit.h>
 #include "matroxfb_base.h"
 
 struct matroxfb_dh_fb_info {
diff --git a/drivers/video/pvr2fb.c b/drivers/video/pvr2fb.c
index df2909ae704c..2ba959a83eb0 100644
--- a/drivers/video/pvr2fb.c
+++ b/drivers/video/pvr2fb.c
@@ -115,11 +115,11 @@ enum { VO_PAL, VO_NTSC, VO_VGA };
 enum { PAL_ARGB1555, PAL_RGB565, PAL_ARGB4444, PAL_ARGB8888 };
 
 struct pvr2_params { unsigned int val; char *name; };
-static struct pvr2_params cables[] __initdata = {
+static struct pvr2_params cables[] __devinitdata = {
         { CT_VGA, "VGA" }, { CT_RGB, "RGB" }, { CT_COMPOSITE, "COMPOSITE" },
 };
 
-static struct pvr2_params outputs[] __initdata = {
+static struct pvr2_params outputs[] __devinitdata = {
         { VO_PAL, "PAL" }, { VO_NTSC, "NTSC" }, { VO_VGA, "VGA" },
 };
 
@@ -147,16 +147,16 @@ static struct pvr2fb_par {
 
 static struct fb_info *fb_info;
 
-static struct fb_fix_screeninfo pvr2_fix __initdata = {
+static struct fb_fix_screeninfo pvr2_fix __devinitdata = {
         .id =           "NEC PowerVR2",
-        .type =         FB_TYPE_PACKED_PIXELS,
-        .visual =       FB_VISUAL_TRUECOLOR,
+        .type =         FB_TYPE_PACKED_PIXELS,
+        .visual =       FB_VISUAL_TRUECOLOR,
         .ypanstep =     1,
         .ywrapstep =    1,
-        .accel =        FB_ACCEL_NONE,
+        .accel =        FB_ACCEL_NONE,
 };
 
-static struct fb_var_screeninfo pvr2_var __initdata = {
+static struct fb_var_screeninfo pvr2_var __devinitdata = {
         .xres =         640,
         .yres =         480,
         .xres_virtual = 640,
@@ -195,10 +195,6 @@ static unsigned int shdma = PVR2_CASCADE_CHAN;
 static unsigned int pvr2dma = ONCHIP_NR_DMA_CHANNELS;
 #endif
 
-/* Interface used by the world */
-
-int pvr2fb_setup(char*);
-
 static int pvr2fb_setcolreg(unsigned int regno, unsigned int red, unsigned int green, unsigned int blue,
                             unsigned int transp, struct fb_info *info);
 static int pvr2fb_blank(int blank, struct fb_info *info);
@@ -227,12 +223,12 @@ static struct fb_ops pvr2fb_ops = {
 #ifdef CONFIG_SH_DMA
         .fb_write       = pvr2fb_write,
 #endif
-        .fb_fillrect    = cfb_fillrect,
+        .fb_fillrect    = cfb_fillrect,
         .fb_copyarea    = cfb_copyarea,
         .fb_imageblit   = cfb_imageblit,
 };
 
-static struct fb_videomode pvr2_modedb[] __initdata = {
+static struct fb_videomode pvr2_modedb[] __devinitdata = {
     /*
      * Broadcast video modes (PAL and NTSC).  I'm unfamiliar with
      * PAL-M and PAL-N, but from what I've read both modes parallel PAL and
@@ -252,7 +248,7 @@ static struct fb_videomode pvr2_modedb[] __initdata = {
         /* 640x480 @ 60hz (VGA) */
         "vga_640x480", 60, 640, 480, VGA_CLK, 38, 33, 0, 18, 146, 26,
         0, FB_VMODE_YWRAP
-    }, 
+    },
 };
 
 #define NUM_TOTAL_MODES  ARRAY_SIZE(pvr2_modedb)
@@ -262,7 +258,7 @@ static struct fb_videomode pvr2_modedb[] __initdata = {
 #define DEFMODE_VGA     2
 
 static int defmode = DEFMODE_NTSC;
-static char *mode_option __initdata = NULL;
+static char *mode_option __devinitdata = NULL;
 
 static inline void pvr2fb_set_pal_type(unsigned int type)
 {
@@ -293,7 +289,7 @@ static void set_color_bitfields(struct fb_var_screeninfo *var)
 {
         switch (var->bits_per_pixel) {
             case 16:        /* RGB 565 */
-                pvr2fb_set_pal_type(PAL_RGB565);
+                pvr2fb_set_pal_type(PAL_RGB565);
                 var->red.offset = 11;    var->red.length = 5;
                 var->green.offset = 5;   var->green.length = 6;
                 var->blue.offset = 0;    var->blue.length = 5;
@@ -306,7 +302,7 @@ static void set_color_bitfields(struct fb_var_screeninfo *var)
                 var->transp.offset = 0;  var->transp.length = 0;
                 break;
             case 32:        /* ARGB 8888 */
-                pvr2fb_set_pal_type(PAL_ARGB8888);
+                pvr2fb_set_pal_type(PAL_ARGB8888);
                 var->red.offset = 16;    var->red.length = 8;
                 var->green.offset = 8;   var->green.length = 8;
                 var->blue.offset = 0;    var->blue.length = 8;
@@ -379,13 +375,13 @@ static int pvr2fb_set_par(struct fb_info *info)
         var->vmode &= FB_VMODE_MASK;
         if (var->vmode & FB_VMODE_INTERLACED && video_output != VO_VGA)
                 par->is_interlaced = 1;
-        /* 
+        /*
          * XXX: Need to be more creative with this (i.e. allow doublecan for
          * PAL/NTSC output).
          */
         if (var->vmode & FB_VMODE_DOUBLE && video_output == VO_VGA)
                 par->is_doublescan = 1;
-        
+
         par->hsync_total = var->left_margin + var->xres + var->right_margin +
                            var->hsync_len;
         par->vsync_total = var->upper_margin + var->yres + var->lower_margin +
@@ -408,7 +404,7 @@ static int pvr2fb_set_par(struct fb_info *info)
         } else {
                 /* VGA mode */
                 /* XXX: What else needs to be checked? */
-                /* 
+                /*
                  * XXX: We have a little freedom in VGA modes, what ranges
                  * should be here (i.e. hsync/vsync totals, etc.)?
                  */
@@ -419,8 +415,8 @@ static int pvr2fb_set_par(struct fb_info *info)
         /* Calculate the remainding offsets */
         par->diwstart_h = par->borderstart_h + var->left_margin;
         par->diwstart_v = par->borderstart_v + var->upper_margin;
-        par->borderstop_h = par->diwstart_h + var->xres + 
-                            var->right_margin;    
+        par->borderstop_h = par->diwstart_h + var->xres +
+                            var->right_margin;
         par->borderstop_v = par->diwstart_v + var->yres +
                             var->lower_margin;
 
@@ -465,12 +461,12 @@ static int pvr2fb_check_var(struct fb_var_screeninfo *var, struct fb_info *info)
         set_color_bitfields(var);
 
         if (var->vmode & FB_VMODE_YWRAP) {
-                if (var->xoffset || var->yoffset < 0 || 
+                if (var->xoffset || var->yoffset < 0 ||
                     var->yoffset >= var->yres_virtual) {
                         var->xoffset = var->yoffset = 0;
                 } else {
                         if (var->xoffset > var->xres_virtual - var->xres ||
-                            var->yoffset > var->yres_virtual - var->yres || 
+                            var->yoffset > var->yres_virtual - var->yres ||
                             var->xoffset < 0 || var->yoffset < 0)
                                 var->xoffset = var->yoffset = 0;
                 }
@@ -478,7 +474,7 @@ static int pvr2fb_check_var(struct fb_var_screeninfo *var, struct fb_info *info)
                 var->xoffset = var->yoffset = 0;
         }
 
-        /* 
+        /*
          * XXX: Need to be more creative with this (i.e. allow doublecan for
          * PAL/NTSC output).
          */
@@ -507,7 +503,7 @@ static int pvr2fb_check_var(struct fb_var_screeninfo *var, struct fb_info *info)
                 var->vsync_len    = par->borderstop_v +
                                    (par->vsync_total - par->borderstop_v);
         }
-                
+
         hsync_total = var->left_margin + var->xres + var->right_margin +
                       var->hsync_len;
         vtotal = var->upper_margin + var->yres + var->lower_margin +
@@ -531,7 +527,7 @@ static int pvr2fb_check_var(struct fb_var_screeninfo *var, struct fb_info *info)
                         }
                 }
         }
-        
+
         /* Check memory sizes */
         line_length = get_line_length(var->xres_virtual, var->bits_per_pixel);
         if (line_length * var->yres_virtual > info->fix.smem_len)
@@ -552,7 +548,7 @@ static void pvr2_update_display(struct fb_info *info)
                   DISP_DIWADDRS);
 }
 
-/* 
+/*
  * Initialize the video mode.  Currently, the 16bpp and 24bpp modes aren't
  * very stable.  It's probably due to the fact that a lot of the 2D video
  * registers are still undocumented.
@@ -592,18 +588,18 @@ static void pvr2_init_display(struct fb_info *info)
         /* display window start position */
         fb_writel(par->diwstart_h, DISP_DIWHSTRT);
         fb_writel((par->diwstart_v << 16) | par->diwstart_v, DISP_DIWVSTRT);
-        
+
         /* misc. settings */
         fb_writel((0x16 << 16) | par->is_lowres, DISP_DIWCONF);
 
         /* clock doubler (for VGA), scan doubler, display enable */
-        fb_writel(((video_output == VO_VGA) << 23) | 
+        fb_writel(((video_output == VO_VGA) << 23) |
                   (par->is_doublescan << 1) | 1, DISP_DIWMODE);
 
         /* bits per pixel */
         fb_writel(fb_readl(DISP_DIWMODE) | (--bytesperpixel << 2), DISP_DIWMODE);
 
-        /* video enable, color sync, interlace, 
+        /* video enable, color sync, interlace,
          * hsync and vsync polarity (currently unused) */
         fb_writel(0x100 | ((par->is_interlaced /*|4*/) << 4), DISP_SYNCCONF);
 }
@@ -657,7 +653,7 @@ static irqreturn_t pvr2fb_interrupt(int irq, void *dev_id)
 static int pvr2_init_cable(void)
 {
         if (cable_type < 0) {
-                fb_writel((fb_readl(PCTRA) & 0xfff0ffff) | 0x000a0000, 
+                fb_writel((fb_readl(PCTRA) & 0xfff0ffff) | 0x000a0000,
                           PCTRA);
                 cable_type = (fb_readw(PDTRA) >> 8) & 3;
         }
@@ -687,7 +683,7 @@ static ssize_t pvr2fb_write(struct fb_info *info, const char *buf,
         pages = kmalloc(nr_pages * sizeof(struct page *), GFP_KERNEL);
         if (!pages)
                 return -ENOMEM;
-        
+
         down_read(&current->mm->mmap_sem);
         ret = get_user_pages(current, current->mm, (unsigned long)buf,
                              nr_pages, WRITE, 0, pages, NULL);
@@ -700,7 +696,7 @@ static ssize_t pvr2fb_write(struct fb_info *info, const char *buf,
         }
 
         dma_configure_channel(shdma, 0x12c1);
-        
+
         dst   = (unsigned long)fb_info->screen_base + *ppos;
         start = (unsigned long)page_address(pages[0]);
         end   = (unsigned long)page_address(pages[nr_pages]);
@@ -744,7 +740,7 @@ out_unmap:
         kfree(pages);
 
         return ret;
-} 
+}
 #endif /* CONFIG_SH_DMA */
 
 /**
@@ -765,21 +761,21 @@ out_unmap:
  * in for flexibility anyways. Who knows, maybe someone has tv-out on a
  * PCI-based version of these things ;-)
  */
-static int __init pvr2fb_common_init(void)
+static int __devinit pvr2fb_common_init(void)
 {
         struct pvr2fb_par *par = currentpar;
         unsigned long modememused, rev;
 
         fb_info->screen_base = ioremap_nocache(pvr2_fix.smem_start,
                                                pvr2_fix.smem_len);
-        
+
         if (!fb_info->screen_base) {
                 printk(KERN_ERR "pvr2fb: Failed to remap smem space\n");
                 goto out_err;
         }
 
         par->mmio_base = (unsigned long)ioremap_nocache(pvr2_fix.mmio_start,
-                                                        pvr2_fix.mmio_len);
+                                                        pvr2_fix.mmio_len);
         if (!par->mmio_base) {
                 printk(KERN_ERR "pvr2fb: Failed to remap mmio space\n");
                 goto out_err;
@@ -820,7 +816,7 @@ static int __init pvr2fb_common_init(void)
         printk("fb%d: %s (rev %ld.%ld) frame buffer device, using %ldk/%ldk of video memory\n",
                fb_info->node, fb_info->fix.id, (rev >> 4) & 0x0f, rev & 0x0f,
                modememused >> 10, (unsigned long)(fb_info->fix.smem_len >> 10));
-        printk("fb%d: Mode %dx%d-%d pitch = %ld cable: %s video output: %s\n", 
+        printk("fb%d: Mode %dx%d-%d pitch = %ld cable: %s video output: %s\n",
                fb_info->node, fb_info->var.xres, fb_info->var.yres,
                fb_info->var.bits_per_pixel,
                get_line_length(fb_info->var.xres, fb_info->var.bits_per_pixel),
@@ -878,8 +874,8 @@ static int __init pvr2fb_dc_init(void)
                         video_output = VO_NTSC;
                 }
         }
-        
-        /* 
+
+        /*
          * Nothing exciting about the DC PVR2 .. only a measly 8MiB.
          */
         pvr2_fix.smem_start     = 0xa5000000;   /* RAM starts here */
@@ -903,7 +899,7 @@ static int __init pvr2fb_dc_init(void)
         return pvr2fb_common_init();
 }
 
-static void pvr2fb_dc_exit(void)
+static void __exit pvr2fb_dc_exit(void)
 {
         if (fb_info->screen_base) {
                 iounmap(fb_info->screen_base);
@@ -987,13 +983,13 @@ static int __init pvr2fb_pci_init(void)
         return pci_register_driver(&pvr2fb_pci_driver);
 }
 
-static void pvr2fb_pci_exit(void)
+static void __exit pvr2fb_pci_exit(void)
 {
         pci_unregister_driver(&pvr2fb_pci_driver);
 }
 #endif /* CONFIG_PCI */
 
-static int __init pvr2_get_param(const struct pvr2_params *p, const char *s,
+static int __devinit pvr2_get_param(const struct pvr2_params *p, const char *s,
                                    int val, int size)
 {
         int i;
@@ -1021,7 +1017,7 @@ static int __init pvr2_get_param(const struct pvr2_params *p, const char *s,
  */
 
 #ifndef MODULE
-int __init pvr2fb_setup(char *options)
+static int __init pvr2fb_setup(char *options)
 {
         char *this_opt;
         char cable_arg[80];
@@ -1061,7 +1057,7 @@ static struct pvr2_board {
         int (*init)(void);
         void (*exit)(void);
         char name[16];
-} board_list[] = {
+} board_driver[] = {
 #ifdef CONFIG_SH_DREAMCAST
         { pvr2fb_dc_init, pvr2fb_dc_exit, "Sega DC PVR2" },
 #endif
@@ -1071,7 +1067,7 @@ static struct pvr2_board {
         { 0, },
 };
 
-int __init pvr2fb_init(void)
+static int __init pvr2fb_init(void)
 {
         int i, ret = -ENODEV;
         int size;
@@ -1095,8 +1091,8 @@ int __init pvr2fb_init(void)
 
         currentpar = (struct pvr2fb_par *)(fb_info + 1);
 
-        for (i = 0; i < ARRAY_SIZE(board_list); i++) {
-                struct pvr2_board *pvr_board = board_list + i;
+        for (i = 0; i < ARRAY_SIZE(board_driver); i++) {
+                struct pvr2_board *pvr_board = board_driver + i;
 
                 if (!pvr_board->init)
                         continue;
@@ -1118,13 +1114,13 @@ static void __exit pvr2fb_exit(void)
 {
         int i;
 
-        for (i = 0; i < ARRAY_SIZE(board_list); i++) {
-                struct pvr2_board *pvr_board = board_list + i;
+        for (i = 0; i < ARRAY_SIZE(board_driver); i++) {
+                struct pvr2_board *pvr_board = board_driver + i;
 
                 if (pvr_board->exit)
                         pvr_board->exit();
         }
-                
+
 #ifdef CONFIG_SH_STORE_QUEUES
         sq_unmap(pvr2fb_map);
 #endif
@@ -1139,4 +1135,3 @@ module_exit(pvr2fb_exit);
 MODULE_AUTHOR("Paul Mundt <lethal@linux-sh.org>, M. R. Brown <mrbrown@0xd6.org>");
 MODULE_DESCRIPTION("Framebuffer driver for NEC PowerVR 2 based graphics boards");
 MODULE_LICENSE("GPL");
-
diff --git a/drivers/video/sis/sis_main.c b/drivers/video/sis/sis_main.c
index a30e1e13d8be..93d07ef85276 100644
--- a/drivers/video/sis/sis_main.c
+++ b/drivers/video/sis/sis_main.c
@@ -5789,7 +5789,7 @@ sisfb_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
         ivideo->warncount = 0;
         ivideo->chip_id = pdev->device;
         ivideo->chip_vendor = pdev->vendor;
-        pci_read_config_byte(pdev, PCI_REVISION_ID, &ivideo->revision_id);
+        ivideo->revision_id = pdev->revision;
         ivideo->SiS_Pr.ChipRevision = ivideo->revision_id;
         pci_read_config_word(pdev, PCI_COMMAND, &reg16);
         ivideo->sisvga_enabled = reg16 & 0x01;
diff --git a/drivers/video/sstfb.c b/drivers/video/sstfb.c
index 62fa5500361d..5eff28ce4f4d 100644
--- a/drivers/video/sstfb.c
+++ b/drivers/video/sstfb.c
@@ -1348,7 +1348,7 @@ static int __devinit sstfb_probe(struct pci_dev *pdev,
         f_ddprintk("found device : %s\n", spec->name);
 
         par->dev = pdev;
-        pci_read_config_byte(pdev, PCI_REVISION_ID, &par->revision);
+        par->revision = pdev->revision;
 
         fix->mmio_start = pci_resource_start(pdev,0);
         fix->mmio_len   = 0x400000;
diff --git a/drivers/video/tgafb.c b/drivers/video/tgafb.c
index f0fde6ea7c36..5c0dab628099 100644
--- a/drivers/video/tgafb.c
+++ b/drivers/video/tgafb.c
@@ -1625,8 +1625,7 @@ tgafb_register(struct device *dev)
         par->tga_regs_base = mem_base + TGA_REGS_OFFSET;
         par->tga_type = tga_type;
         if (tga_bus_pci)
-                pci_read_config_byte(to_pci_dev(dev), PCI_REVISION_ID,
-                                     &par->tga_chip_rev);
+                par->tga_chip_rev = (to_pci_dev(dev))->revision;
         if (tga_bus_tc)
                 par->tga_chip_rev = TGA_READ_REG(par, TGA_START_REG) & 0xff;