On 27.05.2019 23:49, Lance Wang wrote:
On Saturday, May 25, 2019, James Almer <jamr...@gmail.com> wrote:On 5/22/2019 3:59 AM, lance.lmw...@gmail.com wrote:From: Limin Wang <lance.lmw...@gmail.com> The testing command for the HDR10 output with nvenc: $ ./ffmpeg_g -y -i 4K.mp4 -c:v hevc_nvenc -g 7 -color_primaries bt2020-colorspace bt2020_ncl -color_trc smpte2084 -sei hdr10 \-master_display "G(13250,34500)B(7500,3000)R(34000,16000)WP(15635,16450)L(10000000,50)" -max_cll "0, 0" test.tsPlease notice it is preferable to use the frame sei side data thanmaster_display and max_cll paramters config--- libavcodec/nvenc.c | 129 ++++++++++++++++++++++++++++++++++++++++ libavcodec/nvenc.h | 18 ++++++ libavcodec/nvenc_hevc.c | 11 ++++ 3 files changed, 158 insertions(+) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 75dda6d689..3fd0eca4a5 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -22,6 +22,9 @@ #include "config.h" #include "nvenc.h" +#include "cbs_h265.h"This doesn't seem right. The encoder isn't using this framework at all. You're apparently including this only to get the H265RawSEIMasteringDisplayColourVolume and H265RawSEIContentLightLevelInfo structs, which you don't really need to fill sei_data[i].payload OK, I'll remove the dependent for the two structure and update the patch.
I'm still not convinced that all this belong in an encoder.Can it really not be made into a filter that adds it as extra data to frames? There got to be other frame consumers that use this kind of data.
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
