[PATCH 30/33] drm/omap: verify that fb plane pitches are the same
laurent.pinchart at ideasonboard.com
Fri Feb 26 08:55:48 UTC 2016
On Thursday 25 February 2016 17:56:57 Tomi Valkeinen wrote:
> On 24/02/16 01:02, Laurent Pinchart wrote:
> > On Friday 19 February 2016 11:48:05 Tomi Valkeinen wrote:
> >> 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 <tomi.valkeinen at ti.com>
> > Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> > We can probably simplify the implementation further as most of the checks
> > in the loop apply to the first place only. Would you like me to give it a
> > go ?
> "place" is plane?
> I don't quite understand what you mean. The checks apply to all planes.
My point is that if you add a check to ensure that the pitch is identical in
all planes, the other tests on the pitch don't have to be repeated for every
More information about the dri-devel