drm/amd/display: Renoir chroma viewport WA change formula
authorJoseph Gravenor <joseph.gravenor@amd.com>
Thu, 17 Oct 2019 15:56:34 +0000 (11:56 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 19 Nov 2019 15:12:51 +0000 (10:12 -0500)
[why]
we want to increase the pte row plus 1 line if chroma viewport
height is integer multiple of the pte row height

[how]
instead of ceiling viewport height, we floor it. this allows
us to accommodate both cases: those where the chroma viewport
height is integer multiple of the pte row height and those where
it is not

Signed-off-by: Joseph Gravenor <joseph.gravenor@amd.com>
Reviewed-by: Tony Cheng <Tony.Cheng@amd.com>
Acked-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn21/dcn21_hubp.c

index 1ddd6ae..d86b6b6 100644 (file)
@@ -204,8 +204,8 @@ void hubp21_set_viewport(
                        PTE_ROW_HEIGHT_LINEAR, &pte_row_height);
 
                pte_row_height = 1 << (pte_row_height + 3);
-               pte_rows = (viewport_c->height + pte_row_height - 1) / pte_row_height;
-               patched_viewport_height = pte_rows * pte_row_height + 3;
+               pte_rows = (viewport_c->height / pte_row_height) + 1;
+               patched_viewport_height = pte_rows * pte_row_height + 1;
        }