OMAP: DSS2: VENC: don't call platform_enable/disable() twice
authorJani Nikula <ext-jani.1.nikula@nokia.com>
Wed, 24 Mar 2010 10:59:37 +0000 (11:59 +0100)
committerTomi Valkeinen <tomi.valkeinen@nokia.com>
Tue, 18 May 2010 11:14:30 +0000 (14:14 +0300)
platform_enable/disable() is already called in venc_power_on/off(), so
don't do it again in venc_panel_enable/disable().

Signed-off-by: Jani Nikula <ext-jani.1.nikula@nokia.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
drivers/video/omap2/dss/venc.c

index f0ba573..eff3505 100644 (file)
@@ -479,12 +479,6 @@ static int venc_panel_enable(struct omap_dss_device *dssdev)
                goto err1;
        }
 
-       if (dssdev->platform_enable) {
-               r = dssdev->platform_enable(dssdev);
-               if (r)
-                       goto err2;
-       }
-
        venc_power_on(dssdev);
 
        venc.wss_data = 0;
@@ -494,13 +488,9 @@ static int venc_panel_enable(struct omap_dss_device *dssdev)
        /* wait couple of vsyncs until enabling the LCD */
        msleep(50);
 
-       mutex_unlock(&venc.venc_lock);
-
-       return r;
-err2:
-       venc_power_off(dssdev);
 err1:
        mutex_unlock(&venc.venc_lock);
+
        return r;
 }
 
@@ -524,9 +514,6 @@ static void venc_panel_disable(struct omap_dss_device *dssdev)
        /* wait at least 5 vsyncs after disabling the LCD */
        msleep(100);
 
-       if (dssdev->platform_disable)
-               dssdev->platform_disable(dssdev);
-
        dssdev->state = OMAP_DSS_DISPLAY_DISABLED;
 end:
        mutex_unlock(&venc.venc_lock);