[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 #38 from Joshua M. Doe <oss at nvl.army.mil> 2013-08-27 16:44:31 UTC ---
(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...

> In addition, I'd like to see the following things cleaned up in a follow up
> patch:
> [...]

Will do.

-- 
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.