diff options
| author | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2012-12-07 16:47:28 +0200 | 
|---|---|---|
| committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2012-12-07 16:51:36 +0200 | 
| commit | 636f4e1b45c6204c6912cefa2bdbe22e00784a43 (patch) | |
| tree | a3b93ff53f91c714e9817c75de0060c3827a91c7 /drivers/video/omap2/omapfb/omapfb-main.c | |
| parent | 09645d258334c650cb65a93f55d9acdcd4420d44 (diff) | |
| download | olio-linux-3.10-636f4e1b45c6204c6912cefa2bdbe22e00784a43.tar.xz olio-linux-3.10-636f4e1b45c6204c6912cefa2bdbe22e00784a43.zip  | |
OMAPFB: move dssdev->sync call out from omapfb_realloc_fbmem
Currently omapfb_realloc_fbmem() calls dssdev->sync to ensure any
possible frame update is finished. This patch moves the call to
dssdev->sync from omapfb_realloc_fbmem to the callers of
omapfb_realloc_fbmem.
This keeps dssdev related calls out from omapfb_realloc_fbmem, which
makes sense as the function should only deal with fb memory. Also, this
seems to avoid a lockdep warning about possible circular locking.
However, the exact reason for that warning is still unclear.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video/omap2/omapfb/omapfb-main.c')
| -rw-r--r-- | drivers/video/omap2/omapfb/omapfb-main.c | 4 | 
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/video/omap2/omapfb/omapfb-main.c b/drivers/video/omap2/omapfb/omapfb-main.c index 1a69d7ca92e..85130fa7ae5 100644 --- a/drivers/video/omap2/omapfb/omapfb-main.c +++ b/drivers/video/omap2/omapfb/omapfb-main.c @@ -1611,7 +1611,6 @@ int omapfb_realloc_fbmem(struct fb_info *fbi, unsigned long size, int type)  {  	struct omapfb_info *ofbi = FB2OFB(fbi);  	struct omapfb2_device *fbdev = ofbi->fbdev; -	struct omap_dss_device *display = fb2display(fbi);  	struct omapfb2_mem_region *rg = ofbi->region;  	unsigned long old_size = rg->size;  	unsigned long old_paddr = rg->paddr; @@ -1626,9 +1625,6 @@ int omapfb_realloc_fbmem(struct fb_info *fbi, unsigned long size, int type)  	if (old_size == size && old_type == type)  		return 0; -	if (display && display->driver->sync) -			display->driver->sync(display); -  	omapfb_free_fbmem(fbi);  	if (size == 0) {  |