[FFmpeg-devel] [PATCH] add yuva420p support

[Aurelien Jacobs]

On Mon, 24 Sep 2007 14:08:45 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:

> Hi
> 
> On Mon, Sep 24, 2007 at 01:48:22PM +0200, Aurelien Jacobs wrote:
> > Hi,
> > 
> > I'm currently implementing support for VP6A inside the vp6 codec. This
> > will output planar YUV with an additional alpha plan.
> > The attached patch add support for this new colorspace. For now this
> > colorspace is displayed and converted the same way as yuv420p
> > (ie. ignoring the alpha plan). Adding support for converting yuva420p
> > to yuv420p into libswscale, using the alpha plan information and
> > rendering on a uniform color background would be great, but is not
> > implemented for now.
> > 
> > Is this patch OK ?
> [...]
> > Index: libavutil/avutil.h
> > ===================================================================
> > --- libavutil/avutil.h	(revision 10550)
> > +++ libavutil/avutil.h	(working copy)
> > @@ -105,6 +105,7 @@
> >      PIX_FMT_GRAY16LE,  ///<        Y        , 16bpp, little-endian
> >      PIX_FMT_YUV440P,   ///< Planar YUV 4:4:0 (1 Cr & Cb sample per 1x2 Y samples)
> >      PIX_FMT_YUVJ440P,  ///< Planar YUV 4:4:0 full scale (jpeg)
> > +    PIX_FMT_YUVA420P,  ///< Planar YUV 4:2:0, 12bpp, (1 Cr & Cb sample per 2x2 Y samples) + one alpha plane (same resolution as Y plane)
> 
> its not 12bpp
> also id write (1 Cr & Cb sample per 2x2 Y & A samples)

Right.

> the rest of the patch looks ok

Applied with modified comment.

Aurel