[Mesa-dev] [PATCH] radeon/vce Handle H.264 level 5.2

[Leo Liu]

Hi Andy,

Please rebase all your reviewed patches, and add RB, and send them to me 
I will commit them for you.

Cheers,

Leo



On 12/05/2016 03:14 PM, Andy Furniss wrote:
> Hi, I did a v2, can it be applied?
>
> https://patchwork.freedesktop.org/patch/118006/
>
> Christian König wrote:
>> Am 25.10.2016 um 00:39 schrieb Andy Furniss:
>>> For gstreamer
>>> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=91281
>>>
>>> Signed-off-by: Andy Furniss <adf.lists at gmail.com>
>>> ---
>>>   src/gallium/drivers/radeon/radeon_vce.c | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/src/gallium/drivers/radeon/radeon_vce.c
>>> b/src/gallium/drivers/radeon/radeon_vce.c
>>> index ef93e46..6d6d414 100644
>>> --- a/src/gallium/drivers/radeon/radeon_vce.c
>>> +++ b/src/gallium/drivers/radeon/radeon_vce.c
>>> @@ -178,13 +178,13 @@ static unsigned get_cpb_num(struct rvce_encoder
>>> *enc)
>>>       case 41:
>>>           dpb = 32768;
>>>           break;
>>> -    default:
>>>       case 42:
>>>           dpb = 34816;
>>>           break;
>>>       case 50:
>>>           dpb = 110400;
>>>           break;
>>> +    default:
>>>       case 51:
>>
>> Setting the default to the maximum is probably a good idea, but I would
>> also add an explicit "case 52:" here as well just for documentation
>> purposes.
>>
>> With that fixed the patch is Reviewed-by: Christian König
>> <christian.koenig at amd.com>.
>>
>> Regards,
>> Christian.
>>
>>>           dpb = 184320;
>>>           break;
>>
>>
>>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev