On 28/11/17 07:46, Ruiling Song wrote:
> Signed-off-by: Ruiling Song <ruiling.s...@intel.com>
> ---
>  libavcodec/vaapi_encode_h265.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c
> index 3ae92a7..32b8bc6 100644
> --- a/libavcodec/vaapi_encode_h265.c
> +++ b/libavcodec/vaapi_encode_h265.c
> @@ -219,7 +219,7 @@ static int 
> vaapi_encode_h265_init_sequence_params(AVCodecContext *avctx)
>          .general_non_packed_constraint_flag = 1,
>          .general_frame_only_constraint_flag = 1,
>  
> -        .general_level_idc     = avctx->level,
> +        .general_level_idc     = avctx->level * 3,
>      };
>      
> vps->profile_tier_level.general_profile_compatibility_flag[avctx->profile & 
> 31] = 1;
>  
> 
The documentation has always said "profile and level set the values of 
general_profile_idc and general_level_idc respectively" 
(<http://ffmpeg.org/ffmpeg-codecs.html#VAAPI-encoders>) - the code disagreed 
for a while, but that was made consistent in 
00179664bccd1dd6fa0d1c40db453528757bf6f7.

Why do you want to change it to be general_level_idc / 3 instead?

Thanks,

- Mark
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to