<div dir="auto">Awesome, thanks! </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 13 Jul, 2021, 06:41 Laurent Pinchart, <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Vedant,<br>
<br>
Thank you for the patch.<br>
<br>
On Tue, Jul 06, 2021 at 07:03:46PM +0900, <a href="mailto:paul.elder@ideasonboard.com" target="_blank" rel="noreferrer">paul.elder@ideasonboard.com</a> wrote:<br>
> Hi Vedant,<br>
> <br>
> "Fixes: " doesn't go in the subject.<br>
> <br>
> I sent you this as an example :/<br>
> <br>
> <a href="https://git.linuxtv.org/libcamera.git/commit/?id=1684c3f930b2a27884037bc38856477b80cddd50" rel="noreferrer noreferrer" target="_blank">https://git.linuxtv.org/libcamera.git/commit/?id=1684c3f930b2a27884037bc38856477b80cddd50</a><br>
> <br>
> On Fri, Jul 02, 2021 at 05:38:49PM +0530, Vedant Paranjape wrote:<br>
> > This patch fixes V4L2 pixel format for formats::XBGR8888 by updating<br>
> > the existing matching to V4L2_PIX_FMT_RGBX32<br>
> <br>
> $ git blame src/libcamera/formats.cpp | grep XRGB32<br>
> 4fd6bb332 (Niklas Sölund 2020-09-30 16:10:07 +0200 201) .v4l2Format = V4L2PixelFormat(V4L2_PIX_FMT_XRGB32),<br>
> $ git show 4fd6bb332 --oneline | head -n 1<br>
> 4fd6bb33 libcamera: Add support for XRGB8888 and XBGR8888<br>
<br>
Tip: Add the following to you ~/.gitconfig<br>
<br>
[pretty]<br>
        fixes = Fixes: %h (\"%s\")<br>
<br>
which can also be done with<br>
<br>
git config --global --add pretty.fixes 'Fixes: %h ("%s")'<br>
<br>
You can then run<br>
<br>
git show --pretty=fixes -s 4fd6bb332<br>
<br>
With a bit more shell scripting, adding<br>
<br>
gpf() {<br>
        git show --pretty=fixes -s ${1:-HEAD} | xclip -selection clipboard<br>
}<br>
<br>
to your .bashrc, you can then type<br>
<br>
gpf 4fd6bb332<br>
<br>
and you'll get the Fixes: line in your clipboard, ready to be pasted.<br>
<br>
> So right here,<br>
> <br>
> Fixes: 4fd6bb33 ("libcamera: Add support for XRGB8888 and XBGR8888")<br>
> <br>
> > Signed-off-by: Vedant Paranjape <<a href="mailto:vedantparanjape160201@gmail.com" target="_blank" rel="noreferrer">vedantparanjape160201@gmail.com</a>><br>
> <br>
> Other than that, looks good.<br>
> <br>
> Reviewed-by: Paul Elder <<a href="mailto:paul.elder@ideasonboard.com" target="_blank" rel="noreferrer">paul.elder@ideasonboard.com</a>><br>
<br>
With the above fixed,<br>
<br>
Reviewed-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank" rel="noreferrer">laurent.pinchart@ideasonboard.com</a>><br>
<br>
> > ---<br>
> >  src/libcamera/formats.cpp          | 2 +-<br>
> >  src/libcamera/v4l2_pixelformat.cpp | 2 +-<br>
> >  2 files changed, 2 insertions(+), 2 deletions(-)<br>
> > <br>
> > diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp<br>
> > index 59a34853..578ac4af 100644<br>
> > --- a/src/libcamera/formats.cpp<br>
> > +++ b/src/libcamera/formats.cpp<br>
> > @@ -198,7 +198,7 @@ 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>
> > diff --git a/src/libcamera/v4l2_pixelformat.cpp b/src/libcamera/v4l2_pixelformat.cpp<br>
> > index 87d728fe..59a01e02 100644<br>
> > --- a/src/libcamera/v4l2_pixelformat.cpp<br>
> > +++ b/src/libcamera/v4l2_pixelformat.cpp<br>
> > @@ -52,7 +52,7 @@ 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_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>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>