<div dir="ltr">Will do, thanks! <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 25, 2021 at 3:09 PM <<a href="mailto:paul.elder@ideasonboard.com">paul.elder@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 Vedant,<br>
<br>
On Wed, Jun 23, 2021 at 08:16:56PM +0530, Vedant Paranjape wrote:<br>
> On running libcamera gstreamer element on Raspberry Pi 4B+, there was a<br>
> warning as follows:<br>
> <br>
> [3:23:59.717513107] [3577] WARN V4L2 v4l2_pixelformat.cpp:180 Unsupported V4L2 pixel format XB24<br>
> <br>
> This patch fixes the following warning, by adding BGRX8888 pixel format<br>
<br>
s/following//<br>
<br>
> in libcamera, and updating the existing mapping of V4L2_PIX_FMT_XRGB32<br>
> to it's corresponding libcamera pixel format. Additionally, adds a new<br>
<br>
s/it's/its/<br>
<br>
You should mention what the "corresponding libcamera pixel format" is.<br>
<br>
s/adds/add/<br>
<br>
> mapping of V4L2_PIX_FMT_RBGX32 to formats::XBGR8888<br>
> <br>
> Signed-off-by: Vedant Paranjape <<a href="mailto:vedantparanjape160201@gmail.com" target="_blank">vedantparanjape160201@gmail.com</a>><br>
<br>
With those changes,<br>
<br>
Reviewed-by: Paul Elder <<a href="mailto:paul.elder@ideasonboard.com" target="_blank">paul.elder@ideasonboard.com</a>><br>
<br>
> ---<br>
>  src/libcamera/formats.cpp          | 12 +++++++++++-<br>
>  src/libcamera/v4l2_pixelformat.cpp |  3 ++-<br>
>  2 files changed, 13 insertions(+), 2 deletions(-)<br>
> <br>
> diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp<br>
> index 55822f4c..e41bfc05 100644<br>
> --- a/src/libcamera/formats.cpp<br>
> +++ b/src/libcamera/formats.cpp<br>
> @@ -198,13 +198,23 @@ const std::map<PixelFormat, PixelFormatInfo> pixelFormatInfo{<br>
>       { formats::XBGR8888, {<br>
>               .name = "XBGR8888",<br>
>               .format = formats::XBGR8888,<br>
> -             .v4l2Format = V4L2PixelFormat(V4L2_PIX_FMT_XRGB32),<br>
> +             .v4l2Format = V4L2PixelFormat(V4L2_PIX_FMT_RGBX32),<br>
>               .bitsPerPixel = 32,<br>
>               .colourEncoding = PixelFormatInfo::ColourEncodingRGB,<br>
>               .packed = false,<br>
>               .pixelsPerGroup = 1,<br>
>               .planes = {{ { 4, 1 }, { 0, 0 }, { 0, 0 } }},<br>
>       } },<br>
> +     { formats::BGRX8888, {<br>
> +             .name = "BGRX8888",<br>
> +             .format = formats::BGRX8888,<br>
> +             .v4l2Format = V4L2PixelFormat(V4L2_PIX_FMT_XRGB32),<br>
> +             .bitsPerPixel = 32,<br>
> +             .colourEncoding = PixelFormatInfo::ColourEncodingRGB,<br>
> +             .packed = false,<br>
> +             .pixelsPerGroup = 1,<br>
> +             .planes = {{ { 4, 1 }, { 0, 0 }, { 0, 0 } }},<br>
> +     } },    <br>
>       { formats::ABGR8888, {<br>
>               .name = "ABGR8888",<br>
>               .format = formats::ABGR8888,<br>
> diff --git a/src/libcamera/v4l2_pixelformat.cpp b/src/libcamera/v4l2_pixelformat.cpp<br>
> index 166d93cf..78a08303 100644<br>
> --- a/src/libcamera/v4l2_pixelformat.cpp<br>
> +++ b/src/libcamera/v4l2_pixelformat.cpp<br>
> @@ -51,7 +51,8 @@ const std::map<V4L2PixelFormat, PixelFormat> vpf2pf{<br>
>       { V4L2PixelFormat(V4L2_PIX_FMT_RGB24), formats::BGR888 },<br>
>       { V4L2PixelFormat(V4L2_PIX_FMT_BGR24), formats::RGB888 },<br>
>       { V4L2PixelFormat(V4L2_PIX_FMT_XBGR32), formats::XRGB8888 },<br>
> -     { V4L2PixelFormat(V4L2_PIX_FMT_XRGB32), formats::XBGR8888 },<br>
> +     { V4L2PixelFormat(V4L2_PIX_FMT_XRGB32), formats::BGRX8888 },<br>
> +     { V4L2PixelFormat(V4L2_PIX_FMT_RGBX32), formats::XBGR8888 },<br>
>       { V4L2PixelFormat(V4L2_PIX_FMT_RGBA32), formats::ABGR8888 },<br>
>       { V4L2PixelFormat(V4L2_PIX_FMT_ABGR32), formats::ARGB8888 },<br>
>       { V4L2PixelFormat(V4L2_PIX_FMT_ARGB32), formats::BGRA8888 },<br>
> -- <br>
> 2.25.1<br>
> <br>
</blockquote></div>