fbdev: i810: use appopriate log interface dev_info

There were many printk log interfaces which do no had
any KERN_INFO with them and they can be replaced with
dev_info which will allow better log level handling
making messages clear and manageable.

No functional changes to the driver behavior are introduced.
Only the logging method has been replaced as per modern
kernel coding guidelines.

Signed-off-by: Shi Hao <i.shihao.999@gmail.com>
Signed-off-by: Helge Deller <deller@gmx.de>

diff --git a/drivers/video/fbdev/i810/i810_main.c b/drivers/video/fbdev/i810/i810_main.c
index d73a795fe1bef3f750abb63cff79f5b2bc7a5c5c..10b914a24114cee4f03c665c2ffd6eea36f54ec8 100644 (file)
--- a/drivers/video/fbdev/i810/i810_main.c
+++ b/drivers/video/fbdev/i810/i810_main.c
@@ -1012,7 +1012,7 @@ static int i810_check_params(struct fb_var_screeninfo *var,
                                                      var->bits_per_pixel);
                        vidmem = line_length * info->var.yres;
                        if (vxres < var->xres) {
-                               printk("i810fb: required video memory, "
+                               dev_info(&par->dev->dev, "required video memory, "
                                       "%d bytes, for %dx%d-%d (virtual) "
                                       "is out of range\n",
                                       vidmem, vxres, vyres,
@@ -1067,9 +1067,9 @@ static int i810_check_params(struct fb_var_screeninfo *var,
                                |(info->monspecs.hfmax-HFMAX)
                                |(info->monspecs.vfmin-VFMIN)
                                |(info->monspecs.vfmax-VFMAX);
-                       printk("i810fb: invalid video mode%s\n",
-                              default_sync ? "" : ". Specifying "
-                              "vsyncN/hsyncN parameters may help");
+                       dev_err(&par->dev->dev, "invalid video mode%s\n",
+                               default_sync ? "" : ". Specifying "
+                               "vsyncN/hsyncN parameters may help");
                        retval = -EINVAL;
                }
        }
@@ -1674,19 +1674,19 @@ static int i810_alloc_agp_mem(struct fb_info *info)
        size = par->fb.size + par->iring.size;
 
        if (!(bridge = agp_backend_acquire(par->dev))) {
-               printk("i810fb_alloc_fbmem: cannot acquire agpgart\n");
+               dev_warn(&par->dev->dev, "cannot acquire agpgart\n");
                return -ENODEV;
        }
        if (!(par->i810_gtt.i810_fb_memory =
              agp_allocate_memory(bridge, size >> 12, AGP_NORMAL_MEMORY))) {
-               printk("i810fb_alloc_fbmem: can't allocate framebuffer "
+               dev_warn(&par->dev->dev, "can't allocate framebuffer "
                       "memory\n");
                agp_backend_release(bridge);
                return -ENOMEM;
        }
        if (agp_bind_memory(par->i810_gtt.i810_fb_memory,
                            par->fb.offset)) {
-               printk("i810fb_alloc_fbmem: can't bind framebuffer memory\n");
+               dev_warn(&par->dev->dev, "can't bind framebuffer memory\n");
                agp_backend_release(bridge);
                return -EBUSY;
        }
@@ -1694,14 +1694,14 @@ static int i810_alloc_agp_mem(struct fb_info *info)
        if (!(par->i810_gtt.i810_cursor_memory =
              agp_allocate_memory(bridge, par->cursor_heap.size >> 12,
                                  AGP_PHYSICAL_MEMORY))) {
-               printk("i810fb_alloc_cursormem:  can't allocate "
+               dev_warn(&par->dev->dev, "can't allocate "
                       "cursor memory\n");
                agp_backend_release(bridge);
                return -ENOMEM;
        }
        if (agp_bind_memory(par->i810_gtt.i810_cursor_memory,
                            par->cursor_heap.offset)) {
-               printk("i810fb_alloc_cursormem: cannot bind cursor memory\n");
+               dev_warn(&par->dev->dev, "cannot bind cursor memory\n");
                agp_backend_release(bridge);
                return -EBUSY;
        }
@@ -1844,7 +1844,7 @@ static int i810_allocate_pci_resource(struct i810fb_par *par,
        int err;
 
        if ((err = pci_enable_device(par->dev))) {
-               printk("i810fb_init: cannot enable device\n");
+               dev_err(&par->dev->dev, "cannot enable device\n");
                return err;
        }
        par->res_flags |= PCI_DEVICE_ENABLED;
@@ -1859,14 +1859,14 @@ static int i810_allocate_pci_resource(struct i810fb_par *par,
                par->mmio_start_phys = pci_resource_start(par->dev, 0);
        }
        if (!par->aperture.size) {
-               printk("i810fb_init: device is disabled\n");
+               dev_warn(&par->dev->dev, "device is disabled\n");
                return -ENOMEM;
        }
 
        if (!request_mem_region(par->aperture.physical,
                                par->aperture.size,
                                i810_pci_list[entry->driver_data])) {
-               printk("i810fb_init: cannot request framebuffer region\n");
+               dev_warn(&par->dev->dev, "cannot request framebuffer region\n");
                return -ENODEV;
        }
        par->res_flags |= FRAMEBUFFER_REQ;
@@ -1874,14 +1874,14 @@ static int i810_allocate_pci_resource(struct i810fb_par *par,
        par->aperture.virtual = ioremap_wc(par->aperture.physical,
                                           par->aperture.size);
        if (!par->aperture.virtual) {
-               printk("i810fb_init: cannot remap framebuffer region\n");
+               dev_warn(&par->dev->dev, "cannot remap framebuffer region\n");
                return -ENODEV;
        }
 
        if (!request_mem_region(par->mmio_start_phys,
                                MMIO_SIZE,
                                i810_pci_list[entry->driver_data])) {
-               printk("i810fb_init: cannot request mmio region\n");
+               dev_warn(&par->dev->dev, "cannot request mmio region\n");
                return -ENODEV;
        }
        par->res_flags |= MMIO_REQ;
@@ -1889,7 +1889,7 @@ static int i810_allocate_pci_resource(struct i810fb_par *par,
        par->mmio_start_virtual = ioremap(par->mmio_start_phys,
                                                  MMIO_SIZE);
        if (!par->mmio_start_virtual) {
-               printk("i810fb_init: cannot remap mmio region\n");
+               dev_warn(&par->dev->dev, "cannot remap mmio region\n");
                return -ENODEV;
        }
 
@@ -1921,12 +1921,12 @@ static void i810fb_find_init_mode(struct fb_info *info)
        }
 
        if (!err)
-               printk("i810fb_init_pci: DDC probe successful\n");
+               dev_info(&par->dev->dev, "DDC probe successful\n");
 
        fb_edid_to_monspecs(par->edid, specs);
 
        if (specs->modedb == NULL)
-               printk("i810fb_init_pci: Unable to get Mode Database\n");
+               dev_info(&par->dev->dev, "Unable to get Mode Database\n");
 
        fb_videomode_to_modelist(specs->modedb, specs->modedb_len,
                                 &info->modelist);
@@ -2072,7 +2072,7 @@ static int i810fb_init_pci(struct pci_dev *dev,
 
        if (err < 0) {
                i810fb_release_resource(info, par);
-               printk("i810fb_init: cannot register framebuffer device\n");
+               dev_warn(&par->dev->dev, "cannot register framebuffer device\n");
                return err;
        }
 
@@ -2084,10 +2084,10 @@ static int i810fb_init_pci(struct pci_dev *dev,
        vfreq = hfreq/(info->var.yres + info->var.upper_margin +
                       info->var.vsync_len + info->var.lower_margin);
 
-       printk("I810FB: fb%d         : %s v%d.%d.%d%s\n"
-              "I810FB: Video RAM   : %dK\n"
-              "I810FB: Monitor     : H: %d-%d KHz V: %d-%d Hz\n"
-              "I810FB: Mode        : %dx%d-%dbpp@%dHz\n",
+       dev_info(&par->dev->dev, "fb%d         : %s v%d.%d.%d%s\n"
+               "Video RAM   : %dK\n"
+               "Monitor     : H: %d-%d KHz V: %d-%d Hz\n"
+               "Mode        : %dx%d-%dbpp@%dHz\n",
               info->node,
               i810_pci_list[entry->driver_data],
               VERSION_MAJOR, VERSION_MINOR, VERSION_TEENIE, BRANCH_VERSION,
@@ -2137,7 +2137,7 @@ static void i810fb_remove_pci(struct pci_dev *dev)
 
        unregister_framebuffer(info);
        i810fb_release_resource(info, par);
-       printk("cleanup_module:  unloaded i810 framebuffer device\n");
+       dev_info(&par->dev->dev, "unloaded i810 framebuffer device\n");
 }
 
 #ifndef MODULE