drm/omap: Don't store video mode internally for external encoders
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Wed, 6 Jun 2018 13:18:43 +0000 (16:18 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 3 Sep 2018 13:13:30 +0000 (16:13 +0300)
The omap_dss_device .set_timings() operation for external encoders
stores the video mode in the device data structure. That mode is then
never used again. Drop it.

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>
drivers/gpu/drm/omapdrm/displays/encoder-opa362.c
drivers/gpu/drm/omapdrm/displays/encoder-tfp410.c
drivers/gpu/drm/omapdrm/displays/encoder-tpd12s015.c

index 824f302..05d1286 100644 (file)
@@ -25,8 +25,6 @@ struct panel_drv_data {
        struct omap_dss_device dssdev;
 
        struct gpio_desc *enable_gpio;
-
-       struct videomode vm;
 };
 
 #define to_panel_data(x) container_of(x, struct panel_drv_data, dssdev)
@@ -90,13 +88,10 @@ static void opa362_disable(struct omap_dss_device *dssdev)
 static void opa362_set_timings(struct omap_dss_device *dssdev,
                               const struct videomode *vm)
 {
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
        struct omap_dss_device *src = dssdev->src;
 
        dev_dbg(dssdev->dev, "set_timings\n");
 
-       ddata->vm = *vm;
-
        src->ops->set_timings(src, vm);
 }
 
index de42339..86c90c1 100644 (file)
@@ -20,8 +20,6 @@ struct panel_drv_data {
        struct omap_dss_device dssdev;
 
        struct gpio_desc *pd_gpio;
-
-       struct videomode vm;
 };
 
 #define to_panel_data(x) container_of(x, struct panel_drv_data, dssdev)
@@ -87,11 +85,8 @@ static void tfp410_fix_timings(struct videomode *vm)
 static void tfp410_set_timings(struct omap_dss_device *dssdev,
                               const struct videomode *vm)
 {
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
        struct omap_dss_device *src = dssdev->src;
 
-       ddata->vm = *vm;
-
        src->ops->set_timings(src, vm);
 }
 
index d9a5902..1e24559 100644 (file)
@@ -28,8 +28,6 @@ struct panel_drv_data {
        struct gpio_desc *ct_cp_hpd_gpio;
        struct gpio_desc *ls_oe_gpio;
        struct gpio_desc *hpd_gpio;
-
-       struct videomode vm;
 };
 
 #define to_panel_data(x) container_of(x, struct panel_drv_data, dssdev)
@@ -96,11 +94,8 @@ static void tpd_disable(struct omap_dss_device *dssdev)
 static void tpd_set_timings(struct omap_dss_device *dssdev,
                            const struct videomode *vm)
 {
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
        struct omap_dss_device *src = dssdev->src;
 
-       ddata->vm = *vm;
-
        src->ops->set_timings(src, vm);
 }