[Bug 607342] patch that Add 2 news plugins jp462bayer and bayer2rgb2

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=607342
  GStreamer | gst-plugins-bad | git

--- Comment #39 from Sebastian Dröge (slomo) <slomo at circular-chaos.org> 2013-08-30 10:21:42 UTC ---
(In reply to comment #38)
> (In reply to comment #36)
> > Nearly approved, pending resolution on caps:
> > 
> > +    GST_STATIC_CAPS ("video/x-bayer,format=(string){bggr,grbg,gbrg,rggb},"
> > +       
> > "width=(int)[1,MAX],height=(int)[1,MAX],framerate=(fraction)[0/1,MAX];"
> > +        "video/x-bayer,format=(string){bggr16,grbg16,gbrg16,rggb16},"
> > +        "bpp=(int){10,12,14,16},endianness={1234,4321},width=(int)[1,MAX],"
> > +        "height=(int)[1,MAX],framerate=(fraction)[0/1,MAX]")
> > 
> > In the time since this patch was first created, we switched to using format to
> > indicate endianness in audio/x-raw and video/x-raw.  I think should be changed
> > to match.
> 
> Do you want to keep Bayer in video/x-bayer, or put it in video/x-raw? I suppose
> the latter would suggest adding it to gstvideo, but it probably wouldn't fit
> well within that model.
> 
> To indicate endianness, we'd have format={BGGR16_BE, BGGR16_LE, GRBG16_BE,
> ...}, using upper case to match video/x-raw format, or all lower case to be
> consistent with existing video/x-bayer formats...

video/x-bayer and I guess lower-case for consistency with the existing bayer
formats... unfortunately we can't change them all to upper-case now

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.