<div dir="ltr">Hi Laurent<div><br></div><div>Ah yes, forgot about "fixes" tags. Let me try again...!</div><div><br></div><div>Thanks</div><div>David</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 13 Feb 2023 at 10:12, Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</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">Hi David,<br>
<br>
Thank you for the patch.<br>
<br>
On Mon, Feb 13, 2023 at 09:19:34AM +0000, David Plowman via libcamera-devel wrote:<br>
> This fixes a small error in the validateColorSpaces method where the<br>
> colour space was not being set to Raw if it had an "unset" (null<br>
> option) value.<br>
<br>
Should this have a Fixes: tag ?<br>
<br>
> Signed-off-by: David Plowman <<a href="mailto:david.plowman@raspberrypi.com" target="_blank">david.plowman@raspberrypi.com</a>><br>
> ---<br>
>  src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 5 ++---<br>
>  1 file changed, 2 insertions(+), 3 deletions(-)<br>
> <br>
> diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp<br>
> index c0dd9551..84120954 100644<br>
> --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp<br>
> +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp<br>
> @@ -449,10 +449,9 @@ CameraConfiguration::Status RPiCameraConfiguration::validateColorSpaces([[maybe_<br>
>               /* First fix up raw streams to have the "raw" colour space. */<br>
>               if (isRaw(cfg.pixelFormat)) {<br>
>                       /* If there was no value here, that doesn't count as "adjusted". */<br>
> -                     if (cfg.colorSpace && cfg.colorSpace != ColorSpace::Raw) {<br>
> +                     if (cfg.colorSpace && cfg.colorSpace != ColorSpace::Raw)<br>
>                               status = Adjusted;<br>
> -                             cfg.colorSpace = ColorSpace::Raw;<br>
> -                     }<br>
> +                     cfg.colorSpace = ColorSpace::Raw;<br>
>                       continue;<br>
>               }<br>
>  <br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>