On 08/12/16 01:13, Zhao Yakui wrote: > On 12/07/2016 01:48 PM, Xiang, Haihao wrote: >> Now we have the same way for frame rate for each codec in i965_encoder.c, it >> would be better to use the result directly > > Hi, Mark > > Can you refresh your patch based on Haihao's suggestion? > Now the frame rate is parsed in the function of > intel_encoder_check_framerate_parameter. (It is defined in i965_encoder.c) > and it is stored as framerate_per_100s. > So the vp9 can use the framerate_per_100s for the frame_rate_m. The > frame_rate_d can be 100.
Right, I went a bit further than that. The following removes all use of any of the RC parameter buffers (RC, HRD, framerate) from the specific encoders, moving top-level RC code into i965_encoder.c. Tested with VP8, VP9, H.264 and H.265 on Kaby Lake. I haven't tested the low-power H.264 encoder (gen9_vdenc.c) because it is only enabled for Skylake using CQP at the moment, though there are some edits there. Thanks, - Mark _______________________________________________ Libva mailing list Libva@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libva
