diff options
| author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2018-06-05 01:06:54 +0200 |
|---|---|---|
| committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2018-09-03 15:13:29 +0200 |
| commit | bb23800c887da3a96297ec3d4a09b6bd887ce503 (patch) | |
| tree | b21b2aff4bf8dba701bb7ad94ca82951b7a9e272 /drivers/gpu/drm/omapdrm/displays/panel-tpo-td028ttec1.c | |
| parent | drm/omap: Make the video_mode pointer to .set_timings() const (diff) | |
| download | linux-bb23800c887da3a96297ec3d4a09b6bd887ce503.tar.xz linux-bb23800c887da3a96297ec3d4a09b6bd887ce503.zip | |
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/panel-tpo-td028ttec1.c')
| -rw-r--r-- | drivers/gpu/drm/omapdrm/displays/panel-tpo-td028ttec1.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/omapdrm/displays/panel-tpo-td028ttec1.c b/drivers/gpu/drm/omapdrm/displays/panel-tpo-td028ttec1.c index 087f62f4311b..b2a16c470f73 100644 --- a/drivers/gpu/drm/omapdrm/displays/panel-tpo-td028ttec1.c +++ b/drivers/gpu/drm/omapdrm/displays/panel-tpo-td028ttec1.c @@ -187,8 +187,6 @@ static int td028ttec1_panel_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; |