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

Subject: media: atomisp: move a debug printf to a better place
Author:  Mauro Carvalho Chehab <[email protected]>
Date:    Thu Nov 4 11:20:37 2021 +0000

The sensor width/height report is alread being printed after
its calculus. The only reason for an extra debug printk is
when dis is used. So, change its message to reflect and move
it to be inside the if checks.

Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/staging/media/atomisp/pci/atomisp_cmd.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.c 
b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
index feb75491a273..76c9e89afb48 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_cmd.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
@@ -5583,6 +5583,10 @@ static int atomisp_set_fmt_to_snr(struct video_device 
*vdev,
                                       pad, set_fmt, &pad_state, &vformat);
                if (ret)
                        return ret;
+
+               dev_dbg(isp->dev, "video dis: sensor width: %d, height: %d\n",
+                       ffmt->width, ffmt->height);
+
                if (ffmt->width < req_ffmt->width ||
                    ffmt->height < req_ffmt->height) {
                        req_ffmt->height -= dvs_env_h;
@@ -5593,8 +5597,6 @@ static int atomisp_set_fmt_to_snr(struct video_device 
*vdev,
                        asd->params.video_dis_en = false;
                }
        }
-       dev_dbg(isp->dev, "sensor width: %d, height: %d\n",
-               ffmt->width, ffmt->height);
        vformat.which = V4L2_SUBDEV_FORMAT_ACTIVE;
        ret = v4l2_subdev_call(isp->inputs[asd->input_curr].camera, pad,
                               set_fmt, NULL, &vformat);

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to