[FFmpeg-devel] [PATCH] v210 decoder
Tue Feb 17 20:16:10 CET 2009
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
>>> 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
>>> 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
More information about the ffmpeg-devel