<div dir="ltr">I'm fine with this change and it adds consistency.<div><br></div><div>lgtm</div><div><br></div><div>Thanks,</div><div><br></div><div>Sean<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 15, 2016 at 4:26 AM, Mark Thompson <span dir="ltr"><<a href="mailto:sw@jkqxz.net" target="_blank">sw@jkqxz.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 15/12/16 08:54, Xiang, Haihao wrote:<br>
><br>
> Hi Mark,<br>
><br>
> Thanks for your patch, is there any benefit to use a fraction? using<br>
> 100 as framerate_den works well in the driver.<br>
<br>
</span>Everything the driver interacts with directly, on both sides, uses a fraction:<br>
<br>
* In VAAPI, VAEncMiscParameterFrameRate contains a 16 / 16 fraction.<br>
<br>
* Newer hardware uses a fraction directly where it requires a framerate (gen9_vp9_encoder.c:1204, gen9_vdenc.c:1648).<br>
<br>
* H.264 and H.265 streams contain num_units_in_tick / time_scale.<br>
<br>
* gstreamer (GstVideoInfo.fps_[nd]), libyami (VideoFrameRate.frameRate{Num,<wbr>Denom}) and libavcodec (AVCodecContext.{time_base,<wbr>framerate} are fraction structures) all represent framerate as a fraction.<br>
<br>
The driver should use a fraction as well to preserve the exact value and be consistent with everything it interacts with.<br>
<br>
Do you have any other concerns about the patch?  If there are additional references outside the core driver code which I have missed somehow then I would be happy to update them as well.<br>
<br>
Thanks,<br>
<br>
- Mark<br>
<div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>_________________<br>
Libva mailing list<br>
<a href="mailto:Libva@lists.freedesktop.org">Libva@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libva" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/libva</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Sean V. Kelley <<a href="mailto:sean.v.kelley@intel.com" target="_blank">sean.v.kelley@intel.com</a>><br>Open Source Technology Center / SSG<br>Intel Corp.<br></div>
</div></div></div>