OMAP: DSS2: adjust YUV overlay width to be even
authorTomi Valkeinen <tomi.valkeinen@nokia.com>
Thu, 3 Jun 2010 13:27:46 +0000 (16:27 +0300)
committerTomi Valkeinen <tomi.valkeinen@nokia.com>
Thu, 5 Aug 2010 13:52:11 +0000 (16:52 +0300)
An overlay in YUV mode has to have an even input width, because data for
each pixel is divided between two adjacent pixels.

The algorithm handling manual update overlay adjusting may adjust the
overlay width to be odd.

This patch adds a check for that situation, and makes the width even.
The width is increased by one if it is possible (the unadjusted input
width is larger than the width), and decreased by one if increasing is
not possible.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
drivers/video/omap2/dss/manager.c

index 4b09fcb432b3de140c8bf6b0416010283e9ba478..6a649ab5539ea3492c9f85599c93c891581b3dbb 100644 (file)
@@ -813,6 +813,21 @@ static int configure_overlay(enum omap_plane plane)
 
                w = w * outw / orig_outw;
                h = h * outh / orig_outh;
+
+               /* YUV mode overlay's input width has to be even and the
+                * algorithm above may adjust the width to be odd.
+                *
+                * Here we adjust the width if needed, preferring to increase
+                * the width if the original width was bigger.
+                */
+               if ((w & 1) &&
+                               (c->color_mode == OMAP_DSS_COLOR_YUV2 ||
+                                c->color_mode == OMAP_DSS_COLOR_UYVY)) {
+                       if (orig_w > w)
+                               w += 1;
+                       else
+                               w -= 1;
+               }
        }
 
        r = dispc_setup_plane(plane,