make video/x-bayer a first class gstreamer citizen

[Philippe De Muyter]

On Thu, Nov 24, 2016 at 08:39:18PM -0500, Nicolas Dufresne wrote:
> Le vendredi 25 novembre 2016 à 02:17 +0100, Philippe De Muyter a
> écrit :
> > 
> > Could video/x-bayer,format=bggr be replaced by video/x-
> > raw,format=bggr
> >  (or BGGR) ?
> 
> It's hard to say if this can be changed now or not. Probably that the
> old video/x-bayer would need to stay. Meanwhile, someone need to study
> the libgstvideo library and find what may not fit, or what is needed to
> make it fit. We managed to retrofit tile formats few years ago. With
> enough effort, it could be possible to do the same for bayer.

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

best regards

Philippe

-- 
Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles