[FFmpeg-devel] [PATCH 1/5] avutil: Add YUV444P10_LSB and YUV444P12_LSB pixel formats

[Philip Langdale]

On Mon, 8 Oct 2018 11:55:09 +0200
Hendrik Leppkes <h.leppkes at gmail.com> wrote:

> On Mon, Oct 8, 2018 at 10:40 AM Timo Rothenpieler
> <timo at rothenpieler.org> wrote:
> >
> >
> > I don't think it's YUVJ all over again, as it was the exact same bit
> > layout than normal YUV, while this one is actually different.
> >  
> 
> Its also the same bit layout as YUV444P16, with one piece of
> additional metadata tacked on top - seems quite similar to YUVJ to me.

So, this argument applies equally to P010 vs P016 doesn't it? Yet we
all think P010 is a completely reasonable format to have, and in fact
when I added it to cuvid/nvdec originally, I only did it as P016, and
we all agreed that it should explicitly handle P010.

I don't have a strong opinion on this one - I first wrote this patch as
YUV444P16 as well and Timo asked for the explicit formats - but I think
the consistency argument makes sense.

And yes, I messed up the names. I renamed all to _MSB and have updated
locally. Will repost after we're agreed on the principle.

Thanks,
 
--phil