[Libva] [Libva-intel-driver][PATCH] Set default framerate to 30fps if user doesn't set the corresponding sequence parameter

[Zhao Yakui]

On 09/27/2016 10:49 PM, Xiang, Haihao wrote:
> User can update framerate using VAEncMiscParameterTypeFrameRate buffer later

It looks good to me.

Add: Reviewed-by: Zhao Yakui <yakui.zhao at intel.com>

Thanks
    Yakui

>
> Signed-off-by: Xiang, Haihao<haihao.xiang at intel.com>
> ---
>   src/i965_encoder.c | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/src/i965_encoder.c b/src/i965_encoder.c
> index 7e44a5a..8587fd5 100644
> --- a/src/i965_encoder.c
> +++ b/src/i965_encoder.c
> @@ -312,7 +312,12 @@ intel_encoder_check_brc_h264_sequence_parameter(VADriverContextP ctx,
>
>       assert(seq_param);
>       bits_per_second = seq_param->bits_per_second; // for the highest layer
> -    framerate_per_100s = seq_param->time_scale * 100 / (2 * seq_param->num_units_in_tick); // for the highest layer
> +
> +    if (!seq_param->num_units_in_tick || !seq_param->time_scale)
> +        framerate_per_100s = 3000;
> +    else
> +        framerate_per_100s = seq_param->time_scale * 100 / (2 * seq_param->num_units_in_tick); // for the highest layer
> +
>       encoder_context->brc.num_iframes_in_gop = 1; // Always 1
>
>       if (seq_param->intra_period == 0) { // E.g. IDRPP... / IDR(PBB)... (no IDR/I any more)