fsl-diu-fb: fix regression with uninitalized fb_info->mm_lock mutex
authorKrzysztof Helt <krzysztof.h1@wp.pl>
Wed, 8 Jul 2009 19:59:17 +0000 (21:59 +0200)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 8 Jul 2009 21:18:35 +0000 (14:18 -0700)
Remove call to the fsl_diu_set_par before the register_framebuffer().

This fixes a problem with uninitialized the fb_info->mm_lock mutex
introduced by the commit 537a1bf059f " fbdev: add mutex for fb_mmap
locking"

Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Tested-by: "Kai Jiang" <b18973@freescale.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/video/fsl-diu-fb.c

index 0bf2190928d0e712e12174cc120e9021a1f869f8..72d68b3dc478ec4292b6eef69025de3d25a43a52 100644 (file)
@@ -1223,12 +1223,6 @@ static int __devinit install_fb(struct fb_info *info)
                return -EINVAL;
        }
 
-       if (fsl_diu_set_par(info)) {
-               printk(KERN_ERR "fb_set_par failed");
-               fb_dealloc_cmap(&info->cmap);
-               return -EINVAL;
-       }
-
        if (register_framebuffer(info) < 0) {
                printk(KERN_ERR "register_framebuffer failed");
                unmap_video_memory(info);