drm/omap: Remove unneeded fallback for missing .check_timings()

The .check_timings() operation is present in all panels and connectors.
The fallback that uses .get_timings() in the absence of .check_timings()
is thus unneeded.

While it could be argued that the fallback implements a useful check
that should be extended to cover all fixed-resolution panels, the code
is currently unused and gets in the way of the ongoing refactoring.
Remove it, a similar feature can always be added later.

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>

1 files changed, 1 insertions, 24 deletions

diff --git a/drivers/gpu/drm/omapdrm/omap_connector.c b/drivers/gpu/drm/omapdrm/omap_connector.c
index 06c48a64b745..b58d9a0bb53d 100644
--- a/drivers/gpu/drm/omapdrm/omap_connector.c
+++ b/drivers/gpu/drm/omapdrm/omap_connector.c
@@ -261,30 +261,7 @@ static int omap_connector_mode_valid(struct drm_connector *connector,
         drm_display_mode_to_videomode(mode, &vm);
         mode->vrefresh = drm_mode_vrefresh(mode);
 
-        /*
-         * if the panel driver doesn't have a check_timings, it's most likely
-         * a fixed resolution panel, check if the timings match with the
-         * panel's timings
-         */
-        if (dssdev->ops->check_timings) {
-                r = dssdev->ops->check_timings(dssdev, &vm);
-        } else {
-                struct videomode t = {0};
-
-                dssdev->ops->get_timings(dssdev, &t);
-
-                /*
-                 * Ignore the flags, as we don't get them from
-                 * drm_display_mode_to_videomode.
-                 */
-                t.flags = 0;
-
-                if (memcmp(&vm, &t, sizeof(vm)))
-                        r = -EINVAL;
-                else
-                        r = 0;
-        }
-
+        r = dssdev->ops->check_timings(dssdev, &vm);
         if (!r) {
                 /* check if vrefresh is still valid */
                 new_mode = drm_mode_duplicate(dev, mode);