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