drm/radeon/pm: Remove redundant ternary operators

LiaoYuanhong-vivo · alexdeucher · commit a055658793dd · 2025-09-05T17:38:40.000-04:00
For ternary operators in the form of "a ? true : false", if 'a' itself
returns a boolean result, the ternary operator can be omitted. Remove
redundant ternary operators to clean up the code.

Signed-off-by: Liao Yuanhong &lt;liaoyuanhong@vivo.com&gt;
Signed-off-by: Alex Deucher &lt;alexander.deucher@amd.com&gt;
diff --git a/drivers/gpu/drm/radeon/radeon_pm.c b/drivers/gpu/drm/radeon/radeon_pm.c
@@ -907,8 +907,7 @@ static void radeon_dpm_thermal_work_handler(struct work_struct *work)
 
 static bool radeon_dpm_single_display(struct radeon_device *rdev)
 {
-	bool single_display = (rdev->pm.dpm.new_active_crtc_count < 2) ?
-		true : false;
+	bool single_display = rdev->pm.dpm.new_active_crtc_count < 2;
 
 	/* check if the vblank period is too short to adjust the mclk */
 	if (single_display && rdev->asic->dpm.vblank_too_short) {

Original file line number	Diff line number	Diff line change
`@@ -907,8 +907,7 @@ static void radeon_dpm_thermal_work_handler(struct work_struct *work)`
`907`	`907`
`908`	`908`	`static bool radeon_dpm_single_display(struct radeon_device *rdev)`
`909`	`909`	`{`
`910`		`- bool single_display = (rdev->pm.dpm.new_active_crtc_count < 2) ?`
`911`		`- true : false;`
	`910`	`+ bool single_display = rdev->pm.dpm.new_active_crtc_count < 2;`
`912`	`911`
`913`	`912`	`/* check if the vblank period is too short to adjust the mclk */`
`914`	`913`	`if (single_display && rdev->asic->dpm.vblank_too_short) {`