sm501fb: fix regression with uninitalized fb_info->mm_lock mutex
authorKrzysztof Helt <krzysztof.h1@wp.pl>
Thu, 9 Jul 2009 18:15:44 +0000 (20:15 +0200)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 10 Jul 2009 03:44:44 +0000 (20:44 -0700)
Remove call to the fb_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>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/video/sm501fb.c

index 16d4f4c7d52b5b02e6fcff002b8c0d03160b371c..924d79462780c5eb34ee79f5dfa0fb61dec193f8 100644 (file)
@@ -1540,9 +1540,6 @@ static int sm501fb_init_fb(struct fb_info *fb,
        if (ret)
                dev_err(info->dev, "check_var() failed on initial setup?\n");
 
-       /* ensure we've activated our new configuration */
-       (fb->fbops->fb_set_par)(fb);
-
        return 0;
 }