Commit 3e03525c authored by Xichao Zhao's avatar Xichao Zhao Committed by Alex Deucher
Browse files

drm/radeon: replace min/max nesting with clamp() 



The clamp() macro explicitly expresses the intent of constraining
a value within bounds.Therefore, replacing min(max(a, b), c) and
max(min(a,b),c) with clamp(val, lo, hi) can improve code readability.

Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarXichao Zhao <zhao.xichao@vivo.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent f101c13a

drivers/gpu/drm/radeon/radeon_display.c

+2 −2
Original line number Diff line number Diff line
@@ -926,10 +926,10 @@ static void avivo_get_fb_ref_div(unsigned nom, unsigned den, unsigned post_div, 
				 unsigned *fb_div, unsigned *ref_div)

{

	/* limit reference * post divider to a maximum */

	ref_div_max = max(min(100 / post_div, ref_div_max), 1u);

	ref_div_max = clamp(100 / post_div, 1u, ref_div_max);



	/* get matching reference and feedback divider */

	*ref_div = min(max(den/post_div, 1u), ref_div_max);

	*ref_div = clamp(den / post_div, 1u, ref_div_max);

	*fb_div = DIV_ROUND_CLOSEST(nom * *ref_div * post_div, den);



	/* limit fb divider to its maximum */