[Libva] [PATCH] i965_device_info: add CBR and VBR rate control flags for vdenc

[Sreerenj]

On 11.08.2016 19:47, Sean V Kelley wrote:
> Hi Sree,
>
>
> On Thu, 2016-08-11 at 14:55 +0300, sreerenj.balachandran at intel.com wrote:
>> From: Sreerenj Balachandran <sreerenj.balachandran at intel.com <mailto:sreerenj.balachandran at intel.com>>
>>
>> Low power encode through VDENC supports both CBR and VBR.
>> Make sure vaGetConfigAttribute correctly returns the supported
>> list of rate control modes in VAEntrypointEncSliceLP entrypoint.
>
> This was intentional.  I'd prefer to delay this until we get the 
> GuC/HuC kernel patches merged.  BRC depends on the HuC firmware...

IIUC, HuC support is from BXT onwards, right?
Which means no BRC for SKL VDENC ??

>
> https://patchwork.freedesktop.org/series/9011/
>
> I'm also trying to get an interface (outside of debugfs) for the 
> driver to detect HuC FW loading
> status in the  revised/new patch set. We'll be reusing I915_GETPARAM 
> likely.
>
>
> Thanks,
>
> Sean
>
>> ---
>>   src/i965_device_info.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/i965_device_info.c b/src/i965_device_info.c
>> index 47fd50a..a3a1027 100644
>> --- a/src/i965_device_info.c
>> +++ b/src/i965_device_info.c
>> @@ -361,7 +361,7 @@ static struct hw_codec_info skl_hw_codec_info = {
>>       .has_hevc_encoding = 1,
>>       .has_lp_h264_encoding = 1,
>>   
>> -    .lp_h264_brc_mode = VA_RC_CQP,
>> +    .lp_h264_brc_mode = VA_RC_CQP | VA_RC_CBR | VA_RC_VBR,
>>   
>>       .num_filters = 5,
>>       .filters = {
>

-- 
Thanks
Sree

---------------------------------------------------------------------
Intel Finland Oy
Registered Address: PL 281, 00181 Helsinki 
Business Identity Code: 0357606 - 4 
Domiciled in Helsinki 

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libva/attachments/20160812/435a38a7/attachment-0001.html>