Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: use color description for h264 and hevc

2020-10-28 Thread Zivkovic, Milos
Hello, Thanks for taking the time to review the patch. Regarding the VideoFullRange flag, it's actually set conditionally in the updated patch (not appearing on patchwork though). As for the ColourDescriptionPresent flag, I'll update the patch so it's set only if color_primaries, color_trc and co

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: use color description for h264 and hevc

2020-10-28 Thread Mark Thompson
On 10/10/2020 18:27, Zivkovic, Milos wrote: Hello, I've attached a patch that forwards the color description from AVCodecContext to mfxExtVideoSignalInfo. It has been discussed on the #ffmpeg and #ffmpeg-devel IRC channel a couple of times in the last few days. > From 724c8e64f9ca9356464f1596bb

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: use color description for h264 and hevc

2020-10-28 Thread Zivkovic, Milos
Hello, I've attached an updated patch that properly sets the mfxExtVideoSignalInfo::VideoFullRange flag. On Sat, Oct 10, 2020 at 7:27 PM Zivkovic, Milos wrote: > > Hello, > > I've attached a patch that forwards the color description from > AVCodecContext to mfxExtVideoSignalInfo. It has been dis

[FFmpeg-devel] [PATCH] avcodec/qsvenc: use color description for h264 and hevc

2020-10-10 Thread Zivkovic, Milos
Hello, I've attached a patch that forwards the color description from AVCodecContext to mfxExtVideoSignalInfo. It has been discussed on the #ffmpeg and #ffmpeg-devel IRC channel a couple of times in the last few days. 0001-avcodec-qsvenc-use-color-description-for-h264-and-hevc.patch Description: