[FFmpeg-devel] [PATCH v4 2/2] avfilter: Add tonemap vaapi filter

Xiang, Haihao haihao.xiang at intel.com
Mon Oct 28 02:29:21 EET 2019


On Wed, 2019-10-23 at 08:26 +0200, Carl Eugen Hoyos wrote:
> 
> Am 23.10.2019 um 06:48 schrieb Xiang, Haihao <haihao.xiang at intel.com>:
> 
> > > > On 11/09/2019 06:39, Zachary Zhou wrote:
> > > > It supports ICL platform.
> > > > H2H (HDR to HDR): P010 -> A2R10G10B10
> > > > H2S (HDR to SDR): P010 -> ARGB
> > > 
> > > The input format doesn't have any alpha so the output shouldn't
> > > either.  Not
> > > sure what the first case wants, but the second should be AV_PIX_FMT_0RGB
> > > (or
> > > some swizzle thereof).
> > Usually the default alpha is taken as 1.0 when alpha is not specified, so
> > how
> > about setting alpha to 1.0 too when doing P010-> A2R10G10B10/ARGB in this
> > filter? On the other hand, we need to send the output of tonemapping to
> > HDR/SDR
> > display and the HW supports A2R10G10B10/ARGB instead of 0R10G10B10/0RGB.
> 
> The HDR->SDR output should be 0RGB (or one of its friends), not ARGB.

The recent driver can support P010 output for H2H and NV12 output for H2S. I
think we may update this patch to support YUV output instead of RGB output.
What's your opinion? @Carl @Mark

Thanks
Haihao


> 
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list