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 <[email protected]>
Sorry Yakui, didn't see your reply until after I reviewed and pushed. Thanks, Sean > > Thanks > Yakui > > > > > > > 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) > > _______________________________________________ > Libva mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/libva
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Libva mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libva
