OMAP: DSS2: HDMI: remove error prints in check_timings

check_timings() is supposed to be used to verify if timings are ok or
not. Currently the HDMI driver prints error messages if the timings are
not ok. This is not right, as it is no error to give invalid timings to
check_timings().

Remove the error prints.

Cc: Mythri P K <mythripk@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

diff --git a/drivers/video/omap2/dss/hdmi.c b/drivers/video/omap2/dss/hdmi.c
index bf85cbadc8042b0f445a71de0a1b51f5c4f7f039..fb85ce5a8ab41af52b6ca1f9d9c7fc6684d04b58 100644 (file)
--- a/drivers/video/omap2/dss/hdmi.c
+++ b/drivers/video/omap2/dss/hdmi.c
@@ -406,7 +406,6 @@ int omapdss_hdmi_display_check_timing(struct omap_dss_device *dssdev,
 
        cm = hdmi_get_code(timings);
        if (cm.code == -1) {
-               DSSERR("Invalid timing entered\n");
                return -EINVAL;
        }
 

diff --git a/drivers/video/omap2/dss/hdmi_panel.c b/drivers/video/omap2/dss/hdmi_panel.c
index 79a3a5ac8f1cbf0ae49b662cf7af0ec6c83002eb..71aa8134f4fbe9d1290d5b2249777f5d65e92ef9 100644 (file)
--- a/drivers/video/omap2/dss/hdmi_panel.c
+++ b/drivers/video/omap2/dss/hdmi_panel.c
@@ -170,11 +170,7 @@ static int hdmi_check_timings(struct omap_dss_device *dssdev,
        mutex_lock(&hdmi.hdmi_lock);
 
        r = omapdss_hdmi_display_check_timing(dssdev, timings);
-       if (r) {
-               DSSERR("Timing cannot be applied\n");
-               goto err;
-       }
-err:
+
        mutex_unlock(&hdmi.hdmi_lock);
        return r;
 }