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

[Sean V Kelley]

On Wed, 2016-09-28 at 09:04 +0800, Zhao Yakui wrote:
> 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>

Sorry Yakui, didn't see your reply until after I reviewed and pushed.

Thanks,

Sean

> 
> 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)
> 
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libva
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/libva/attachments/20161004/9163cde5/attachment.sig>