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

[Marc Hoffman]

Michael Niedermayer writes:
 > 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

Is this better?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: libswscale.patch
Type: text/x-patch
Size: 26142 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070505/e9fa250c/attachment.bin>