[vlc-devel] [PATCH] I420->RGB neon converter

[Rémi Denis-Courmont]

Le mardi 6 septembre 2011 23:44:55 XilasZ, vous avez écrit :
> Yes, YV12 should be easy, i think one vld2.u8 instead of two vld1.u8 would
> do the trick.

You're mixing YV12 with NV12 here.

> Of course i didn't used all arm neon optimization tricks, i'm new to arm,
> neon and asm, that's the first piece of asm i've ever wrote.
> I read about those the two pipelines and the bubbling, but i didn't care,
> digesting arm+neon+asm+converter at the same time was enough for a first
> try.
> Eventually that will come later, from me or another more experienced arm
> dev.

> RGB bitmasks are completely ignored, what could happen ?

The most obvious error is the RGB ends up in the wrong endianness. If you 
don't care about this corner case, you should simply fallback to swscale.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
http://fi.linkedin.com/in/remidenis