From: Aric Cyr <[email protected]>

[why & how]
By moving bw_ctx field to the end of the dc_state the state can be
cleared more efficiently without resulting in large DML memcpy
operations, resulting in better mode enumeration performance on some
platforms.

Acked-by: Aurabindo Pillai <[email protected]>
Signed-off-by: Aric Cyr <[email protected]>
Reviewed-by: Nevenko Stupar <[email protected]>
---
 .../gpu/drm/amd/display/dc/inc/core_types.h    | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/inc/core_types.h 
b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
index 525f8f0b8732..b093ea495468 100644
--- a/drivers/gpu/drm/amd/display/dc/inc/core_types.h
+++ b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
@@ -546,15 +546,6 @@ struct dc_state {
         */
        struct resource_context res_ctx;
 
-       /**
-        * @bw_ctx: The output from bandwidth and watermark calculations and 
the DML
-        *
-        * Each context must have its own instance of VBA, and in order to
-        * initialize and obtain IP and SOC, the base DML instance from DC is
-        * initially copied into every context.
-        */
-       struct bw_context bw_ctx;
-
        /**
         * @pp_display_cfg: PowerPlay clocks and settings
         * Note: this is a big struct, do *not* put on stack!
@@ -569,6 +560,15 @@ struct dc_state {
 
        struct clk_mgr *clk_mgr;
 
+       /**
+        * @bw_ctx: The output from bandwidth and watermark calculations and 
the DML
+        *
+        * Each context must have its own instance of VBA, and in order to
+        * initialize and obtain IP and SOC, the base DML instance from DC is
+        * initially copied into every context.
+        */
+       struct bw_context bw_ctx;
+
        /**
         * @refcount: refcount reference
         *
-- 
2.39.0

Reply via email to