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

[Christian König]

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;