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, 2 insertions, 14 deletions

diff --git a/drivers/gpu/drm/omapdrm/omap_encoder.c b/drivers/gpu/drm/omapdrm/omap_encoder.c
index 94b75d018e71..a6dce480b2cf 100644
--- a/drivers/gpu/drm/omapdrm/omap_encoder.c
+++ b/drivers/gpu/drm/omapdrm/omap_encoder.c
@@ -101,21 +101,9 @@ static int omap_encoder_update(struct drm_encoder *encoder,
         struct omap_dss_device *dssdev = omap_encoder->display;
         int ret;
 
-        if (dssdev->ops->check_timings) {
-                ret = dssdev->ops->check_timings(dssdev, vm);
-        } else {
-                struct videomode t = {0};
-
-                dssdev->ops->get_timings(dssdev, &t);
-
-                if (memcmp(vm, &t, sizeof(*vm)))
-                        ret = -EINVAL;
-                else
-                        ret = 0;
-        }
-
+        ret = dssdev->ops->check_timings(dssdev, vm);
         if (ret) {
-                dev_err(dev->dev, "could not set timings: %d\n", ret);
+                dev_err(dev->dev, "invalid timings: %d\n", ret);
                 return ret;
         }