Re: [libav-devel] [Libav-devel][Patch] Force high_444 profile for 444 inputs for H264

2016-10-07 Thread Luca Barbato
On 06/10/16 13:21, Yogender Gupta wrote: >>> I'd fold it in the current check for ctx->data_pix_fmt that is >>> already present. > I kept it separate to maintain consistency as it is set for HEVC > I see what you mean, I guess the patch can go in as is then. lu

Re: [libav-devel] [Libav-devel][Patch] Force high_444 profile for 444 inputs for H264

2016-10-06 Thread Yogender Gupta
>> I'd fold it in the current check for ctx->data_pix_fmt that is already >> present. I kept it separate to maintain consistency as it is set for HEVC >> Currently we are setting FF_PROFILE_H264_HIGH_444 should it be changed to >> FF_PROFILE_H264_HIGH_444_PREDICTIVE as well?

Re: [libav-devel] [Libav-devel][Patch] Force high_444 profile for 444 inputs for H264

2016-10-06 Thread Luca Barbato
On 06/10/16 11:10, Yogender Gupta wrote: > +if (ctx->data_pix_fmt == AV_PIX_FMT_YUV444P) { > +cc->profileGUID = NV_ENC_H264_PROFILE_HIGH_444_GUID; > +avctx->profile = FF_PROFILE_H264_HIGH_444_PREDICTIVE; > +} I'd fold it in the current check for ctx->data_pix_fmt that is

[libav-devel] [Libav-devel][Patch] Force high_444 profile for 444 inputs for H264

2016-10-06 Thread Yogender Gupta
This patches forces high_444 profile selection for yuv444p inputs for nvenc_h264. Thanks, Yogender --- This email message is for the sole use of the intended recipient(s) and may contain confidential information.