Commit 47d9c6fa authored by chiminghao's avatar chiminghao Committed by Alex Deucher

drm:amdgpu:remove unneeded variable

return value form directly instead of
taking this in another redundant variable.
Reported-by: default avatarZeal Robot <zealci@zte.com.cm>
Signed-off-by: default avatarchiminghao <chi.minghao@zte.com.cn>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent c08d2f8b
...@@ -37,12 +37,9 @@ ...@@ -37,12 +37,9 @@
long amdgpu_kms_compat_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) long amdgpu_kms_compat_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
{ {
unsigned int nr = DRM_IOCTL_NR(cmd); unsigned int nr = DRM_IOCTL_NR(cmd);
int ret;
if (nr < DRM_COMMAND_BASE) if (nr < DRM_COMMAND_BASE)
return drm_compat_ioctl(filp, cmd, arg); return drm_compat_ioctl(filp, cmd, arg);
ret = amdgpu_drm_ioctl(filp, cmd, arg); return amdgpu_drm_ioctl(filp, cmd, arg);
return ret;
} }
...@@ -1745,7 +1745,7 @@ static int gmc_v9_0_hw_init(void *handle) ...@@ -1745,7 +1745,7 @@ static int gmc_v9_0_hw_init(void *handle)
{ {
struct amdgpu_device *adev = (struct amdgpu_device *)handle; struct amdgpu_device *adev = (struct amdgpu_device *)handle;
bool value; bool value;
int r, i; int i;
/* The sequence of these two function calls matters.*/ /* The sequence of these two function calls matters.*/
gmc_v9_0_init_golden_registers(adev); gmc_v9_0_init_golden_registers(adev);
...@@ -1780,9 +1780,7 @@ static int gmc_v9_0_hw_init(void *handle) ...@@ -1780,9 +1780,7 @@ static int gmc_v9_0_hw_init(void *handle)
if (adev->umc.funcs && adev->umc.funcs->init_registers) if (adev->umc.funcs && adev->umc.funcs->init_registers)
adev->umc.funcs->init_registers(adev); adev->umc.funcs->init_registers(adev);
r = gmc_v9_0_gart_enable(adev); return gmc_v9_0_gart_enable(adev);
return r;
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment