[Mesa-dev] [PATCH] st/va encode handle ntsc framerate rate control

[Andy Furniss]

Mark Thompson wrote:
> On 27/01/17 22:06, Andy Furniss wrote:

>> Could use floats I guess like I did here.
>>
>> https://cgit.freedesktop.org/mesa/mesa/commit/?id=a5993022275c20061ac025d9adc26c5f9d02afee
>>
>> I don't know what the preference is between float and int.
>
> I think float is fine here?  I don't know what the preference is either...  (uint64_t would also be big enough, I think.)
>
> As an alternative source of inspiration, <https://cgit.freedesktop.org/vaapi/intel-driver/tree/src/gen6_mfc_common.c#n93> is full of doubles ;)

I sent a float v2 patch to the list.

Float vs double didn't seem to make any difference at normal bitrates so 
I just used float.