media: rcar-vin: Use correct pixel format when aligning format
authorNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Wed, 4 Dec 2019 17:57:59 +0000 (18:57 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Thu, 23 Jan 2020 12:09:08 +0000 (13:09 +0100)
When aligning the format the pixel format that is being processed shall
be used to figure out alignment constraints, not the currently active
pixel format. The alignment might be part of a try operation and shall
not be effected by the active format. Fix this by looking at the correct
pixel format.

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/platform/rcar-vin/rcar-v4l2.c

index 9e2e63f..5ff565e 100644 (file)
@@ -144,7 +144,7 @@ static void rvin_format_align(struct rvin_dev *vin, struct v4l2_pix_format *pix)
        }
 
        /* HW limit width to a multiple of 32 (2^5) for NV12/16 else 2 (2^1) */
-       switch (vin->format.pixelformat) {
+       switch (pix->pixelformat) {
        case V4L2_PIX_FMT_NV12:
        case V4L2_PIX_FMT_NV16:
                walign = 5;