drm/amdgpu: Use ternary operator in vcn_v1_0_start()

paulmenzel · alexdeucher · commit 07d01469325c · 2022-03-15T15:01:12.000-04:00
Remove the boilerplate of declaring a variable and using an if else
statement by using the ternary operator.

Reviewed-by: James Zhu &lt;James.Zhu@amd.com&gt;
Signed-off-by: Paul Menzel &lt;pmenzel@molgen.mpg.de&gt;
Signed-off-by: Alex Deucher &lt;alexander.deucher@amd.com&gt;
diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c b/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
@@ -1102,13 +1102,8 @@ static int vcn_v1_0_start_dpg_mode(struct amdgpu_device *adev)
 
 static int vcn_v1_0_start(struct amdgpu_device *adev)
 {
-	int r;
-
-	if (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG)
-		r = vcn_v1_0_start_dpg_mode(adev);
-	else
-		r = vcn_v1_0_start_spg_mode(adev);
-	return r;
+	return (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG) ?
+		vcn_v1_0_start_dpg_mode(adev) : vcn_v1_0_start_spg_mode(adev);
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -1102,13 +1102,8 @@ static int vcn_v1_0_start_dpg_mode(struct amdgpu_device *adev)`
`1102`	`1102`
`1103`	`1103`	`static int vcn_v1_0_start(struct amdgpu_device *adev)`
`1104`	`1104`	`{`
`1105`		`- int r;`
`1106`		`-`
`1107`		`- if (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG)`
`1108`		`- r = vcn_v1_0_start_dpg_mode(adev);`
`1109`		`- else`
`1110`		`- r = vcn_v1_0_start_spg_mode(adev);`
`1111`		`- return r;`
	`1105`	`+ return (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG) ?`
	`1106`	`+ vcn_v1_0_start_dpg_mode(adev) : vcn_v1_0_start_spg_mode(adev);`
`1112`	`1107`	`}`
`1113`	`1108`
`1114`	`1109`	`/**`