On Tue, 2016-09-27 at 22:49 +0800, Xiang, Haihao wrote:
> User can update framerate using VAEncMiscParameterTypeFrameRate
> buffer later


I am okay with this.  lgtm, applied.

Thanks,

Sean

> 
> Signed-off-by: Xiang, Haihao <[email protected]>
> ---
>  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)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Libva mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libva

Reply via email to