This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: am437x-vpfe: Fix missing first line
Author:  Benoit Parrot <bpar...@ti.com>
Date:    Fri Sep 20 14:05:43 2019 -0300

Previous generation of this driver were hard coded to handle
encoder/decoder where the first line never contains any data and
was therefore always skipped, however when dealing with actual
camera sensors the first line is always present.

Signed-off-by: Benoit Parrot <bpar...@ti.com>
Signed-off-by: Jyri Sarha <jsa...@ti.com>
Acked-by: Lad Prabhakar <prabhakar.cse...@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org>

 drivers/media/platform/am437x/am437x-vpfe.c | 4 ----
 1 file changed, 4 deletions(-)

---

diff --git a/drivers/media/platform/am437x/am437x-vpfe.c 
b/drivers/media/platform/am437x/am437x-vpfe.c
index a3d22f90e64c..1521c072f3e3 100644
--- a/drivers/media/platform/am437x/am437x-vpfe.c
+++ b/drivers/media/platform/am437x/am437x-vpfe.c
@@ -345,13 +345,9 @@ static void vpfe_ccdc_setwin(struct vpfe_ccdc *ccdc,
        if (frm_fmt == CCDC_FRMFMT_INTERLACED) {
                vert_nr_lines = (image_win->height >> 1) - 1;
                vert_start >>= 1;
-               /* Since first line doesn't have any data */
-               vert_start += 1;
                /* configure VDINT0 */
                val = (vert_start << VPFE_VDINT_VDINT0_SHIFT);
        } else {
-               /* Since first line doesn't have any data */
-               vert_start += 1;
                vert_nr_lines = image_win->height - 1;
                /*
                 * configure VDINT0 and VDINT1. VDINT1 will be at half

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to