drm/omap: remove unnecessary pitch round-up
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 18 Apr 2016 15:11:29 +0000 (18:11 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 6 Jun 2016 07:14:27 +0000 (10:14 +0300)
omapdrm checks if the pixel stride is divisible by 8. This is meant to
ensure that the byte stride is 32, which is required by SGX.

The check is not correct, as it checks for pixels, not bytes, and thus
needlessly increases the stride for, e.g., NV12.

Also, SGX driver is not supported in the mainline, and the TI's SGX
driver nowadays does the memory allocation itself and doesn't rely on
omapdrm to figure out the correct pitch.

So we can just remove the whole roundup.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
drivers/gpu/drm/omapdrm/omap_drv.h

index 3f823c36891267a388db643176c3662333f4a779..edc796fc8279838082347dc196f779ef85c72da2 100644 (file)
@@ -244,10 +244,7 @@ static inline int align_pitch(int pitch, int width, int bpp)
        int bytespp = (bpp + 7) / 8;
        /* in case someone tries to feed us a completely bogus stride: */
        pitch = max(pitch, width * bytespp);
-       /* PVR needs alignment to 8 pixels.. right now that is the most
-        * restrictive stride requirement..
-        */
-       return roundup(pitch, 8 * bytespp);
+       return pitch;
 }
 
 /* map crtc to vblank mask */