[FFmpeg-devel] PATCH BlackFin YUV2 to RGB Color Space Converters

[Michael Niedermayer]

Hi

On Sat, May 05, 2007 at 06:33:57AM -0400, Marc Hoffman wrote:
Content-Description: message body text
> Diego Biurrun writes:
>  > On Sat, May 05, 2007 at 05:50:39AM -0400, Marc Hoffman wrote:
>  > Content-Description: message body text
>  > > Diego Biurrun writes:
>  > >  > On Sat, May 05, 2007 at 05:16:24AM -0400, Marc Hoffman wrote:
>  > >  > > Luca Barbato writes:
>  > >  > >  > Marc Hoffman wrote:
>  > >  > >  > > Blackfin optimized YUV420 to RGB CSC Color Space Converters.
> 
>  > (custom-set-variables '(show-trailing-whitespace t))
> 
> thanks, I've been going crazy with this space stuff.
[...]
> +static int bfin_yuv420_bgr565 (SwsContext *c,
> +                               unsigned char **in, int *instrides,
> +                               int srcSliceY, int srcSliceH,
> +                               unsigned char **oplanes, int *outstrides)
> +{
> +    c->rmask = 0x001f * 0x00010001U;
> +    c->gmask = 0x07e0 * 0x00010001U;
> +    c->bmask = 0xf800 * 0x00010001U;

the *mask init code could be moved into core_yuv420_rgb() which would avoid
it being duplicated


[...]

> +#ifdef ARCH_BFIN
> +    uint32_t oy,oc,zero,cy,crv,rmask,cbu,bmask,cgu,cgv,gmask;
> +#endif
> +
> +
>  } SwsContext;

this is unreadable

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070505/0517d04e/attachment.pgp>