[FFmpeg-devel] [PATCH] Nvidia NVENC 10-bit HEVC encoding and rate control lookahead support

[Carl Eugen Hoyos]

Hi!

2016-08-23 19:10 GMT+02:00 Oliver Collyer <ovcollyer at mac.com>:
> +    AV_PIX_FMT_YUV420P10LE,

I know this is theoretical but the Nvidia header seems to indicate
native endianness to me, so this should be AV_PIX_FMT_YUV420P10

> +    AV_PIX_FMT_YUV444P10LE

But after reading the rest of the patch:
Shouldn't this be AV_PIX_FMT_YUV444P16?

And instead of YUV420P10, shouldn't you use P010LE?

In any case, please split the rate control patch from the 10bit patch.

Carl Eugen

who wonders now how the Microsoft headers define the ten bit
yuv420 semi-planar format...