drm/rcar-du: Clarify comment regarding plane Y source coordinate
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Fri, 14 Jun 2013 18:54:16 +0000 (20:54 +0200)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Fri, 9 Aug 2013 21:17:42 +0000 (23:17 +0200)
The R8A7790 DU documentation contains further information regarding the
plane Y source coordinate. Update the comment accordingly.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
drivers/gpu/drm/rcar-du/rcar_du_plane.c

index a65f81d..38ebd20 100644 (file)
@@ -103,9 +103,12 @@ void rcar_du_plane_update_base(struct rcar_du_plane *plane)
        struct rcar_du_device *rcdu = plane->dev;
        unsigned int index = plane->hwindex;
 
-       /* According to the datasheet the Y position is expressed in raster line
-        * units. However, 32bpp formats seem to require a doubled Y position
-        * value. Similarly, for the second plane, NV12 and NV21 formats seem to
+       /* The Y position is expressed in raster line units and must be doubled
+        * for 32bpp formats, according to the R8A7790 datasheet. No mention of
+        * doubling the Y position is found in the R8A7779 datasheet, but the
+        * rule seems to apply there as well.
+        *
+        * Similarly, for the second plane, NV12 and NV21 formats seem to
         * require a halved Y position value.
         */
        rcar_du_plane_write(rcdu, index, PnSPXR, plane->src_x);