Re: [PATCH] drm/radeon: fix "force the UVD DPB into VRAM as well"

2017-05-23 Thread Alex Deucher
On Thu, May 18, 2017 at 8:24 AM, Christian König
 wrote:
> From: Christian König 
>
> The DPB must be in VRAM, but not in the first segment.
>
> Signed-off-by: Christian König 
> Tested-by: Arthur Marsh 

Reviewed-by: Alex Deucher 

> ---
>  drivers/gpu/drm/radeon/radeon_uvd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_uvd.c 
> b/drivers/gpu/drm/radeon/radeon_uvd.c
> index fad4a11..0cd0e7b 100644
> --- a/drivers/gpu/drm/radeon/radeon_uvd.c
> +++ b/drivers/gpu/drm/radeon/radeon_uvd.c
> @@ -621,7 +621,7 @@ static int radeon_uvd_cs_reloc(struct radeon_cs_parser *p,
> }
>
> /* TODO: is this still necessary on NI+ ? */
> -   if ((cmd == 0 || cmd == 1 || cmd == 0x3) &&
> +   if ((cmd == 0 || cmd == 0x3) &&
> (start >> 28) != (p->rdev->uvd.gpu_addr >> 28)) {
> DRM_ERROR("msg/fb buffer %LX-%LX out of 256MB segment!\n",
>   start, end);
> --
> 2.7.4
>
> ___
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


[PATCH] drm/radeon: fix "force the UVD DPB into VRAM as well"

2017-05-18 Thread Christian König
From: Christian König 

The DPB must be in VRAM, but not in the first segment.

Signed-off-by: Christian König 
Tested-by: Arthur Marsh 
---
 drivers/gpu/drm/radeon/radeon_uvd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/radeon/radeon_uvd.c 
b/drivers/gpu/drm/radeon/radeon_uvd.c
index fad4a11..0cd0e7b 100644
--- a/drivers/gpu/drm/radeon/radeon_uvd.c
+++ b/drivers/gpu/drm/radeon/radeon_uvd.c
@@ -621,7 +621,7 @@ static int radeon_uvd_cs_reloc(struct radeon_cs_parser *p,
}
 
/* TODO: is this still necessary on NI+ ? */
-   if ((cmd == 0 || cmd == 1 || cmd == 0x3) &&
+   if ((cmd == 0 || cmd == 0x3) &&
(start >> 28) != (p->rdev->uvd.gpu_addr >> 28)) {
DRM_ERROR("msg/fb buffer %LX-%LX out of 256MB segment!\n",
  start, end);
-- 
2.7.4

___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx