diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2016-07-16 09:05:45 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-07-29 14:36:54 -0400 |
commit | b4fc5972c18d59b42d503d96147549a97784f912 (patch) | |
tree | c363477a64f5b655aa6c8a7bc3741d9512c95537 | |
parent | eb09d7a7c956f8c0edcdcf4eeb96f953587cd16e (diff) |
drm/amdgpu: Change assignment for a variable in amdgpu_cgs_acpi_eval_object()
Indicate successful function execution only at the end.
Thus omit initialisation for the variable "result" at the beginning.
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c index f88e82a989d9..d637b8fc7dce 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | |||
@@ -920,7 +920,7 @@ static int amdgpu_cgs_acpi_eval_object(struct cgs_device *cgs_device, | |||
920 | struct cgs_acpi_method_argument *argument; | 920 | struct cgs_acpi_method_argument *argument; |
921 | uint32_t i, count; | 921 | uint32_t i, count; |
922 | acpi_status status; | 922 | acpi_status status; |
923 | int result = 0; | 923 | int result; |
924 | 924 | ||
925 | handle = ACPI_HANDLE(&adev->pdev->dev); | 925 | handle = ACPI_HANDLE(&adev->pdev->dev); |
926 | if (!handle) | 926 | if (!handle) |
@@ -1061,6 +1061,7 @@ static int amdgpu_cgs_acpi_eval_object(struct cgs_device *cgs_device, | |||
1061 | params++; | 1061 | params++; |
1062 | } | 1062 | } |
1063 | 1063 | ||
1064 | result = 0; | ||
1064 | free_obj: | 1065 | free_obj: |
1065 | kfree(obj); | 1066 | kfree(obj); |
1066 | free_input: | 1067 | free_input: |