[PATCH 3/3] drm/omap: print error instead of WARN() if plane setup fails

[Tomi Valkeinen]

omap_plane_atomic_update() does WARN_ON() if dispc rejects the given
plane config. Change that to dev_err() to lessen the possible spam.

To fix this correctly, the plane setup needs much more work by creating
a check function for dispc setup, so that we could reliably check the
config in atomic_check, instead of only noticing the problem when
programming dispc.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
---
 drivers/gpu/drm/omapdrm/omap_plane.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c 
b/drivers/gpu/drm/omapdrm/omap_plane.c
index 5664200ef05b..4f39855d80fe 100644
--- a/drivers/gpu/drm/omapdrm/omap_plane.c
+++ b/drivers/gpu/drm/omapdrm/omap_plane.c
@@ -135,7 +135,9 @@ static void omap_plane_atomic_update(struct drm_plane 
*plane,
        /* and finally, update omapdss: */
        ret = dispc_ovl_setup(omap_plane->id, &info, false,
                              omap_crtc_timings(state->crtc), false);
-       if (WARN_ON(ret)) {
+       if (ret) {
+               dev_err(plane->dev->dev, "Failed to setup plane %s\n",
+                       omap_plane->name);
                dispc_ovl_enable(omap_plane->id, false);
                return;
        }
-- 
2.5.0