summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorLaurent Pinchart2015-06-19 13:51:22 +0200
committerMauro Carvalho Chehab2015-07-17 15:02:48 +0200
commitdf5c3e7c8a87a4384ff7a0adba16baae9a40a566 (patch)
tree7741eba8b5964817e7823599bd7e8d12eadf43d5 /drivers
parent[media] v4l: xilinx: missing error code (diff)
downloadkernel-qcow2-linux-df5c3e7c8a87a4384ff7a0adba16baae9a40a566.tar.gz
kernel-qcow2-linux-df5c3e7c8a87a4384ff7a0adba16baae9a40a566.tar.xz
kernel-qcow2-linux-df5c3e7c8a87a4384ff7a0adba16baae9a40a566.zip
[media] v4l: vsp1: Fix plane stride and size checks
The checks need to be performed on up to two planes, as the third plane, if present, must have the same stride and size as the second plane. The code incorrectly performs the checks on at least two planes instead of at most two planes, fix it. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/media/platform/vsp1/vsp1_video.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/vsp1/vsp1_video.c b/drivers/media/platform/vsp1/vsp1_video.c
index 770e08dc03f1..3c124c14ce14 100644
--- a/drivers/media/platform/vsp1/vsp1_video.c
+++ b/drivers/media/platform/vsp1/vsp1_video.c
@@ -245,7 +245,7 @@ static int __vsp1_video_try_format(struct vsp1_video *video,
* the datasheet, strides not aligned to a multiple of 128 bytes result
* in image corruption.
*/
- for (i = 0; i < max(info->planes, 2U); ++i) {
+ for (i = 0; i < min(info->planes, 2U); ++i) {
unsigned int hsub = i > 0 ? info->hsub : 1;
unsigned int vsub = i > 0 ? info->vsub : 1;
unsigned int align = 128;