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

Subject: media: atomisp: Propagate v4l2_mbus_framefmt.field to CSI port sink pad
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Fri Apr 12 15:27:53 2024 +0100

Most sensor drivers report V4L2_FIELD_NONE as v4l2_mbus_framefmt.field
value. Propagate this to the v4l2_mbus_framefmt for the CSI port to pass
the link validation done by media_pipeline_start().

Keep the v4l2_mbus_framefmt.field for the source pad of the CSI port
as V4L2_FIELD_ANY to match the ISP sink pad.

Reviewed-by: Andy Shevchenko <a...@kernel.org>
Signed-off-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/staging/media/atomisp/pci/atomisp_csi2.c | 3 +++
 1 file changed, 3 insertions(+)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_csi2.c 
b/drivers/staging/media/atomisp/pci/atomisp_csi2.c
index 89118438a3b6..e939a09a279b 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_csi2.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_csi2.c
@@ -107,9 +107,12 @@ int atomisp_csi2_set_ffmt(struct v4l2_subdev *sd,
                actual_ffmt->height = clamp_t(u32, ffmt->height,
                                              ATOM_ISP_MIN_HEIGHT,
                                              ATOM_ISP_MAX_HEIGHT);
+               actual_ffmt->field = ffmt->field;
 
                tmp_ffmt = *ffmt = *actual_ffmt;
 
+               /* Always use V4L2_FIELD_ANY to match the ISP sink pad */
+               tmp_ffmt.field = V4L2_FIELD_ANY;
                return atomisp_csi2_set_ffmt(sd, sd_state, which,
                                             CSI2_PAD_SOURCE,
                                             &tmp_ffmt);

Reply via email to