[PATCH 09/12] drm/amd/display: Revert "Guard ASSR with internal display flag"

[Bindu Ramamurthy]

From: Stylon Wang <stylon.w...@amd.com>

[Why]
1. Previous patch regresses on some embedded panels.
2. Project coreboot doesn't support passing of internal display flag.

[How]
This reverts "Guard ASSR with internal display flag" commit.

Signed-off-by: Stylon Wang <stylon.w...@amd.com>
Reviewed-by: Wesley Chalmers <wesley.chalm...@amd.com>
Acked-by: Bindu Ramamurthy <bind...@amd.com>
---
 .../gpu/drm/amd/display/dc/core/dc_link_dp.c  | 58 ++++++-------------
 1 file changed, 17 insertions(+), 41 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
index 4326ac577756..10b749ef7fbb 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
@@ -1760,42 +1760,6 @@ enum link_training_result 
dc_link_dp_perform_link_training(
        return status;
 }
 
-static enum dp_panel_mode try_enable_assr(struct dc_stream_state *stream)
-{
-       struct dc_link *link = stream->link;
-       enum dp_panel_mode panel_mode = dp_get_panel_mode(link);
-#ifdef CONFIG_DRM_AMD_DC_HDCP
-       struct cp_psp *cp_psp = &stream->ctx->cp_psp;
-#endif
-
-       /* ASSR must be supported on the panel */
-       if (panel_mode == DP_PANEL_MODE_DEFAULT)
-               return panel_mode;
-
-       /* eDP or internal DP only */
-       if (link->connector_signal != SIGNAL_TYPE_EDP &&
-               !(link->connector_signal == SIGNAL_TYPE_DISPLAY_PORT &&
-                link->is_internal_display))
-               return DP_PANEL_MODE_DEFAULT;
-
-#ifdef CONFIG_DRM_AMD_DC_HDCP
-       if (cp_psp && cp_psp->funcs.enable_assr) {
-               if (!cp_psp->funcs.enable_assr(cp_psp->handle, link)) {
-                       /* since eDP implies ASSR on, change panel
-                        * mode to disable ASSR
-                        */
-                       panel_mode = DP_PANEL_MODE_DEFAULT;
-               }
-       } else
-               panel_mode = DP_PANEL_MODE_DEFAULT;
-
-#else
-       /* turn off ASSR if the implementation is not compiled in */
-       panel_mode = DP_PANEL_MODE_DEFAULT;
-#endif
-       return panel_mode;
-}
-
 bool perform_link_training_with_retries(
        const struct dc_link_settings *link_setting,
        bool skip_video_pattern,
@@ -1808,7 +1772,7 @@ bool perform_link_training_with_retries(
        uint8_t delay_between_attempts = LINK_TRAINING_RETRY_DELAY;
        struct dc_stream_state *stream = pipe_ctx->stream;
        struct dc_link *link = stream->link;
-       enum dp_panel_mode panel_mode;
+       enum dp_panel_mode panel_mode = dp_get_panel_mode(link);
        struct link_encoder *link_enc;
        enum link_training_result status = LINK_TRAINING_CR_FAIL_LANE0;
        struct dc_link_settings current_setting = *link_setting;
@@ -1845,11 +1809,23 @@ bool perform_link_training_with_retries(
                        msleep(delay_dp_power_up_in_ms);
                }
 
-               panel_mode = try_enable_assr(stream);
+#ifdef CONFIG_DRM_AMD_DC_HDCP
+               if (panel_mode == DP_PANEL_MODE_EDP) {
+                       struct cp_psp *cp_psp = &stream->ctx->cp_psp;
+
+                       if (cp_psp && cp_psp->funcs.enable_assr) {
+                               if (!cp_psp->funcs.enable_assr(cp_psp->handle, 
link)) {
+                                       /* since eDP implies ASSR on, change 
panel
+                                        * mode to disable ASSR
+                                        */
+                                       panel_mode = DP_PANEL_MODE_DEFAULT;
+                               }
+                       } else
+                               panel_mode = DP_PANEL_MODE_DEFAULT;
+               }
+#endif
+
                dp_set_panel_mode(link, panel_mode);
-               DC_LOG_DETECTION_DP_CAPS("Link: %d ASSR enabled: %d\n",
-                        link->link_index,
-                        panel_mode != DP_PANEL_MODE_DEFAULT);
 
                if (link->aux_access_disabled) {
                        dc_link_dp_perform_link_training_skip_aux(link, 
&current_setting);
-- 
2.25.1

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