[FFmpeg-devel] [PATCH] v210 decoder

[Baptiste Coudurier]

On 2/17/2009 10:38 AM, Reimar D?ffinger wrote:
> On Tue, Feb 17, 2009 at 06:53:08PM +0100, Michael Niedermayer wrote:
>> On Tue, Feb 17, 2009 at 06:26:01PM +0100, Reimar D?ffinger wrote:
>>> this is a decoder for
>>> http://samples.mplayerhq.hu/V-codecs/v210/v210_720p.avi
>>> it is not actually a proper codec, it is just little-endian
>>> UYVY with 10 bits per component and aligned to 32 bit every 3
>>> components.
>>> The last part makes it a bit awkward to deal with, which is also the
>>> reason why a favour a decoder instead of adding a pixfmt, IMHO if
>>> we want to support 10-bit pixfmts the planar or 444 variants would
>>> be enough and less painful to do...
>> I can accept codecs instead of pixfmts to deal with obscure packing
>> of pixels.
>> But i dont plan to accept code that always truncates 10 bit to 8
> 
> Well, from my point of view that's silly because hardly anyone cares the
> slightest about those extra 2 bits and thus I expect nobody in the foreseeable
> future will write pixfmt support for them (I just see someone in 2007
> gave up at that point)

Some people care, there is even some preliminary 10 bit support in DNxHD
decoder/encoder. I should be able to surprise you soon.

Besides, I agree with Michael that PIX_FMT_YUV422P16 should be added and
AVCodecContext->bits_per_raw_sample should be set to 10(or 30), this way
we could even support 'v216'.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org