[PATCH 51/73] drm/amd/display: Loosen plane_info and scaling_info checks

[Harry Wentland]

From: Andrew Jiang <andrew.ji...@amd.com>

Make it so that differing dcc and plane size fields don't necessarily
result in a full update, along with upscaling modes. This allows us to
save some unnecessary full updates.

Change-Id: I5ea98fa690e8ed136d39e2de0af8cbe10806c4b4
Signed-off-by: Andrew Jiang <andrew.ji...@amd.com>
Reviewed-by: Tony Cheng <tony.ch...@amd.com>
Acked-by: Harry Wentland <harry.wentl...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 802aebaa2e11..de7332cffaa2 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -1130,9 +1130,7 @@ static enum surface_update_type 
get_plane_info_update_type(
        /* Full update parameters */
        temp_plane_info.color_space = u->surface->color_space;
        temp_plane_info.input_tf = u->surface->input_tf;
-       temp_plane_info.dcc = u->surface->dcc;
        temp_plane_info.horizontal_mirror = u->surface->horizontal_mirror;
-       temp_plane_info.plane_size = u->surface->plane_size;
        temp_plane_info.rotation = u->surface->rotation;
        temp_plane_info.stereo_format = u->surface->stereo_format;
 
@@ -1175,14 +1173,23 @@ static enum surface_update_type  
get_scaling_info_update_type(
        if (!u->scaling_info)
                return UPDATE_TYPE_FAST;
 
-       if (u->scaling_info->src_rect.width != u->surface->src_rect.width
-                       || u->scaling_info->src_rect.height != 
u->surface->src_rect.height
-                       || u->scaling_info->clip_rect.width != 
u->surface->clip_rect.width
+       if (u->scaling_info->clip_rect.width != u->surface->clip_rect.width
                        || u->scaling_info->clip_rect.height != 
u->surface->clip_rect.height
                        || u->scaling_info->dst_rect.width != 
u->surface->dst_rect.width
                        || u->scaling_info->dst_rect.height != 
u->surface->dst_rect.height)
                return UPDATE_TYPE_FULL;
 
+       if (u->scaling_info->src_rect.width != u->surface->src_rect.width
+               || u->scaling_info->src_rect.height != 
u->surface->src_rect.height) {
+
+               if (u->scaling_info->src_rect.width > u->surface->src_rect.width
+                               && u->scaling_info->src_rect.height > 
u->surface->src_rect.height)
+                       return UPDATE_TYPE_FULL;
+
+               /* Upscaling does not require a full update */
+               return UPDATE_TYPE_MED;
+       }
+
        if (u->scaling_info->src_rect.x != u->surface->src_rect.x
                        || u->scaling_info->src_rect.y != u->surface->src_rect.y
                        || u->scaling_info->clip_rect.x != 
u->surface->clip_rect.x
-- 
2.14.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx