From: Markus Elfring <elfr...@users.sourceforge.net>
Date: Fri, 28 Feb 2025 17:32:45 +0100

Replace nested max() calls by single max3() call in this
function implementation.

This issue was transformed by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net>
---
 drivers/gpu/drm/radeon/radeon_uvd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/radeon/radeon_uvd.c 
b/drivers/gpu/drm/radeon/radeon_uvd.c
index 058a1c8451b2..ded5747a58d1 100644
--- a/drivers/gpu/drm/radeon/radeon_uvd.c
+++ b/drivers/gpu/drm/radeon/radeon_uvd.c
@@ -961,7 +961,7 @@ int radeon_uvd_calc_upll_dividers(struct radeon_device 
*rdev,
        unsigned optimal_score = ~0;

        /* loop through vco from low to high */
-       vco_min = max(max(vco_min, vclk), dclk);
+       vco_min = max3(vco_min, vclk, dclk);
        for (vco_freq = vco_min; vco_freq <= vco_max; vco_freq += 100) {

                uint64_t fb_div = (uint64_t)vco_freq * fb_factor;
--
2.48.1

Reply via email to