[Bug 761607] video: add support for P010 video format

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=761607

sreerenj <bsreerenj at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at coaxion.net

--- Comment #7 from sreerenj <bsreerenj at gmail.com> ---
(In reply to Scott D Phillips from comment #5)
> (In reply to sreerenj from comment #2)
> > I  think the values for GstVideoFormatInfo should be like this:
> > @bits = 16
> > @n_components = 3
> > @shift = add the shift array
> > @depth= 10, 10, 10, 0
> 
> bits=16 wouldn't be consistent with I420_10LE for example which also has 10
> bits of data in the low 16 bits of the component. So I guess something like
> this would be better:
> 
> #define DPTH10_10_10_HI   10, 3, { 6, 6, 6, 0 }, { 10, 10, 10, 0 }
> 
> (taking DPTH10_10_10 and putting 6 for the shifts)

I still think the @bits field should be 16...
Let's wait for opinion from other developers...
Adding slomo to CC..

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.