Re: [PATCH 2/3] drm/amdgpu: add psp ecc support

2017-07-30 Thread Huang Rui
On Fri, Jul 28, 2017 at 05:11:18PM +0800, Junwei Zhang wrote: > Signed-off-by: Junwei Zhang > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 17 ++--- > drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h | 22 ++ > 2 files changed, 36 insertions(+), 3 deletions(-) > > diff

Re: [PATCH 1/3] drm/amdgpu: add psp bootloader command list

2017-07-30 Thread Huang Rui
On Fri, Jul 28, 2017 at 05:11:17PM +0800, Junwei Zhang wrote: > Signed-off-by: Junwei Zhang > --- Reviewed-by: Huang Rui > drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h | 10 ++ > drivers/gpu/drm/amd/amdgpu/psp_v3_1.c | 4 ++-- > 2 files changed, 12 insertions(+), 2 deletions(-) > > diff

Re: [PATCH 1/3] drm/amdgpu: only move VM BOs in the LRU during validation v2

2017-07-30 Thread zhoucm1
The series is Acked-by: Chunming Zhou On 2017年07月29日 19:32, Christian König wrote: From: Christian König This should save us a bunch of command submission overhead. v2: move the LRU move to the right place to avoid the move for the root BO and handle the shadow BOs as well. This turned

Re: [PATCH] drm/amdgpu: Fix undue fallthroughs in golden registers initialization

2017-07-30 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Sun, Jul 30, 2017 at 10:18 AM, Jean Delvare wrote: > As I was staring at the si_init_golden_registers code, I noticed that > the Pitcairn initialization silently falls through the Cape Verde > initialization, and the Oland initialization falls through the Hain

[PATCH] drm/amdgpu: Fix dce_v6_0_disable_dce warning

2017-07-30 Thread Jean Delvare
Include a missing header to get rid of the following warning: drivers/gpu/drm/amd/amdgpu/dce_v6_0.c:521:6: warning: no previous prototype for 'dce_v6_0_disable_dce' [-Wmissing-prototypes] void dce_v6_0_disable_dce(struct amdgpu_device *adev) ^ Signed-off-by: Jean Delvare Cc: Alex Deucher

[PATCH] drm/amdgpu: Fix undue fallthroughs in golden registers initialization

2017-07-30 Thread Jean Delvare
As I was staring at the si_init_golden_registers code, I noticed that the Pitcairn initialization silently falls through the Cape Verde initialization, and the Oland initialization falls through the Hainan initialization. However there is no comment stating that this is intentional, and the radeon

[PATCH] drm/radeon: Make radeon_atif_handler static

2017-07-30 Thread Jean Delvare
There are no external users of function radeon_atif_handler so it can be static. Signed-off-by: Jean Delvare Cc: Alex Deucher Cc: "Christian König" --- drivers/gpu/drm/radeon/radeon_acpi.c |2 +- drivers/gpu/drm/radeon/radeon_acpi.h |3 --- 2 files changed, 1 insertion(+), 4 deletions(

[PATCH] drm/amdgpu: Fix amdgpu_pm_acpi_event_handler warning

2017-07-30 Thread Jean Delvare
Include a missing header to get rid of the following warning: drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c:65:6: warning: no previous prototype for ‘amdgpu_pm_acpi_event_handler’ [-Wmissing-prototypes] void amdgpu_pm_acpi_event_handler(struct amdgpu_device *adev) ^ Signed-off-by: Jean Delvare

[PATCH] drm/amdgpu: Make amdgpu_atif_handler static

2017-07-30 Thread Jean Delvare
There are no external users of function amdgpu_atif_handler so it can be static. Signed-off-by: Jean Delvare Cc: Alex Deucher Cc: "Christian König" --- drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-4.12.orig/drivers/gpu/drm/amd/amd