1 From 9a2eab654b11d27bcc5a32ebd374f6c9acc38ce4 Mon Sep 17 00:00:00 2001
2 From: Aman Gupta <aman@tmm1.net>
3 Date: Fri, 23 Aug 2019 16:29:07 -0700
4 Subject: [PATCH] staging: bcm2835-codec: remove unnecessary padding on
7 The ISP and ENCODE roles have the same underlying hardware. Neither requires vertical alignment.
9 Signed-off-by: Aman Gupta <aman@tmm1.net>
11 .../staging/vc04_services/bcm2835-codec/bcm2835-v4l2-codec.c | 4 ++--
12 1 file changed, 2 insertions(+), 2 deletions(-)
14 --- a/drivers/staging/vc04_services/bcm2835-codec/bcm2835-v4l2-codec.c
15 +++ b/drivers/staging/vc04_services/bcm2835-codec/bcm2835-v4l2-codec.c
16 @@ -1050,12 +1050,12 @@ static int vidioc_try_fmt(struct bcm2835
17 f->fmt.pix_mp.height = MIN_H;
20 - * For codecs the buffer must have a vertical alignment of 16
21 + * For decoders the buffer must have a vertical alignment of 16
23 * The selection will reflect any cropping rectangle when only
24 * some of the pixels are active.
26 - if (ctx->dev->role != ISP)
27 + if (ctx->dev->role == DECODE)
28 f->fmt.pix_mp.height = ALIGN(f->fmt.pix_mp.height, 16);
30 f->fmt.pix_mp.num_planes = 1;