aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2016-07-16 09:05:45 -0400
committerAlex Deucher <alexander.deucher@amd.com>2016-07-29 14:36:54 -0400
commitb4fc5972c18d59b42d503d96147549a97784f912 (patch)
treec363477a64f5b655aa6c8a7bc3741d9512c95537
parenteb09d7a7c956f8c0edcdcf4eeb96f953587cd16e (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.c3
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;
1064free_obj: 1065free_obj:
1065 kfree(obj); 1066 kfree(obj);
1066free_input: 1067free_input: