OSDN Git Service

drm/omap: dsi: drop useless sync()
authorSebastian Reichel <sebastian.reichel@collabora.com>
Tue, 15 Dec 2020 10:45:53 +0000 (12:45 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Tue, 15 Dec 2020 14:08:22 +0000 (16:08 +0200)
The DSI sync() function only locks the bus and then releases
it again. Currently the only invocation is directly before
update(), which locks the bus anyways.

Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20201215104657.802264-21-tomi.valkeinen@ti.com
drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
drivers/gpu/drm/omapdrm/dss/omapdss.h
drivers/gpu/drm/omapdrm/omap_crtc.c

index c1afb71..f1e917a 100644 (file)
@@ -844,23 +844,6 @@ err:
        return r;
 }
 
-static int dsicm_sync(struct omap_dss_device *dssdev)
-{
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
-       struct omap_dss_device *src = ddata->src;
-
-       dev_dbg(&ddata->dsi->dev, "sync\n");
-
-       mutex_lock(&ddata->lock);
-       src->ops->dsi.bus_lock(src);
-       src->ops->dsi.bus_unlock(src);
-       mutex_unlock(&ddata->lock);
-
-       dev_dbg(&ddata->dsi->dev, "sync done\n");
-
-       return 0;
-}
-
 static int _dsicm_enable_te(struct panel_drv_data *ddata, bool enable)
 {
        struct omap_dss_device *src = ddata->src;
@@ -949,7 +932,6 @@ static const struct omap_dss_device_ops dsicm_ops = {
 
 static const struct omap_dss_driver dsicm_dss_driver = {
        .update         = dsicm_update,
-       .sync           = dsicm_sync,
 };
 
 static int dsicm_probe_of(struct mipi_dsi_device *dsi)
index d1812f0..2f57f95 100644 (file)
@@ -388,7 +388,6 @@ struct omap_dss_device {
 struct omap_dss_driver {
        int (*update)(struct omap_dss_device *dssdev,
                               u16 x, u16 y, u16 w, u16 h);
-       int (*sync)(struct omap_dss_device *dssdev);
 };
 
 struct dss_device *omapdss_get_dss(void);
index f598c03..a503002 100644 (file)
@@ -379,9 +379,6 @@ static void omap_crtc_manual_display_update(struct work_struct *data)
                return;
        }
 
-       if (dssdrv->sync)
-               dssdrv->sync(dssdev);
-
        ret = dssdrv->update(dssdev, 0, 0, mode->hdisplay, mode->vdisplay);
        if (ret < 0) {
                spin_lock_irq(&dev->event_lock);