<div dir="ltr"><div dir="ltr">Hi Jacopo,<div><br></div><div>Thank you for your patch.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 15 Jul 2022 at 14:56, Jacopo Mondi <<a href="mailto:jacopo@jmondi.org">jacopo@jmondi.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If the pixel format gets adjusted to the default NV12 format during<br>
validation, it gets currently ignored and the user-supplied format is<br>
used nonetheless.<br>
<br>
Fix it by using the adjusted pixel format.<br>
<br>
Signed-off-by: Jacopo Mondi <<a href="mailto:jacopo@jmondi.org" target="_blank">jacopo@jmondi.org</a>><br></blockquote><div><br></div><div>Looks right to me.</div><div><br></div><div>Reviewed-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com">naush@raspberrypi.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp<br>
index fdc24cd530c2..8c8813038483 100644<br>
--- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp<br>
+++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp<br>
@@ -523,7 +523,7 @@ CameraConfiguration::Status RPiCameraConfiguration::validate()<br>
                }<br>
<br>
                V4L2DeviceFormat format;<br>
-               format.fourcc = V4L2PixelFormat::fromPixelFormat(cfg.pixelFormat);<br>
+               format.fourcc = V4L2PixelFormat::fromPixelFormat(cfgPixFmt);<br>
                format.size = cfg.size;<br>
                format.colorSpace = cfg.colorSpace;<br>
<br>
--<br>
2.36.1<br>
<br>
</blockquote></div></div>