From: Tomi Valkeinen Date: Fri, 8 Jan 2016 13:51:02 +0000 (+0200) Subject: drm/omap: verify that fb plane pitches are the same X-Git-Tag: v5.15~13916^2~24^2~48 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=be4f235c43f06ae749b8f99b04f899d1e9bd561e;p=platform%2Fkernel%2Flinux-starfive.git drm/omap: verify that fb plane pitches are the same The DSS hardware uses the same ROW_INC value for both Y and UV planes for NV12 format. This means that the pitches of the Y and UV planes have to match. omapdrm doesn't check this at the moment, and this can lead into a broken NV12 fb on the screen. This patch adds the check. Signed-off-by: Tomi Valkeinen Reviewed-by: Laurent Pinchart --- diff --git a/drivers/gpu/drm/omapdrm/omap_fb.c b/drivers/gpu/drm/omapdrm/omap_fb.c index ad202df..481512d 100644 --- a/drivers/gpu/drm/omapdrm/omap_fb.c +++ b/drivers/gpu/drm/omapdrm/omap_fb.c @@ -449,6 +449,14 @@ struct drm_framebuffer *omap_framebuffer_init(struct drm_device *dev, goto fail; } + if (i > 0 && pitch != mode_cmd->pitches[i - 1]) { + dev_err(dev->dev, + "pitches are not the same between framebuffer planes %d != %d\n", + pitch, mode_cmd->pitches[i - 1]); + ret = -EINVAL; + goto fail; + } + plane->bo = bos[i]; plane->offset = mode_cmd->offsets[i]; plane->pitch = pitch;