diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2018-06-05 02:06:54 +0300 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2018-09-03 16:13:29 +0300 |
commit | bb23800c887da3a96297ec3d4a09b6bd887ce503 (patch) | |
tree | b21b2aff4bf8dba701bb7ad94ca82951b7a9e272 /drivers/gpu/drm/omapdrm/displays/connector-dvi.c | |
parent | 8fe1d36100c84ab1b501771252755d75ddcb6bff (diff) | |
download | linux-bb23800c887da3a96297ec3d4a09b6bd887ce503.tar.xz |
drm/omap: Remove duplicate calls to .set_timings() operation
The omap_dss_device .set_timings() operations are called directly from
omap_encoder_update(), and indirectly from the omap_dss_device .enable()
operation. The latter is called from omap_encoder_enable(), right after
calling omap_encoder_update(). The .set_timings() operation it thus
called twice in a row. Fix it by removing the indirect call.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/gpu/drm/omapdrm/displays/connector-dvi.c')
-rw-r--r-- | drivers/gpu/drm/omapdrm/displays/connector-dvi.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/gpu/drm/omapdrm/displays/connector-dvi.c b/drivers/gpu/drm/omapdrm/displays/connector-dvi.c index 8f953303ece6..b89555ed53a0 100644 --- a/drivers/gpu/drm/omapdrm/displays/connector-dvi.c +++ b/drivers/gpu/drm/omapdrm/displays/connector-dvi.c @@ -69,7 +69,6 @@ static void dvic_disconnect(struct omap_dss_device *src, static int dvic_enable(struct omap_dss_device *dssdev) { - struct panel_drv_data *ddata = to_panel_data(dssdev); struct omap_dss_device *src = dssdev->src; int r; @@ -79,8 +78,6 @@ static int dvic_enable(struct omap_dss_device *dssdev) if (omapdss_device_is_enabled(dssdev)) return 0; - src->ops->set_timings(src, &ddata->vm); - r = src->ops->enable(src); if (r) return r; |