drm/omap: Remove .get_timings() operation from display connectors

The analog TV, DVI and HDMI connectors all report timing information
through the .get_timings() information.

For analog TV outputs the information is queried from the encoder, so
the operation is unused. Remove it.

For HDMI outputs the display pipeline provides EDID capability, so the
operation is unused as well. Remove it.

For DVI outputs the operation is also unused if the pipeline provides
EDID capability. Otherwise (when the DDC bus is not connected) we
shouldn't hardcode a single mode, but instead report no mode and let the
KMS core add default modes. This is achieved by removing the operation.

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>

diff --git a/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c b/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c
index a9e2a366a8513ec08a9f49c6dc8bf5db9864dc95..4866bf8ed5244618245adcad0101460dffd6a256 100644 (file)
--- a/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c
+++ b/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c
@@ -20,23 +20,6 @@ struct panel_drv_data {
        struct omap_dss_device dssdev;
 
        struct device *dev;
-
-       struct videomode vm;
-};
-
-static const struct videomode tvc_pal_vm = {
-       .hactive        = 720,
-       .vactive        = 574,
-       .pixelclock     = 13500000,
-       .hsync_len      = 64,
-       .hfront_porch   = 12,
-       .hback_porch    = 68,
-       .vsync_len      = 5,
-       .vfront_porch   = 5,
-       .vback_porch    = 41,
-
-       .flags          = DISPLAY_FLAGS_INTERLACED | DISPLAY_FLAGS_HSYNC_LOW |
-                         DISPLAY_FLAGS_VSYNC_LOW,
 };
 
 #define to_panel_data(x) container_of(x, struct panel_drv_data, dssdev)
@@ -93,22 +76,11 @@ static void tvc_disable(struct omap_dss_device *dssdev)
 static void tvc_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);
 }
 
-static void tvc_get_timings(struct omap_dss_device *dssdev,
-                           struct videomode *vm)
-{
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
-
-       *vm = ddata->vm;
-}
-
 static const struct omap_dss_device_ops tvc_ops = {
        .connect                = tvc_connect,
        .disconnect             = tvc_disconnect,
@@ -117,7 +89,6 @@ static const struct omap_dss_device_ops tvc_ops = {
        .disable                = tvc_disable,
 
        .set_timings            = tvc_set_timings,
-       .get_timings            = tvc_get_timings,
 };
 
 static int tvc_probe(struct platform_device *pdev)
@@ -132,8 +103,6 @@ static int tvc_probe(struct platform_device *pdev)
        platform_set_drvdata(pdev, ddata);
        ddata->dev = &pdev->dev;
 
-       ddata->vm = tvc_pal_vm;
-
        dssdev = &ddata->dssdev;
        dssdev->ops = &tvc_ops;
        dssdev->dev = &pdev->dev;

diff --git a/drivers/gpu/drm/omapdrm/displays/connector-dvi.c b/drivers/gpu/drm/omapdrm/displays/connector-dvi.c
index a9e2f1181987d0d23084c749d6e72de43e5500d3..818a4dc452e017eee0c4455acc5da81ade6b75a9 100644 (file)
--- a/drivers/gpu/drm/omapdrm/displays/connector-dvi.c
+++ b/drivers/gpu/drm/omapdrm/displays/connector-dvi.c
@@ -19,28 +19,9 @@
 
 #include "../dss/omapdss.h"
 
-static const struct videomode dvic_default_vm = {
-       .hactive        = 640,
-       .vactive        = 480,
-
-       .pixelclock     = 23500000,
-
-       .hfront_porch   = 48,
-       .hsync_len      = 32,
-       .hback_porch    = 80,
-
-       .vfront_porch   = 3,
-       .vsync_len      = 4,
-       .vback_porch    = 7,
-
-       .flags          = DISPLAY_FLAGS_HSYNC_HIGH | DISPLAY_FLAGS_VSYNC_HIGH,
-};
-
 struct panel_drv_data {
        struct omap_dss_device dssdev;
 
-       struct videomode vm;
-
        struct i2c_adapter *i2c_adapter;
 
        struct gpio_desc *hpd_gpio;
@@ -100,22 +81,11 @@ static void dvic_disable(struct omap_dss_device *dssdev)
 static void dvic_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);
 }
 
-static void dvic_get_timings(struct omap_dss_device *dssdev,
-                            struct videomode *vm)
-{
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
-
-       *vm = ddata->vm;
-}
-
 static int dvic_ddc_read(struct i2c_adapter *adapter,
                unsigned char *buf, u16 count, u8 offset)
 {
@@ -223,7 +193,6 @@ static const struct omap_dss_device_ops dvic_ops = {
        .disable        = dvic_disable,
 
        .set_timings    = dvic_set_timings,
-       .get_timings    = dvic_get_timings,
 
        .read_edid      = dvic_read_edid,
        .detect         = dvic_detect,
@@ -311,8 +280,6 @@ static int dvic_probe(struct platform_device *pdev)
        if (r)
                return r;
 
-       ddata->vm = dvic_default_vm;
-
        dssdev = &ddata->dssdev;
        dssdev->ops = &dvic_ops;
        dssdev->dev = &pdev->dev;

diff --git a/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c b/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c
index 7e449f8a9b5d5f8e9da7a41bee8b19aca3f30ef7..a32e4159242d51b24d9fb91167ad77452ea64158 100644 (file)
--- a/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c
+++ b/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c
@@ -17,20 +17,6 @@
 
 #include "../dss/omapdss.h"
 
-static const struct videomode hdmic_default_vm = {
-       .hactive        = 640,
-       .vactive        = 480,
-       .pixelclock     = 25175000,
-       .hsync_len      = 96,
-       .hfront_porch   = 16,
-       .hback_porch    = 48,
-       .vsync_len      = 2,
-       .vfront_porch   = 11,
-       .vback_porch    = 31,
-
-       .flags          = DISPLAY_FLAGS_HSYNC_LOW | DISPLAY_FLAGS_VSYNC_LOW,
-};
-
 struct panel_drv_data {
        struct omap_dss_device dssdev;
        void (*hpd_cb)(void *cb_data, enum drm_connector_status status);
@@ -39,8 +25,6 @@ struct panel_drv_data {
 
        struct device *dev;
 
-       struct videomode vm;
-
        struct gpio_desc *hpd_gpio;
 };
 
@@ -98,22 +82,11 @@ static void hdmic_disable(struct omap_dss_device *dssdev)
 static void hdmic_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);
 }
 
-static void hdmic_get_timings(struct omap_dss_device *dssdev,
-                             struct videomode *vm)
-{
-       struct panel_drv_data *ddata = to_panel_data(dssdev);
-
-       *vm = ddata->vm;
-}
-
 static bool hdmic_detect(struct omap_dss_device *dssdev)
 {
        struct panel_drv_data *ddata = to_panel_data(dssdev);
@@ -152,7 +125,6 @@ static const struct omap_dss_device_ops hdmic_ops = {
        .disable                = hdmic_disable,
 
        .set_timings            = hdmic_set_timings,
-       .get_timings            = hdmic_get_timings,
 
        .detect                 = hdmic_detect,
        .register_hpd_cb        = hdmic_register_hpd_cb,
@@ -215,8 +187,6 @@ static int hdmic_probe(struct platform_device *pdev)
                        return r;
        }
 
-       ddata->vm = hdmic_default_vm;
-
        dssdev = &ddata->dssdev;
        dssdev->ops = &hdmic_ops;
        dssdev->dev = &pdev->dev;