dmr/amdgpu: Fix compile error with CONFIG_DRM_AMDGPU_GART_DEBUGFS

Submitted by Grodzovsky, Andrey on Aug. 14, 2019, 7:48 p.m.

Details

Message ID 1565812118-25941-1-git-send-email-andrey.grodzovsky@amd.com
State New
Headers show
Series "dmr/amdgpu: Fix compile error with CONFIG_DRM_AMDGPU_GART_DEBUGFS" ( rev: 1 ) in AMD X.Org drivers

Not browsing as part of any series.

Commit Message

Grodzovsky, Andrey Aug. 14, 2019, 7:48 p.m.
Double defintion of 'i'

Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
index a67ffff..f204dbd 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
@@ -309,7 +309,7 @@  int amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t offset,
 		     uint64_t flags)
 {
 #ifdef CONFIG_DRM_AMDGPU_GART_DEBUGFS
-	unsigned i,t,p;
+	unsigned t,p;
 #endif
 	int r, i;
 

Comments

Am 14.08.19 um 21:48 schrieb Andrey Grodzovsky:
> Double defintion of 'i'
>
> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

I think in the long term we could as well remove 
CONFIG_DRM_AMDGPU_GART_DEBUGFS and drop the extra debug functionality.

Christian.

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> index a67ffff..f204dbd 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> @@ -309,7 +309,7 @@ int amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t offset,
>   		     uint64_t flags)
>   {
>   #ifdef CONFIG_DRM_AMDGPU_GART_DEBUGFS
> -	unsigned i,t,p;
> +	unsigned t,p;
>   #endif
>   	int r, i;
>