[pull] amdgpu drm-fixes-5.8

2020-07-01 Thread Alex Deucher
Hi Dave, Daniel, Fixes for 5.8. The following changes since commit 5b83c254571b26e6b6110019929dc9729e5c59d1: Merge tag 'amd-drm-fixes-5.8-2020-06-24' of git://people.freedesktop.org/~agd5f/linux into drm-fixes (2020-06-25 14:51:14 +1000) are available in the Git repository at:

Re: [PATCH] drm/amd/display: remove redundant initialization of variable result

2020-07-01 Thread Alex Deucher
On Wed, Jul 1, 2020 at 10:59 AM Colin King wrote: > > From: Colin Ian King > > The variable result is being initialized with a value that is never read > and it is being updated later with a new value. The initialization is > redundant and can be removed. > > Addresses-Coverity: ("Unused

Re: [PATCH] drm/amdgpu: use %u rather than %d for sclk/mclk

2020-07-01 Thread Nirmoy
Acked-by: Nirmoy Das On 7/1/20 6:02 PM, Alex Deucher wrote: Large clock values may overflow and show up as negative. Reported by prOMiNd on IRC. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] drm/amdgpu: use %u rather than %d for sclk/mclk

2020-07-01 Thread Alex Deucher
Large clock values may overflow and show up as negative. Reported by prOMiNd on IRC. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c

[pull] amdgpu, amdkfd, radeon drm-next-5.9

2020-07-01 Thread Alex Deucher
Hi Dave, Daniel, Same great flavor, now with 0.2% more S-o-b. The following changes since commit 9ca1f474cea0edc14a1d7ec933e5472c0ff115d3: Merge tag 'amd-drm-next-5.8-2020-05-27' of git://people.freedesktop.org/~agd5f/linux into drm-next (2020-05-28 16:10:17 +1000) are available in the Git

[PATCH] drm/amd/display: remove redundant initialization of variable result

2020-07-01 Thread Colin King
From: Colin Ian King The variable result is being initialized with a value that is never read and it is being updated later with a new value. The initialization is redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King ---

RE: [PATCH] drm/amdgpu: return an error for hw access in INFO ioctl when in reset

2020-07-01 Thread Li, Dennis
[AMD Official Use Only - Internal Distribution Only] Hi, Christian and Alex Not only amdgpu ioctls, but amdkfd ioctls also have the same issue. Best Regards Dennis Li -Original Message- From: amd-gfx On Behalf Of Christian König Sent: Wednesday, July 1, 2020 4:20 PM To: Alex

RE: [PATCH 1/3] drm/amdgpu/atomfirmware: fix vram_info fetching for renoir

2020-07-01 Thread Quan, Evan
[AMD Official Use Only - Internal Distribution Only] Series is acked-by: Evan Quan -Original Message- From: amd-gfx On Behalf Of Alex Deucher Sent: Friday, June 26, 2020 6:05 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander Subject: [PATCH 1/3] drm/amdgpu/atomfirmware: fix

Re: [PATCH] drm/amdgpu: return an error for hw access in INFO ioctl when in reset

2020-07-01 Thread Christian König
I don't think this is a good idea, we should probably rather wait for the GPU reset to finish by taking the appropriate lock. Christian. Am 01.07.20 um 07:33 schrieb Alex Deucher: ping? On Fri, Jun 26, 2020 at 10:04 AM Alex Deucher wrote: When the GPU is in reset, accessing the hw is