[PATCH v3] gstreamer: Implement caps negotiation for video/x-bayer

Kieran Bingham kieran.bingham at ideasonboard.com
Mon Nov 4 16:14:20 CET 2024


Quoting Kieran Bingham (2024-11-04 15:12:51)
> Quoting Kieran Bingham (2024-11-04 15:02:28)
> > Quoting Nicolas Dufresne (2024-11-04 14:58:01)
> > > Thanks.
> > > 
> > > Le lundi 04 novembre 2024 à 12:36 +0100, mailinglist1 at johanneskirchmair.de a
> > > écrit :
> > > > From: Johannes Kirchmair <johannes.kirchmair at skidata.com>
> > > > 
> > > > The parsing of video/x-bayer sources from string makes it possible to
> > > > use cameras providing e.g SGRBG8 streams via gst-launch.
> > > > 
> > > > Like:
> > > > gst-launch-1.0 libcamerasrc camera-name=<cam> ! video/x-bayer,format=grbg
> > > > 
> > > > Without this change the gstreamer plugin complains about "Unsupported
> > > > media type: video/x-bayer".
> > > > 
> > > > Signed-off-by: Johannes Kirchmair <johannes.kirchmair at skidata.com>
> > > 
> > > Reviewed-by: Nicolas Dufresne <nicolas.dufresne at collabora.com>
> > 
> > 
> > Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> > 
> > 
> > I'll merge this assuming the CI tests all pass cleanly.
> 
> https://gitlab.freedesktop.org/camera/libcamera/-/jobs/66072263 reports:
> 
> ```
> 5c8010e4f8dc2d4e41ce817c03289b1155ccd659 gstreamer: Implement caps negotiation for video/x-bayer
> ------------------------------------------------------------------------------------------------
> No 'Signed-off-by' trailer matching author 'Johannes Kirchmair <johannes.kirchmair at skidata.com>', see Documentation/contributing.rst
> --- src/gstreamer/gstlibcamera-utils.cpp
> +++ src/gstreamer/gstlibcamera-utils.cpp
> @@ -283,7 +283,7 @@
>  static const gchar *
>  bayer_format_to_string(PixelFormat format)
>  {
> -       for (auto &b: bayer_map) {
> +       for (auto &b : bayer_map) {
>                 if (b.format == format)
>                         return b.name;
>         }
> @@ -293,7 +293,7 @@
>  static PixelFormat
>  bayer_format_from_string(const gchar *name)
>  {
> -       for (auto &b: bayer_map) {
> +       for (auto &b : bayer_map) {
>                 if (strcmp(b.name, name) == 0)
>                         return b.format;
>         }
> ---
> 3 potential issues detected, please review
> Cleaning up project directory and file based variables 00:02
> ERROR: Job failed: exit code 1
> ```
> 
> The no Signed-off-by is odd - as I can see it there, so I'll check that.
> Ah - it looks like the patch was missing the '---' separator before the
> changelog. I'll fix that.
> 
> I'll also add the appropriate spaces when merging...
> 
> Johannes, for future patches, please add the post-commit hook to your
> libcamera repository to run the checkstyle locally:

Argh, I did it again. Someone pinged me halfway through sending a mail
and I didn't complete it!

 - To add checkstyle locally, please run:

 cp utils/hooks/post-commit .git/hooks/post-commit

from the top of your libcamera tree.


--
Kieran


More information about the libcamera-devel mailing list