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

[Andy Furniss]

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;
>
>
>