1 files changed, 43 insertions, 0 deletions
diff --git a/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2605-drm-amd-display-Small-comment-on-dc_commit_planes_to.patch b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2605-drm-amd-display-Small-comment-on-dc_commit_planes_to.patch
new file mode 100644
index 00000000..8ae27c56
--- /dev/null
+++ b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2605-drm-amd-display-Small-comment-on-dc_commit_planes_to.patch
@@ -0,0 +1,43 @@
+From 4b76df5fe38920b38208bf45bf01c97398fdbd9a Mon Sep 17 00:00:00 2001
+From: Harry Wentland <harry.wentland@amd.com>
+Date: Mon, 16 Oct 2017 20:10:00 -0400
+Subject: [PATCH 2605/4131] drm/amd/display: Small comment on
+ dc_commit_planes_to_stream
+
+Having one struct on the stack was somewhat confusing. Also mention
+that the whole function should eventually go.
+
+Signed-off-by: Harry Wentland <harry.wentland@amd.com>
+Reviewed-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
+---
+ drivers/gpu/drm/amd/display/dc/core/dc.c | 7 +++++++
+ 1 file changed, 7 insertions(+)
+
+diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
+index 03ecc3d..8c7ca78 100644
+--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
++++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
+@@ -918,6 +918,12 @@ bool dc_post_update_surfaces_to_stream(struct dc *dc)
+ 	return true;
+ }
+ 
++/*
++ * TODO this whole function needs to go
++ *
++ * dc_surface_update is needlessly complex. See if we can just replace this
++ * with a dc_plane_state and follow the atomic model a bit more closely here.
++ */
+ bool dc_commit_planes_to_stream(
+ 		struct dc *dc,
+ 		struct dc_plane_state **plane_states,
+@@ -925,6 +931,7 @@ bool dc_commit_planes_to_stream(
+ 		struct dc_stream_state *dc_stream,
+ 		struct dc_state *state)
+ {
++	/* no need to dynamically allocate this. it's pretty small */
+ 	struct dc_surface_update updates[MAX_SURFACES];
+ 	struct dc_flip_addrs *flip_addr;
+ 	struct dc_plane_info *plane_info;
+-- 
+2.7.4
+