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

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

Reply via email to