[vlc-devel] [PATCH 08/48] fourcc: VDPAU surface types

[Rémi Denis-Courmont]

Le mercredi 3 juillet 2013 21:30:36, Rafaël Carré a écrit :
> Le 03/07/2013 20:08, Rémi Denis-Courmont a écrit :
> > Le mardi 2 juillet 2013 23:41:32, Rafaël Carré a écrit :
> >> Le 02/07/2013 19:51, Rémi Denis-Courmont a écrit :
> >>> VDPAU pictures have zero bits per pixel since they only carry picture
> >>> infos and handles to GPU resources, no actual pixel data. Nevertheless,
> >>> inclusion in chroma description is required for picture setup to work.
> >> 
> >> Good idea, this should be documented in vlc_fourcc.h though (the meaning
> >> of value 0 for pixel_size)
> > 
> > I don't know what the meaning of pixel_size is, so don't count on me to
> > document it.
> 
> Just document value 0 then.

The corresponding formats have zero pixel planes (retrospectively, the video 
filters patches should check the planes count rather than the bits per pixel).

pixel_bits and pixel_size are thus irrelevant mere placeholders. In fact, we 
could just as well put the same values as for the actual formats on the GPU, 
NV12 and UYVY. As I don't know the meaning the pixel_bits and pixel_size, I 
don't know what is correct.

-- 
Rémi Denis-Courmont
http://www.remlab.net/