[FFmpeg-devel] [PATCH] avutil: Add NV61 pixel format and enable IO support for NV16 and NV61 pixel formats

Alexander Strasser eclipse7 at gmx.net
Sat Jun 20 09:43:49 EEST 2020



Am 20. Juni 2020 00:23:53 MESZ schrieb Hendrik Leppkes <h.leppkes at gmail.com>:
>On Fri, Jun 19, 2020 at 9:58 PM Alexander Strasser <eclipse7 at gmx.net>
>wrote:
>>
>> How do others think about adding support for more pixel formats?
>>
>
>A new pixel format should present a clear improvement, a use-case you
>couldn't do before, or could only do with a performance penalty or
>whatever.

Thanks for your quick reply. Generally I agree here.

>If everything that you can do with NV61 you could also do with NV16, a
>format we already have, which value does adding it add to the project

In this case it's true, that you can't do anything special with NV61 you can't do with NV16, but I think they can also be viewed as the same format.

So if you add one, you also add the other. Like when I would add NV12 support I would also add NV21 support.

It has the added value, that whenever a user needs one or the other as input or output it would be available.

On the implementation side it is kind of easy to test, that both work the same with chroma interleaving swapped.

But maybe I misunderstand something important here?

>Completionism should not be a goal. There are hundreds of obscure
>pixel formats that we have no business all adding.

I fully agree that completionism should not be a goal.


  Alexander


More information about the ffmpeg-devel mailing list