2011-11-04 14:48:54 +07:00
|
|
|
/*
|
|
|
|
* Copyright (C) 2011 Texas Instruments
|
|
|
|
* Author: Tomi Valkeinen <tomi.valkeinen@ti.com>
|
|
|
|
*
|
|
|
|
* This program is free software; you can redistribute it and/or modify it
|
|
|
|
* under the terms of the GNU General Public License version 2 as published by
|
|
|
|
* the Free Software Foundation.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope that it will be useful, but WITHOUT
|
|
|
|
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
|
|
|
|
* more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License along with
|
|
|
|
* this program. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#define DSS_SUBSYS_NAME "APPLY"
|
|
|
|
|
|
|
|
#include <linux/kernel.h>
|
|
|
|
#include <linux/slab.h>
|
|
|
|
#include <linux/spinlock.h>
|
|
|
|
#include <linux/jiffies.h>
|
|
|
|
|
|
|
|
#include <video/omapdss.h>
|
|
|
|
|
|
|
|
#include "dss.h"
|
|
|
|
#include "dss_features.h"
|
|
|
|
|
|
|
|
/*
|
|
|
|
* We have 4 levels of cache for the dispc settings. First two are in SW and
|
|
|
|
* the latter two in HW.
|
|
|
|
*
|
|
|
|
* +--------------------+
|
|
|
|
* |overlay/manager_info|
|
|
|
|
* +--------------------+
|
|
|
|
* v
|
|
|
|
* apply()
|
|
|
|
* v
|
|
|
|
* +--------------------+
|
2011-11-15 17:04:43 +07:00
|
|
|
* | info |
|
2011-11-04 14:48:54 +07:00
|
|
|
* +--------------------+
|
|
|
|
* v
|
2011-11-15 16:47:39 +07:00
|
|
|
* write_regs()
|
2011-11-04 14:48:54 +07:00
|
|
|
* v
|
|
|
|
* +--------------------+
|
|
|
|
* | shadow registers |
|
|
|
|
* +--------------------+
|
|
|
|
* v
|
|
|
|
* VFP or lcd/digit_enable
|
|
|
|
* v
|
|
|
|
* +--------------------+
|
|
|
|
* | registers |
|
|
|
|
* +--------------------+
|
|
|
|
*/
|
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data {
|
2011-11-04 14:48:54 +07:00
|
|
|
/* If true, cache changed, but not written to shadow registers. Set
|
|
|
|
* in apply(), cleared when registers written. */
|
|
|
|
bool dirty;
|
|
|
|
/* If true, shadow registers contain changed values not yet in real
|
|
|
|
* registers. Set when writing to shadow registers, cleared at
|
|
|
|
* VSYNC/EVSYNC */
|
|
|
|
bool shadow_dirty;
|
|
|
|
|
|
|
|
struct omap_overlay_info info;
|
|
|
|
|
|
|
|
enum omap_channel channel;
|
|
|
|
|
|
|
|
u32 fifo_low;
|
|
|
|
u32 fifo_high;
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
|
|
|
|
bool extra_info_dirty;
|
|
|
|
bool shadow_extra_info_dirty;
|
|
|
|
|
|
|
|
bool enabled;
|
|
|
|
|
2011-11-04 14:48:54 +07:00
|
|
|
};
|
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data {
|
2011-11-04 14:48:54 +07:00
|
|
|
/* If true, cache changed, but not written to shadow registers. Set
|
|
|
|
* in apply(), cleared when registers written. */
|
|
|
|
bool dirty;
|
|
|
|
/* If true, shadow registers contain changed values not yet in real
|
|
|
|
* registers. Set when writing to shadow registers, cleared at
|
|
|
|
* VSYNC/EVSYNC */
|
|
|
|
bool shadow_dirty;
|
|
|
|
|
|
|
|
struct omap_overlay_manager_info info;
|
|
|
|
|
|
|
|
bool manual_update;
|
|
|
|
bool do_manual_update;
|
2011-11-15 19:43:53 +07:00
|
|
|
|
2011-11-15 20:04:25 +07:00
|
|
|
/* If true, GO bit is up and shadow registers cannot be written.
|
|
|
|
* Never true for manual update displays */
|
|
|
|
bool busy;
|
|
|
|
|
2011-11-15 19:43:53 +07:00
|
|
|
/* If true, a display is enabled using this manager */
|
|
|
|
bool enabled;
|
2011-11-04 14:48:54 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
static struct {
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data ovl_priv_data_array[MAX_DSS_OVERLAYS];
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data mgr_priv_data_array[MAX_DSS_MANAGERS];
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
bool irq_enabled;
|
2011-11-15 17:04:43 +07:00
|
|
|
} dss_data;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:04:43 +07:00
|
|
|
/* protects dss_data */
|
2011-11-15 17:04:10 +07:00
|
|
|
static spinlock_t data_lock;
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
/* lock for blocking functions */
|
|
|
|
static DEFINE_MUTEX(apply_lock);
|
2011-11-15 17:04:10 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
static struct ovl_priv_data *get_ovl_priv(struct omap_overlay *ovl)
|
|
|
|
{
|
2011-11-15 17:04:43 +07:00
|
|
|
return &dss_data.ovl_priv_data_array[ovl->id];
|
2011-11-15 16:56:57 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
static struct mgr_priv_data *get_mgr_priv(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
2011-11-15 17:04:43 +07:00
|
|
|
return &dss_data.mgr_priv_data_array[mgr->id];
|
2011-11-15 17:02:03 +07:00
|
|
|
}
|
|
|
|
|
2011-11-04 14:48:54 +07:00
|
|
|
void dss_apply_init(void)
|
|
|
|
{
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_lock_init(&data_lock);
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static bool ovl_manual_update(struct omap_overlay *ovl)
|
|
|
|
{
|
|
|
|
return ovl->manager->device->caps & OMAP_DSS_DISPLAY_CAP_MANUAL_UPDATE;
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool mgr_manual_update(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
|
|
|
return mgr->device->caps & OMAP_DSS_DISPLAY_CAP_MANUAL_UPDATE;
|
|
|
|
}
|
|
|
|
|
|
|
|
int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
|
|
|
unsigned long timeout = msecs_to_jiffies(500);
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data *mp;
|
2011-11-04 14:48:54 +07:00
|
|
|
u32 irq;
|
|
|
|
int r;
|
|
|
|
int i;
|
|
|
|
struct omap_dss_device *dssdev = mgr->device;
|
|
|
|
|
|
|
|
if (!dssdev || dssdev->state != OMAP_DSS_DISPLAY_ACTIVE)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
if (mgr_manual_update(mgr))
|
|
|
|
return 0;
|
|
|
|
|
2011-11-15 16:20:13 +07:00
|
|
|
irq = dispc_mgr_get_vsync_irq(mgr->id);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp = get_mgr_priv(mgr);
|
2011-11-04 14:48:54 +07:00
|
|
|
i = 0;
|
|
|
|
while (1) {
|
|
|
|
unsigned long flags;
|
|
|
|
bool shadow_dirty, dirty;
|
|
|
|
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
2011-11-15 17:02:03 +07:00
|
|
|
dirty = mp->dirty;
|
|
|
|
shadow_dirty = mp->shadow_dirty;
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
if (!dirty && !shadow_dirty) {
|
|
|
|
r = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* 4 iterations is the worst case:
|
|
|
|
* 1 - initial iteration, dirty = true (between VFP and VSYNC)
|
|
|
|
* 2 - first VSYNC, dirty = true
|
|
|
|
* 3 - dirty = false, shadow_dirty = true
|
|
|
|
* 4 - shadow_dirty = false */
|
|
|
|
if (i++ == 3) {
|
|
|
|
DSSERR("mgr(%d)->wait_for_go() not finishing\n",
|
|
|
|
mgr->id);
|
|
|
|
r = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
r = omap_dispc_wait_for_irq_interruptible_timeout(irq, timeout);
|
|
|
|
if (r == -ERESTARTSYS)
|
|
|
|
break;
|
|
|
|
|
|
|
|
if (r) {
|
|
|
|
DSSERR("mgr(%d)->wait_for_go() timeout\n", mgr->id);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
|
|
|
int dss_mgr_wait_for_go_ovl(struct omap_overlay *ovl)
|
|
|
|
{
|
|
|
|
unsigned long timeout = msecs_to_jiffies(500);
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-04 14:48:54 +07:00
|
|
|
struct omap_dss_device *dssdev;
|
|
|
|
u32 irq;
|
|
|
|
int r;
|
|
|
|
int i;
|
|
|
|
|
|
|
|
if (!ovl->manager)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
dssdev = ovl->manager->device;
|
|
|
|
|
|
|
|
if (!dssdev || dssdev->state != OMAP_DSS_DISPLAY_ACTIVE)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
if (ovl_manual_update(ovl))
|
|
|
|
return 0;
|
|
|
|
|
2011-11-15 16:20:13 +07:00
|
|
|
irq = dispc_mgr_get_vsync_irq(ovl->manager->id);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
op = get_ovl_priv(ovl);
|
2011-11-04 14:48:54 +07:00
|
|
|
i = 0;
|
|
|
|
while (1) {
|
|
|
|
unsigned long flags;
|
|
|
|
bool shadow_dirty, dirty;
|
|
|
|
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
2011-11-15 16:56:57 +07:00
|
|
|
dirty = op->dirty;
|
|
|
|
shadow_dirty = op->shadow_dirty;
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
if (!dirty && !shadow_dirty) {
|
|
|
|
r = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* 4 iterations is the worst case:
|
|
|
|
* 1 - initial iteration, dirty = true (between VFP and VSYNC)
|
|
|
|
* 2 - first VSYNC, dirty = true
|
|
|
|
* 3 - dirty = false, shadow_dirty = true
|
|
|
|
* 4 - shadow_dirty = false */
|
|
|
|
if (i++ == 3) {
|
|
|
|
DSSERR("ovl(%d)->wait_for_go() not finishing\n",
|
|
|
|
ovl->id);
|
|
|
|
r = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
r = omap_dispc_wait_for_irq_interruptible_timeout(irq, timeout);
|
|
|
|
if (r == -ERESTARTSYS)
|
|
|
|
break;
|
|
|
|
|
|
|
|
if (r) {
|
|
|
|
DSSERR("ovl(%d)->wait_for_go() timeout\n", ovl->id);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
static int dss_ovl_write_regs(struct omap_overlay *ovl)
|
2011-11-04 14:48:54 +07:00
|
|
|
{
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-04 14:48:54 +07:00
|
|
|
struct omap_overlay_info *oi;
|
|
|
|
bool ilace, replication;
|
|
|
|
int r;
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
DSSDBGF("%d", ovl->id);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
op = get_ovl_priv(ovl);
|
|
|
|
oi = &op->info;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
if (!op->enabled)
|
2011-11-04 14:48:54 +07:00
|
|
|
return 0;
|
|
|
|
|
|
|
|
replication = dss_use_replication(ovl->manager->device, oi->color_mode);
|
|
|
|
|
|
|
|
ilace = ovl->manager->device->type == OMAP_DISPLAY_TYPE_VENC;
|
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
dispc_ovl_set_channel_out(ovl->id, op->channel);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
r = dispc_ovl_setup(ovl->id, oi, ilace, replication);
|
2011-11-04 14:48:54 +07:00
|
|
|
if (r) {
|
|
|
|
/* this shouldn't happen */
|
2011-11-15 16:47:39 +07:00
|
|
|
DSSERR("dispc_ovl_setup failed for ovl %d\n", ovl->id);
|
|
|
|
dispc_ovl_enable(ovl->id, 0);
|
2011-11-04 14:48:54 +07:00
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
dispc_ovl_set_fifo_threshold(ovl->id, op->fifo_low, op->fifo_high);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
static void dss_ovl_write_regs_extra(struct omap_overlay *ovl)
|
|
|
|
{
|
|
|
|
struct ovl_priv_data *op = get_ovl_priv(ovl);
|
|
|
|
|
|
|
|
DSSDBGF("%d", ovl->id);
|
|
|
|
|
|
|
|
/* note: write also when op->enabled == false, so that the ovl gets
|
|
|
|
* disabled */
|
|
|
|
|
|
|
|
dispc_ovl_enable(ovl->id, op->enabled);
|
|
|
|
}
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
static void dss_mgr_write_regs(struct omap_overlay_manager *mgr)
|
2011-11-04 14:48:54 +07:00
|
|
|
{
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data *mp;
|
2011-11-04 14:48:54 +07:00
|
|
|
struct omap_overlay_manager_info *mi;
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
DSSDBGF("%d", mgr->id);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp = get_mgr_priv(mgr);
|
|
|
|
mi = &mp->info;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
dispc_mgr_setup(mgr->id, mi);
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
/* dss_write_regs() tries to write values from cache to shadow registers.
|
2011-11-04 14:48:54 +07:00
|
|
|
* It writes only to those managers/overlays that are not busy.
|
|
|
|
* returns 0 if everything could be written to shadow registers.
|
|
|
|
* returns 1 if not everything could be written to shadow registers. */
|
2011-11-15 16:47:39 +07:00
|
|
|
static int dss_write_regs(void)
|
2011-11-04 14:48:54 +07:00
|
|
|
{
|
2011-11-15 16:47:39 +07:00
|
|
|
struct omap_overlay *ovl;
|
|
|
|
struct omap_overlay_manager *mgr;
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data *mp;
|
2011-11-04 14:48:54 +07:00
|
|
|
const int num_ovls = dss_feat_get_num_ovls();
|
|
|
|
const int num_mgrs = dss_feat_get_num_mgrs();
|
|
|
|
int i;
|
|
|
|
int r;
|
2011-11-15 20:04:25 +07:00
|
|
|
bool mgr_go[MAX_DSS_MANAGERS] = { false };
|
2011-11-04 14:48:54 +07:00
|
|
|
bool busy;
|
|
|
|
|
|
|
|
r = 0;
|
|
|
|
busy = false;
|
|
|
|
|
|
|
|
/* Commit overlay settings */
|
|
|
|
for (i = 0; i < num_ovls; ++i) {
|
2011-11-15 16:47:39 +07:00
|
|
|
ovl = omap_dss_get_overlay(i);
|
2011-11-15 16:56:57 +07:00
|
|
|
op = get_ovl_priv(ovl);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
if (!op->dirty)
|
2011-11-04 14:48:54 +07:00
|
|
|
continue;
|
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp = get_mgr_priv(ovl->manager);
|
|
|
|
|
|
|
|
if (mp->manual_update && !mp->do_manual_update)
|
2011-11-04 14:48:54 +07:00
|
|
|
continue;
|
|
|
|
|
2011-11-15 20:04:25 +07:00
|
|
|
if (mp->busy) {
|
2011-11-04 14:48:54 +07:00
|
|
|
busy = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
r = dss_ovl_write_regs(ovl);
|
2011-11-04 14:48:54 +07:00
|
|
|
if (r)
|
2011-11-15 16:47:39 +07:00
|
|
|
DSSERR("dss_ovl_write_regs %d failed\n", i);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
op->dirty = false;
|
|
|
|
op->shadow_dirty = true;
|
|
|
|
mgr_go[op->channel] = true;
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
for (i = 0; i < num_ovls; ++i) {
|
|
|
|
ovl = omap_dss_get_overlay(i);
|
|
|
|
op = get_ovl_priv(ovl);
|
|
|
|
|
|
|
|
if (!op->extra_info_dirty)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
mp = get_mgr_priv(ovl->manager);
|
|
|
|
|
|
|
|
if (mp->manual_update && !mp->do_manual_update)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
if (mp->busy) {
|
|
|
|
busy = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
dss_ovl_write_regs_extra(ovl);
|
|
|
|
|
|
|
|
op->extra_info_dirty = false;
|
|
|
|
op->shadow_extra_info_dirty = true;
|
|
|
|
mgr_go[op->channel] = true;
|
|
|
|
}
|
|
|
|
|
2011-11-04 14:48:54 +07:00
|
|
|
/* Commit manager settings */
|
|
|
|
for (i = 0; i < num_mgrs; ++i) {
|
2011-11-15 16:47:39 +07:00
|
|
|
mgr = omap_dss_get_overlay_manager(i);
|
2011-11-15 17:02:03 +07:00
|
|
|
mp = get_mgr_priv(mgr);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
if (!mp->dirty)
|
2011-11-04 14:48:54 +07:00
|
|
|
continue;
|
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
if (mp->manual_update && !mp->do_manual_update)
|
2011-11-04 14:48:54 +07:00
|
|
|
continue;
|
|
|
|
|
2011-11-15 20:04:25 +07:00
|
|
|
if (mp->busy) {
|
2011-11-04 14:48:54 +07:00
|
|
|
busy = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
dss_mgr_write_regs(mgr);
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->dirty = false;
|
|
|
|
mp->shadow_dirty = true;
|
2011-11-04 14:48:54 +07:00
|
|
|
mgr_go[i] = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* set GO */
|
|
|
|
for (i = 0; i < num_mgrs; ++i) {
|
2011-11-15 17:02:03 +07:00
|
|
|
mgr = omap_dss_get_overlay_manager(i);
|
|
|
|
mp = get_mgr_priv(mgr);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
if (!mgr_go[i])
|
|
|
|
continue;
|
|
|
|
|
|
|
|
/* We don't need GO with manual update display. LCD iface will
|
|
|
|
* always be turned off after frame, and new settings will be
|
|
|
|
* taken in to use at next update */
|
2011-11-15 20:04:25 +07:00
|
|
|
if (!mp->manual_update) {
|
|
|
|
mp->busy = true;
|
2011-11-04 14:48:54 +07:00
|
|
|
dispc_mgr_go(i);
|
2011-11-15 20:04:25 +07:00
|
|
|
}
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
if (busy)
|
|
|
|
r = 1;
|
|
|
|
else
|
|
|
|
r = 0;
|
|
|
|
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
|
|
|
void dss_mgr_start_update(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data *mp = get_mgr_priv(mgr);
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-05 15:59:59 +07:00
|
|
|
struct omap_overlay *ovl;
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->do_manual_update = true;
|
2011-11-15 16:47:39 +07:00
|
|
|
dss_write_regs();
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->do_manual_update = false;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-05 15:59:59 +07:00
|
|
|
list_for_each_entry(ovl, &mgr->overlays, list) {
|
2011-11-15 16:56:57 +07:00
|
|
|
op = get_ovl_priv(ovl);
|
|
|
|
op->shadow_dirty = false;
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
op->shadow_extra_info_dirty = false;
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->shadow_dirty = false;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-04 15:22:46 +07:00
|
|
|
dispc_mgr_enable(mgr->id, true);
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 16:18:12 +07:00
|
|
|
static void dss_apply_irq_handler(void *data, u32 mask);
|
|
|
|
|
|
|
|
static void dss_register_vsync_isr(void)
|
|
|
|
{
|
2011-11-15 16:20:13 +07:00
|
|
|
const int num_mgrs = dss_feat_get_num_mgrs();
|
2011-11-15 16:18:12 +07:00
|
|
|
u32 mask;
|
2011-11-15 16:20:13 +07:00
|
|
|
int r, i;
|
2011-11-15 16:18:12 +07:00
|
|
|
|
2011-11-15 16:20:13 +07:00
|
|
|
mask = 0;
|
|
|
|
for (i = 0; i < num_mgrs; ++i)
|
|
|
|
mask |= dispc_mgr_get_vsync_irq(i);
|
2011-11-15 16:18:12 +07:00
|
|
|
|
|
|
|
r = omap_dispc_register_isr(dss_apply_irq_handler, NULL, mask);
|
|
|
|
WARN_ON(r);
|
|
|
|
|
2011-11-15 17:04:43 +07:00
|
|
|
dss_data.irq_enabled = true;
|
2011-11-15 16:18:12 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static void dss_unregister_vsync_isr(void)
|
|
|
|
{
|
2011-11-15 16:20:13 +07:00
|
|
|
const int num_mgrs = dss_feat_get_num_mgrs();
|
2011-11-15 16:18:12 +07:00
|
|
|
u32 mask;
|
2011-11-15 16:20:13 +07:00
|
|
|
int r, i;
|
2011-11-15 16:18:12 +07:00
|
|
|
|
2011-11-15 16:20:13 +07:00
|
|
|
mask = 0;
|
|
|
|
for (i = 0; i < num_mgrs; ++i)
|
|
|
|
mask |= dispc_mgr_get_vsync_irq(i);
|
2011-11-15 16:18:12 +07:00
|
|
|
|
|
|
|
r = omap_dispc_unregister_isr(dss_apply_irq_handler, NULL, mask);
|
|
|
|
WARN_ON(r);
|
|
|
|
|
2011-11-15 17:04:43 +07:00
|
|
|
dss_data.irq_enabled = false;
|
2011-11-15 16:18:12 +07:00
|
|
|
}
|
|
|
|
|
2011-11-04 14:48:54 +07:00
|
|
|
static void dss_apply_irq_handler(void *data, u32 mask)
|
|
|
|
{
|
2011-11-15 16:56:57 +07:00
|
|
|
struct omap_overlay *ovl;
|
2011-11-15 17:02:03 +07:00
|
|
|
struct omap_overlay_manager *mgr;
|
|
|
|
struct mgr_priv_data *mp;
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-04 14:48:54 +07:00
|
|
|
const int num_ovls = dss_feat_get_num_ovls();
|
|
|
|
const int num_mgrs = dss_feat_get_num_mgrs();
|
|
|
|
int i, r;
|
|
|
|
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_lock(&data_lock);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 20:04:25 +07:00
|
|
|
for (i = 0; i < num_mgrs; i++) {
|
|
|
|
mgr = omap_dss_get_overlay_manager(i);
|
|
|
|
mp = get_mgr_priv(mgr);
|
|
|
|
|
|
|
|
mp->busy = dispc_mgr_go_busy(i);
|
|
|
|
}
|
|
|
|
|
2011-11-04 14:48:54 +07:00
|
|
|
for (i = 0; i < num_ovls; ++i) {
|
2011-11-15 16:56:57 +07:00
|
|
|
ovl = omap_dss_get_overlay(i);
|
|
|
|
op = get_ovl_priv(ovl);
|
2011-11-15 20:04:25 +07:00
|
|
|
|
|
|
|
if (!op->enabled)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
mp = get_mgr_priv(ovl->manager);
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
if (!mp->busy) {
|
2011-11-15 16:56:57 +07:00
|
|
|
op->shadow_dirty = false;
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
op->shadow_extra_info_dirty = false;
|
|
|
|
}
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0; i < num_mgrs; ++i) {
|
2011-11-15 17:02:03 +07:00
|
|
|
mgr = omap_dss_get_overlay_manager(i);
|
|
|
|
mp = get_mgr_priv(mgr);
|
2011-11-15 20:04:25 +07:00
|
|
|
|
|
|
|
if (!mp->busy)
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->shadow_dirty = false;
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
r = dss_write_regs();
|
2011-11-04 14:48:54 +07:00
|
|
|
if (r == 1)
|
|
|
|
goto end;
|
|
|
|
|
|
|
|
/* re-read busy flags */
|
2011-11-15 20:04:25 +07:00
|
|
|
for (i = 0; i < num_mgrs; i++) {
|
|
|
|
mgr = omap_dss_get_overlay_manager(i);
|
|
|
|
mp = get_mgr_priv(mgr);
|
|
|
|
|
|
|
|
mp->busy = dispc_mgr_go_busy(i);
|
|
|
|
}
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
/* keep running as long as there are busy managers, so that
|
|
|
|
* we can collect overlay-applied information */
|
|
|
|
for (i = 0; i < num_mgrs; ++i) {
|
2011-11-15 20:04:25 +07:00
|
|
|
mgr = omap_dss_get_overlay_manager(i);
|
|
|
|
mp = get_mgr_priv(mgr);
|
|
|
|
|
|
|
|
if (mp->busy)
|
2011-11-04 14:48:54 +07:00
|
|
|
goto end;
|
|
|
|
}
|
|
|
|
|
2011-11-15 16:18:12 +07:00
|
|
|
dss_unregister_vsync_isr();
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
end:
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_unlock(&data_lock);
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 18:37:33 +07:00
|
|
|
static void omap_dss_mgr_apply_ovl(struct omap_overlay *ovl)
|
2011-11-04 14:48:54 +07:00
|
|
|
{
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
op = get_ovl_priv(ovl);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
if (ovl->manager_changed) {
|
|
|
|
ovl->manager_changed = false;
|
|
|
|
ovl->info_dirty = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!ovl->info_dirty)
|
2011-11-15 18:37:33 +07:00
|
|
|
return;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
ovl->info_dirty = false;
|
2011-11-15 16:56:57 +07:00
|
|
|
op->dirty = true;
|
|
|
|
op->info = ovl->info;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
op->channel = ovl->manager->id;
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static void omap_dss_mgr_apply_mgr(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
2011-11-15 17:02:03 +07:00
|
|
|
struct mgr_priv_data *mp;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp = get_mgr_priv(mgr);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
if (mgr->device_changed) {
|
|
|
|
mgr->device_changed = false;
|
|
|
|
mgr->info_dirty = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!mgr->info_dirty)
|
|
|
|
return;
|
|
|
|
|
|
|
|
if (!mgr->device)
|
|
|
|
return;
|
|
|
|
|
|
|
|
mgr->info_dirty = false;
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->dirty = true;
|
|
|
|
mp->info = mgr->info;
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:02:03 +07:00
|
|
|
mp->manual_update = mgr_manual_update(mgr);
|
2011-11-04 14:48:54 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static void omap_dss_mgr_apply_ovl_fifos(struct omap_overlay *ovl)
|
|
|
|
{
|
2011-11-15 16:56:57 +07:00
|
|
|
struct ovl_priv_data *op;
|
2011-11-04 14:48:54 +07:00
|
|
|
struct omap_dss_device *dssdev;
|
|
|
|
u32 size, burst_size;
|
|
|
|
|
2011-11-15 16:56:57 +07:00
|
|
|
op = get_ovl_priv(ovl);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
dssdev = ovl->manager->device;
|
|
|
|
|
|
|
|
size = dispc_ovl_get_fifo_size(ovl->id);
|
|
|
|
|
|
|
|
burst_size = dispc_ovl_get_burst_size(ovl->id);
|
|
|
|
|
|
|
|
switch (dssdev->type) {
|
|
|
|
case OMAP_DISPLAY_TYPE_DPI:
|
|
|
|
case OMAP_DISPLAY_TYPE_DBI:
|
|
|
|
case OMAP_DISPLAY_TYPE_SDI:
|
|
|
|
case OMAP_DISPLAY_TYPE_VENC:
|
|
|
|
case OMAP_DISPLAY_TYPE_HDMI:
|
|
|
|
default_get_overlay_fifo_thresholds(ovl->id, size,
|
2011-11-15 16:56:57 +07:00
|
|
|
burst_size, &op->fifo_low,
|
|
|
|
&op->fifo_high);
|
2011-11-04 14:48:54 +07:00
|
|
|
break;
|
|
|
|
#ifdef CONFIG_OMAP2_DSS_DSI
|
|
|
|
case OMAP_DISPLAY_TYPE_DSI:
|
|
|
|
dsi_get_overlay_fifo_thresholds(ovl->id, size,
|
2011-11-15 16:56:57 +07:00
|
|
|
burst_size, &op->fifo_low,
|
|
|
|
&op->fifo_high);
|
2011-11-04 14:48:54 +07:00
|
|
|
break;
|
|
|
|
#endif
|
|
|
|
default:
|
|
|
|
BUG();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
int omap_dss_mgr_apply(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
2011-11-05 15:59:59 +07:00
|
|
|
int r;
|
2011-11-04 14:48:54 +07:00
|
|
|
unsigned long flags;
|
2011-11-05 15:59:59 +07:00
|
|
|
struct omap_overlay *ovl;
|
2011-11-15 19:43:53 +07:00
|
|
|
struct mgr_priv_data *mp = get_mgr_priv(mgr);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
DSSDBG("omap_dss_mgr_apply(%s)\n", mgr->name);
|
|
|
|
|
|
|
|
r = dispc_runtime_get();
|
|
|
|
if (r)
|
|
|
|
return r;
|
|
|
|
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
/* Configure overlays */
|
2011-11-05 15:59:59 +07:00
|
|
|
list_for_each_entry(ovl, &mgr->overlays, list)
|
2011-11-04 14:48:54 +07:00
|
|
|
omap_dss_mgr_apply_ovl(ovl);
|
|
|
|
|
|
|
|
/* Configure manager */
|
|
|
|
omap_dss_mgr_apply_mgr(mgr);
|
|
|
|
|
|
|
|
/* Configure overlay fifos */
|
2011-11-05 15:59:59 +07:00
|
|
|
list_for_each_entry(ovl, &mgr->overlays, list)
|
2011-11-04 14:48:54 +07:00
|
|
|
omap_dss_mgr_apply_ovl_fifos(ovl);
|
|
|
|
|
|
|
|
r = 0;
|
2011-11-15 19:43:53 +07:00
|
|
|
if (mp->enabled && !mgr_manual_update(mgr)) {
|
2011-11-15 17:04:43 +07:00
|
|
|
if (!dss_data.irq_enabled)
|
2011-11-15 16:18:12 +07:00
|
|
|
dss_register_vsync_isr();
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 16:47:39 +07:00
|
|
|
dss_write_regs();
|
2011-11-04 14:35:59 +07:00
|
|
|
}
|
2011-11-04 14:48:54 +07:00
|
|
|
|
2011-11-15 17:04:10 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-04 14:48:54 +07:00
|
|
|
|
|
|
|
dispc_runtime_put();
|
|
|
|
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
2011-11-04 15:22:46 +07:00
|
|
|
void dss_mgr_enable(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
2011-11-15 19:43:53 +07:00
|
|
|
struct mgr_priv_data *mp = get_mgr_priv(mgr);
|
|
|
|
unsigned long flags;
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
2011-11-09 20:30:11 +07:00
|
|
|
if (!mgr_manual_update(mgr))
|
|
|
|
dispc_mgr_enable(mgr->id, true);
|
2011-11-15 19:43:53 +07:00
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
mp->enabled = true;
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
|
|
|
|
mutex_unlock(&apply_lock);
|
2011-11-04 15:22:46 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
void dss_mgr_disable(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
2011-11-15 19:43:53 +07:00
|
|
|
struct mgr_priv_data *mp = get_mgr_priv(mgr);
|
|
|
|
unsigned long flags;
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
2011-11-09 20:30:11 +07:00
|
|
|
if (!mgr_manual_update(mgr))
|
|
|
|
dispc_mgr_enable(mgr->id, false);
|
2011-11-15 19:43:53 +07:00
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
mp->enabled = false;
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
|
|
|
|
mutex_unlock(&apply_lock);
|
2011-11-04 15:22:46 +07:00
|
|
|
}
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
int dss_mgr_set_info(struct omap_overlay_manager *mgr,
|
|
|
|
struct omap_overlay_manager_info *info)
|
|
|
|
{
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
mgr->info = *info;
|
|
|
|
mgr->info_dirty = true;
|
|
|
|
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
void dss_mgr_get_info(struct omap_overlay_manager *mgr,
|
|
|
|
struct omap_overlay_manager_info *info)
|
|
|
|
{
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
*info = mgr->info;
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-15 17:15:18 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
int dss_mgr_set_device(struct omap_overlay_manager *mgr,
|
|
|
|
struct omap_dss_device *dssdev)
|
|
|
|
{
|
|
|
|
int r;
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
if (dssdev->manager) {
|
|
|
|
DSSERR("display '%s' already has a manager '%s'\n",
|
|
|
|
dssdev->name, dssdev->manager->name);
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:15:18 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
if ((mgr->supported_displays & dssdev->type) == 0) {
|
|
|
|
DSSERR("display '%s' does not support manager '%s'\n",
|
|
|
|
dssdev->name, mgr->name);
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:15:18 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
dssdev->manager = mgr;
|
|
|
|
mgr->device = dssdev;
|
|
|
|
mgr->device_changed = true;
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
return 0;
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
err:
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
return r;
|
2011-11-15 17:15:18 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
int dss_mgr_unset_device(struct omap_overlay_manager *mgr)
|
|
|
|
{
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
int r;
|
|
|
|
|
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
if (!mgr->device) {
|
|
|
|
DSSERR("failed to unset display, display not set.\n");
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:15:18 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Don't allow currently enabled displays to have the overlay manager
|
|
|
|
* pulled out from underneath them
|
|
|
|
*/
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
if (mgr->device->state != OMAP_DSS_DISPLAY_DISABLED) {
|
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
|
|
|
}
|
2011-11-15 17:15:18 +07:00
|
|
|
|
|
|
|
mgr->device->manager = NULL;
|
|
|
|
mgr->device = NULL;
|
|
|
|
mgr->device_changed = true;
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
|
2011-11-15 17:15:18 +07:00
|
|
|
return 0;
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
err:
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
return r;
|
2011-11-15 17:15:18 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
int dss_ovl_set_info(struct omap_overlay *ovl,
|
|
|
|
struct omap_overlay_info *info)
|
|
|
|
{
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
ovl->info = *info;
|
|
|
|
ovl->info_dirty = true;
|
|
|
|
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
void dss_ovl_get_info(struct omap_overlay *ovl,
|
|
|
|
struct omap_overlay_info *info)
|
|
|
|
{
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
*info = ovl->info;
|
OMAPDSS: APPLY: add missing uses of spinlock
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The other
group will not sleep and can be called from interrupts, and the other
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
apply.c already contains a spinlock, which has been used to protect
(badly) the dss_data. This patch adds locks/unlocks of the spinlock to
the missing places, and the lock should now properly protect dss_data.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:32:57 +07:00
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
int dss_ovl_set_manager(struct omap_overlay *ovl,
|
|
|
|
struct omap_overlay_manager *mgr)
|
|
|
|
{
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
struct ovl_priv_data *op = get_ovl_priv(ovl);
|
|
|
|
unsigned long flags;
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
int r;
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
if (!mgr)
|
|
|
|
return -EINVAL;
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
if (ovl->manager) {
|
|
|
|
DSSERR("overlay '%s' already has a manager '%s'\n",
|
|
|
|
ovl->name, ovl->manager->name);
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
if (op->enabled) {
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-15 17:11:11 +07:00
|
|
|
DSSERR("overlay has to be disabled to change the manager\n");
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
ovl->manager = mgr;
|
|
|
|
list_add_tail(&ovl->list, &mgr->overlays);
|
|
|
|
ovl->manager_changed = true;
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
/* XXX: When there is an overlay on a DSI manual update display, and
|
|
|
|
* the overlay is first disabled, then moved to tv, and enabled, we
|
|
|
|
* seem to get SYNC_LOST_DIGIT error.
|
|
|
|
*
|
|
|
|
* Waiting doesn't seem to help, but updating the manual update display
|
|
|
|
* after disabling the overlay seems to fix this. This hints that the
|
|
|
|
* overlay is perhaps somehow tied to the LCD output until the output
|
|
|
|
* is updated.
|
|
|
|
*
|
|
|
|
* Userspace workaround for this is to update the LCD after disabling
|
|
|
|
* the overlay, but before moving the overlay to TV.
|
|
|
|
*/
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
return 0;
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
err:
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
return r;
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
int dss_ovl_unset_manager(struct omap_overlay *ovl)
|
|
|
|
{
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
struct ovl_priv_data *op = get_ovl_priv(ovl);
|
|
|
|
unsigned long flags;
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
int r;
|
|
|
|
|
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
if (!ovl->manager) {
|
|
|
|
DSSERR("failed to detach overlay: manager not set\n");
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
if (op->enabled) {
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
2011-11-15 17:11:11 +07:00
|
|
|
DSSERR("overlay has to be disabled to unset the manager\n");
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
ovl->manager = NULL;
|
|
|
|
list_del(&ovl->list);
|
|
|
|
ovl->manager_changed = true;
|
|
|
|
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
err:
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool dss_ovl_is_enabled(struct omap_overlay *ovl)
|
|
|
|
{
|
|
|
|
struct ovl_priv_data *op = get_ovl_priv(ovl);
|
|
|
|
unsigned long flags;
|
|
|
|
bool e;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
e = op->enabled;
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
|
|
|
return e;
|
|
|
|
}
|
|
|
|
|
|
|
|
int dss_ovl_enable(struct omap_overlay *ovl)
|
|
|
|
{
|
|
|
|
struct ovl_priv_data *op = get_ovl_priv(ovl);
|
|
|
|
unsigned long flags;
|
|
|
|
int r;
|
|
|
|
|
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
|
|
|
if (ovl->manager == NULL || ovl->manager->device == NULL) {
|
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
|
|
|
}
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
op->enabled = true;
|
|
|
|
op->extra_info_dirty = true;
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
err:
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
|
|
|
int dss_ovl_disable(struct omap_overlay *ovl)
|
|
|
|
{
|
|
|
|
struct ovl_priv_data *op = get_ovl_priv(ovl);
|
|
|
|
unsigned long flags;
|
|
|
|
int r;
|
|
|
|
|
|
|
|
mutex_lock(&apply_lock);
|
|
|
|
|
|
|
|
if (ovl->manager == NULL || ovl->manager->device == NULL) {
|
|
|
|
r = -EINVAL;
|
|
|
|
goto err;
|
|
|
|
}
|
|
|
|
|
|
|
|
spin_lock_irqsave(&data_lock, flags);
|
|
|
|
|
|
|
|
op->enabled = false;
|
|
|
|
op->extra_info_dirty = true;
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&data_lock, flags);
|
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
|
2011-11-15 17:11:11 +07:00
|
|
|
return 0;
|
OMAPDSS: APPLY: rewrite overlay enable/disable
Overlays are currently enabled and disabled with a boolean in the struct
omap_overlay_info. The overlay info is set with ovl->set_overlay_info(),
and made into use with mgr->apply().
This doesn't work properly, as the enable/disable status may affect also
other overlays, for example when using fifo-merge. Thus the enabling and
disabling of the overlay needs to be done outside the normal overlay
configuration.
This patch achieves that by doing the following things:
1) Add function pointers to struct omap_overlay: enable(), disable() and
is_enabled(). These are used to do the obvious. The functions may block.
2) Move the "enabled" field from struct omap_overlay to ovl_priv_data.
3) Add a new route for settings to be applied to the HW, called
"extra_info". The status of the normal info and extra_info are tracked
separately.
The point here is to allow the normal info to be changed and
applied in non-blocking matter, whereas the extra_info can only be
changed when holding the mutex. This makes it possible to, for example,
set the overlay enable flag, apply it, and wait until the HW has taken
the flag into use.
This is not possible if the enable flag would be in the normal info, as
a new value for the flag could be set at any time from the users of
omapdss.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 21:37:53 +07:00
|
|
|
|
OMAPDSS: APPLY: add mutex
The functions in apply.c, called mostly via function pointers in overlay
and overlay_manager structs, will be divided into two groups. The first
group will not sleep and can be called from interrupts, and the second
group may sleep.
The idea is that the non-sleeping functions may only change certain
settings in overlays and managers, and those settings may only affect
the particular overlay/manager. For example, set the base address of the
overlay.
The blocking functions, however, will handle more complex configuration
changes. For example, when an overlay is enabled and fifo-merge feature
is used, we need to do the enable in multiple steps, waiting in between,
and the change affects multiple overlays and managers.
This patch adds the mutex which is used in the blocking functions to
have exclusive access to overlays and overlay managers.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
2011-11-15 19:28:48 +07:00
|
|
|
err:
|
|
|
|
mutex_unlock(&apply_lock);
|
|
|
|
return r;
|
2011-11-15 17:11:11 +07:00
|
|
|
}
|
|
|
|
|