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.li...@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.koe...@amd.com>.

Regards,
Christian.

                dpb = 184320;
                break;


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to