pci: use pci_ioremap_bar() in drivers/video

Use the newly introduced pci_ioremap_bar() function in drivers/video.
pci_ioremap_bar() just takes a pci device and a bar number, with the goal
of making it really hard to get wrong, while also having a central place
to stick sanity checks.

Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Cc: Krzysztof Helt <krzysztof.h1@poczta.fm>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

3 files changed, 7 insertions, 13 deletions

diff --git a/drivers/video/geode/gx1fb_core.c b/drivers/video/geode/gx1fb_core.c
index bb20a2289760..751e491ca8c8 100644
--- a/drivers/video/geode/gx1fb_core.c
+++ b/drivers/video/geode/gx1fb_core.c
@@ -217,8 +217,7 @@ static int __init gx1fb_map_video_memory(struct fb_info *info, struct pci_dev *d
         ret = pci_request_region(dev, 0, "gx1fb (video)");
         if (ret < 0)
                 return ret;
-        par->vid_regs = ioremap(pci_resource_start(dev, 0),
-                                pci_resource_len(dev, 0));
+        par->vid_regs = pci_ioremap_bar(dev, 0);
         if (!par->vid_regs)
                 return -ENOMEM;
 
diff --git a/drivers/video/geode/gxfb_core.c b/drivers/video/geode/gxfb_core.c
index de2b8f9876a5..484118926318 100644
--- a/drivers/video/geode/gxfb_core.c
+++ b/drivers/video/geode/gxfb_core.c
@@ -242,23 +242,21 @@ static int __init gxfb_map_video_memory(struct fb_info *info, struct pci_dev *de
         ret = pci_request_region(dev, 3, "gxfb (video processor)");
         if (ret < 0)
                 return ret;
-        par->vid_regs = ioremap(pci_resource_start(dev, 3),
-                                pci_resource_len(dev, 3));
+        par->vid_regs = pci_ioremap_bar(dev, 3);
         if (!par->vid_regs)
                 return -ENOMEM;
 
         ret = pci_request_region(dev, 2, "gxfb (display controller)");
         if (ret < 0)
                 return ret;
-        par->dc_regs = ioremap(pci_resource_start(dev, 2), pci_resource_len(dev, 2));
+        par->dc_regs = pci_ioremap_bar(dev, 2);
         if (!par->dc_regs)
                 return -ENOMEM;
 
         ret = pci_request_region(dev, 1, "gxfb (graphics processor)");
         if (ret < 0)
                 return ret;
-        par->gp_regs = ioremap(pci_resource_start(dev, 1),
-        pci_resource_len(dev, 1));
+        par->gp_regs = pci_ioremap_bar(dev, 1);
 
         if (!par->gp_regs)
                 return -ENOMEM;
diff --git a/drivers/video/geode/lxfb_core.c b/drivers/video/geode/lxfb_core.c
index 2cd9b74d2225..b965ecdbc604 100644
--- a/drivers/video/geode/lxfb_core.c
+++ b/drivers/video/geode/lxfb_core.c
@@ -379,20 +379,17 @@ static int __init lxfb_map_video_memory(struct fb_info *info,
         if (info->screen_base == NULL)
                 return ret;
 
-        par->gp_regs = ioremap(pci_resource_start(dev, 1),
-                                pci_resource_len(dev, 1));
+        par->gp_regs = pci_ioremap_bar(dev, 1);
 
         if (par->gp_regs == NULL)
                 return ret;
 
-        par->dc_regs = ioremap(pci_resource_start(dev, 2),
-                               pci_resource_len(dev, 2));
+        par->dc_regs = pci_ioremap_bar(dev, 2);
 
         if (par->dc_regs == NULL)
                 return ret;
 
-        par->vp_regs = ioremap(pci_resource_start(dev, 3),
-                               pci_resource_len(dev, 3));
+        par->vp_regs = pci_ioremap_bar(dev, 3);
 
         if (par->vp_regs == NULL)
                 return ret;