make video/x-bayer a first class gstreamer citizen

Nicolas Dufresne nicolas.dufresne at gmail.com
Fri Nov 25 13:40:18 UTC 2016


Le vendredi 25 novembre 2016 à 03:16 +0100, Philippe De Muyter a
écrit :
> Well, if we had GST_VIDEO_FORMAT_ constants for video/x-bayer
> formats,
> 
> in gst_video_info_from_caps, we could set format to a meaningfull
> format
> instead of GST_VIDEO_FORMAT_ENCODED,
> 
> and in fill_planes, we would not fall into
> 
>     case GST_VIDEO_FORMAT_ENCODED:
>       break;
> 
> which silently keeps size set to 0
> 
> There is probably more, but that would be a good start

Sure, but this is nothing we can merge upstream. You have to fill all
the fields and make sure they are fully usable. It's a lot more work.
Someone need to take ownership of this task.

Remember that using gst_video_info_parse_from_string() is not valid for
 anything non video/x-raw, which your code seems to do.

Nicolas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/gstreamer-devel/attachments/20161125/e2ae2d8b/attachment.sig>


More information about the gstreamer-devel mailing list