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

[Andy Furniss]

Leo Liu wrote:
> Hi Andy,
>
> Please rebase all your reviewed patches, and add RB, and send them to me
> I will commit them for you.

Ok, thanks, I've sent two to 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
>
>