[FFmpeg-devel] [PATCH v1 1/2] lavu/pixfmt: add new pixel format a2r10g10b10/a2b10g10r10

[Carl Eugen Hoyos]

Am Fr., 27. Sept. 2019 um 11:02 Uhr schrieb Sun, Xinpeng
<xinpeng.sun at intel.com>:

> > > Add two 10 bit RGBA pixel format for hardware color space conversion
> > > support in VAAPI and QSV:
> > >
> > > 2:10:10:10 10 bit: A2R10G10B10
> > > 2:10:10:10 10 bit: A2B10G10R10
> >
> > Without more explanation, this patch is not ok.

> The main reasons for adding these two format are as follows:
> 1. For most HDR monitors, A2R10G10B10 is used for display format for
> rendering. So this format is important to do 10bit RGB rendering support
> in ffmpeg.

For which operating systems (and video drivers) is this true?
And which video players will profit from this filter?

> 2. HW VPP can do both p010->a2r10g10b10 and a2r10g10b10->p010
> with this patch, which can provide support for hw encode pipeline
> using a2r10g10b10 as input.

But if the pipeline (that you control, no?) would support GBRP10, not
only one (very) specific use case would be supported but all thinkable
use cases, or do I misunderstand?

Afaict, the fact that FFmpeg cannot deal at all with HDR is the most
pressing issue we have atm. I believe that only solving this problem
for one specific use case is not the ideal solution.

Carl Eugen