OMAP: DSS2: fix lock_fb_info() and omapfb_lock() locking order

Framebuffer ioctl processing forces lock_fb_info() -> omapfb_lock()
locking order. Follow that order to avoid possible circular locking
dependency, detected by lockdep.

Signed-off-by: Jani Nikula <ext-jani.1.nikula@nokia.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>

diff --git a/drivers/video/omap2/omapfb/omapfb-ioctl.c b/drivers/video/omap2/omapfb/omapfb-ioctl.c
index 1ffa760b8545c4fdca02915e597dfad70d9baeed..2c0f01c44aabb7e837897661180eb0662ad69733 100644 (file)
--- a/drivers/video/omap2/omapfb/omapfb-ioctl.c
+++ b/drivers/video/omap2/omapfb/omapfb-ioctl.c
@@ -183,13 +183,13 @@ int omapfb_update_window(struct fb_info *fbi,
        struct omapfb2_device *fbdev = ofbi->fbdev;
        int r;
 
-       omapfb_lock(fbdev);
        lock_fb_info(fbi);
+       omapfb_lock(fbdev);
 
        r = omapfb_update_window_nolock(fbi, x, y, w, h);
 
-       unlock_fb_info(fbi);
        omapfb_unlock(fbdev);
+       unlock_fb_info(fbi);
 
        return r;
 }

diff --git a/drivers/video/omap2/omapfb/omapfb-sysfs.c b/drivers/video/omap2/omapfb/omapfb-sysfs.c
index 62bb88f5c192ad3160d57c7c699fb5421010b55a..e104450179859d53cdcda63cedf8406e2a18ef78 100644 (file)
--- a/drivers/video/omap2/omapfb/omapfb-sysfs.c
+++ b/drivers/video/omap2/omapfb/omapfb-sysfs.c
@@ -137,8 +137,8 @@ static ssize_t show_overlays(struct device *dev,
        ssize_t l = 0;
        int t;
 
-       omapfb_lock(fbdev);
        lock_fb_info(fbi);
+       omapfb_lock(fbdev);
 
        for (t = 0; t < ofbi->num_overlays; t++) {
                struct omap_overlay *ovl = ofbi->overlays[t];
@@ -154,8 +154,8 @@ static ssize_t show_overlays(struct device *dev,
 
        l += snprintf(buf + l, PAGE_SIZE - l, "\n");
 
-       unlock_fb_info(fbi);
        omapfb_unlock(fbdev);
+       unlock_fb_info(fbi);
 
        return l;
 }
@@ -195,8 +195,8 @@ static ssize_t store_overlays(struct device *dev, struct device_attribute *attr,
        if (buf[len - 1] == '\n')
                len = len - 1;
 
-       omapfb_lock(fbdev);
        lock_fb_info(fbi);
+       omapfb_lock(fbdev);
 
        if (len > 0) {
                char *p = (char *)buf;
@@ -303,8 +303,8 @@ static ssize_t store_overlays(struct device *dev, struct device_attribute *attr,
 
        r = count;
 out:
-       unlock_fb_info(fbi);
        omapfb_unlock(fbdev);
+       unlock_fb_info(fbi);
 
        return r;
 }