[x264-devel] [PATCHv3 1/1] nv21 input support

[Henrik Gramner]

Apologies for the slow reply.

I'm pretty sure that the plane width should NOT be right shifted by 1.
The chroma planes are indeed half-width, but they are packed together
which doubles the effective width, so your original code was probably
correct in that aspect. The same applies for the checkasm code. Did
you test this?

You could also add NV61 support while you're at it since it would be
trivial to do so.

Nits: keep NV12 before NV21 in x264.h and place NV21 directly below
NV12 in x264_frame_internal_csp.

On Thu, Oct 16, 2014 at 5:12 AM, Yu Xiaolei <dreifachstein at gmail.com> wrote:
> any comment on this patch?
>
> On 06/06/2014 04:05 PM, Yu Xiaolei wrote:
>> nv21 input support
>>
> _______________________________________________
> x264-devel mailing list
> x264-devel at videolan.org
> https://mailman.videolan.org/listinfo/x264-devel