]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
drm/amd/display: Remove ASSERT if significance is zero in math_ceil2
authorRodrigo Siqueira <rodrigo.siqueira@amd.com>
Thu, 4 Jul 2024 17:54:34 +0000 (11:54 -0600)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 24 Jul 2024 21:30:23 +0000 (17:30 -0400)
In the DML math_ceil2 function, there is one ASSERT if the significance
is equal to zero. However, significance might be equal to zero
sometimes, and this is not an issue for a ceil function, but the current
ASSERT will trigger warnings in those cases. This commit removes the
ASSERT if the significance is equal to zero to avoid unnecessary noise.

Cc: Mario Limonciello <mario.limonciello@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
Reviewed-by: Chaitanya Dhere <chaitanya.dhere@amd.com>
Signed-off-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 332315885d3ccc6d8fe99700f3c2e4c24aa65ab7)

drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_standalone_libraries/lib_float_math.c

index defe13436a2c36d7c831eafc7f1dad553964a782..e73579f1a88e502d400f2537a3904d3dadbee852 100644 (file)
@@ -64,8 +64,6 @@ double math_ceil(const double arg)
 
 double math_ceil2(const double arg, const double significance)
 {
-       ASSERT(significance != 0);
-
        return ((int)(arg / significance + 0.99999)) * significance;
 }