[FFmpeg-devel] [RESEND][PATCH] libswscale: Extend the unaccelerated path of the unscaled yuv2rgb special converter with support for rgb444 output format

[Janusz Krzysztofik]

Tuesday 02 March 2010 12:56:14 Michael Niedermayer wrote:
> On Mon, Mar 01, 2010 at 02:09:56AM +0100, Janusz Krzysztofik wrote:
> > In order to play video on a 12-bit display, like OMAP1 frambuffer with
> > 12-bit LCD controller that my Amstrad Delta videophone is equipped with,
> > one or more libswscale converters with rgb444 output pixel format support
> > are required. This patch adds such support to the unaccelerated path of
> > the unscaled yuv2rgb special converter.
> >
> > Created and tested against ffmpeg svn revision 22124, on to off:
> > 	[PATCH v2] Add initial support for 12-bit color mode.
> >
> > Tested on Amstrad Delta with 24bpp IJPG and MP43 sourcers using patched
> > mplayer video output drivers.
> >
> > Signed-off-by: Janusz Krzysztofik <jkrzyszt at tis.icnet.pl>
> >
> > ---
> > Forgot to turn off line wrapping before sending it out, sorry.
> >
> > Thanks,
> > Janusz
>
> [...]
>
> > diff -upr trunk/libswscale/swscale.c trunk.rgb12/libswscale/swscale.c
> > --- trunk/libswscale/swscale.c	2010-02-24 15:55:47.000000000 +0100
> > +++ trunk.rgb12/libswscale/swscale.c	2010-02-25 11:30:58.000000000 +0100
> > @@ -23,11 +23,11 @@
> >
> >  /*
> >    supported Input formats: YV12, I420/IYUV, YUY2, UYVY, BGR32, BGR32_1, BGR24, BGR16, BGR15, RGB32, RGB32_1, RGB24, Y8/Y800, YVU9/IF09, PAL8
> > -  supported output formats: YV12, I420/IYUV, YUY2, UYVY, {BGR,RGB}{1,4,8,15,16,24,32}, Y8/Y800, YVU9/IF09
> > +  supported output formats: YV12, I420/IYUV, YUY2, UYVY, {BGR,RGB}{1,4,8,12,15,16,24,32}, Y8/Y800, YVU9/IF09
> >    {BGR,RGB}{1,4,8,15,16} support dithering 
>
> until the generic scaling code supports it it should not be listed as
> supported

OK, I'll drop this change for now.

> also it seems dithering is missing

To be sure: you didn't mean it's missing from the unscaled yuv2rgb special 
converter rgb444 output format support I'm trying to provide with this patch, 
did you?

Thanks,
Janusz