diff options
author | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2011-11-26 07:29:39 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2011-12-02 01:54:58 -0500 |
commit | a6b24f83601530f69ee36ac48527336e3c79bea8 (patch) | |
tree | 125fee0a1626dc0374d41c552d50426f948de387 /drivers/video | |
parent | 82153eda076ee4df434bb1008f24e46590f76dfc (diff) |
OMAPDSS: APPLY: simplify dss_mgr_enable
dss_mgr_enable() has some extra assigns to mp->enabled, which can be
removed.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/omap2/dss/apply.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/video/omap2/dss/apply.c b/drivers/video/omap2/dss/apply.c index c38c015d60e2..ef13fd65fcc1 100644 --- a/drivers/video/omap2/dss/apply.c +++ b/drivers/video/omap2/dss/apply.c | |||
@@ -1025,16 +1025,14 @@ int dss_mgr_enable(struct omap_overlay_manager *mgr) | |||
1025 | spin_lock_irqsave(&data_lock, flags); | 1025 | spin_lock_irqsave(&data_lock, flags); |
1026 | 1026 | ||
1027 | mp->enabled = true; | 1027 | mp->enabled = true; |
1028 | |||
1028 | r = dss_check_settings(mgr, mgr->device); | 1029 | r = dss_check_settings(mgr, mgr->device); |
1029 | mp->enabled = false; | ||
1030 | if (r) { | 1030 | if (r) { |
1031 | DSSERR("failed to enable manager %d: check_settings failed\n", | 1031 | DSSERR("failed to enable manager %d: check_settings failed\n", |
1032 | mgr->id); | 1032 | mgr->id); |
1033 | goto err; | 1033 | goto err; |
1034 | } | 1034 | } |
1035 | 1035 | ||
1036 | mp->enabled = true; | ||
1037 | |||
1038 | dss_mgr_setup_fifos(mgr); | 1036 | dss_mgr_setup_fifos(mgr); |
1039 | 1037 | ||
1040 | dss_write_regs(); | 1038 | dss_write_regs(); |
@@ -1054,6 +1052,7 @@ out: | |||
1054 | return 0; | 1052 | return 0; |
1055 | 1053 | ||
1056 | err: | 1054 | err: |
1055 | mp->enabled = false; | ||
1057 | spin_unlock_irqrestore(&data_lock, flags); | 1056 | spin_unlock_irqrestore(&data_lock, flags); |
1058 | mutex_unlock(&apply_lock); | 1057 | mutex_unlock(&apply_lock); |
1059 | return r; | 1058 | return r; |