Thanks to the sanity check a few lines above: if (!h_total || !v_total || !pix_clk_khz)
and the computation done afterwards on these non 0 values, we know that 'pixels_per_second', 'pixels_per_frame' and 'refresh_rate' are not 0 The code can be simplified accordingly. Signed-off-by: Christophe JAILLET <christophe.jail...@wanadoo.fr> --- NOT compile tested. Because of some BROKEN in KConfig files. --- drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c b/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c index 091f0d68a045..eafe8561e55e 100644 --- a/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c +++ b/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c @@ -690,21 +690,8 @@ static uint32_t get_dmif_switch_time_us( pixels_per_second = pix_clk_khz * 1000; pixels_per_frame = h_total * v_total; - if (!pixels_per_second || !pixels_per_frame) { - /* avoid division by zero */ - ASSERT(pixels_per_frame); - ASSERT(pixels_per_second); - return single_frame_time_multiplier * min_single_frame_time_us; - } - refresh_rate = pixels_per_second / pixels_per_frame; - if (!refresh_rate) { - /* avoid division by zero*/ - ASSERT(refresh_rate); - return single_frame_time_multiplier * min_single_frame_time_us; - } - frame_time = us_in_sec / refresh_rate; if (frame_time < min_single_frame_time_us) -- 2.34.1