OMAPDSS: APPLY: Don't check manager settings if it is disabled
authorArchit Taneja <archit@ti.com>
Tue, 8 May 2012 12:49:15 +0000 (18:19 +0530)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Wed, 9 May 2012 10:44:07 +0000 (13:44 +0300)
If a manager is disabled, there is no guarantee at any point in time that all
it's parameters are configured. There is always a chance that some more
parameters are yet to be configured by a user of DSS, or by DSS itself.

However, when the manager is enabled, we can be certain that all the parameters
have been configured, as we can't enable a manager with an incomplete
configuration. Therefore, if a manager is disabled, don't check for the validity
of it's parameters or the parameters of the overlays connected to it. Only check
once it is enabled. Add a check in dss_check_settings_low() to achieve the same.

Signed-off-by: Archit Taneja <archit@ti.com>
drivers/video/omap2/dss/apply.c

index 57686f66411c2fd406fbc537bae3ff4e121fc1cc..ad349b3083c1fd0c6659703d517fd8c6d523ad48 100644 (file)
@@ -192,6 +192,9 @@ static int dss_check_settings_low(struct omap_overlay_manager *mgr,
 
        mp = get_mgr_priv(mgr);
 
+       if (!mp->enabled)
+               return 0;
+
        if (applying && mp->user_info_dirty)
                mi = &mp->user_info;
        else