drm/amd/display: Update RN/VGH active display count workaround
authorMichael Strauss <michael.strauss@amd.com>
Mon, 30 Nov 2020 17:14:00 +0000 (12:14 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 23 Dec 2020 20:02:06 +0000 (15:02 -0500)
[WHY]
Virtual signals were previously counted as a workaround to S0i2 hang
which is fixed on Renoir. This blocks S0i3 diags testing.

[HOW]
Stop counting virtual signals as S0i2 hang is fixed on Renoir.

Signed-off-by: Michael Strauss <michael.strauss@amd.com>
Acked-by: Bindu Ramamurthy <bindu.r@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/clk_mgr/dcn21/rn_clk_mgr.c
drivers/gpu/drm/amd/display/dc/clk_mgr/dcn301/vg_clk_mgr.c

index d00b025..9aa1b63 100644 (file)
@@ -75,15 +75,8 @@ int rn_get_active_display_cnt_wa(
        for (i = 0; i < dc->link_count; i++) {
                const struct dc_link *link = dc->links[i];
 
-               /*
-                * Only notify active stream or virtual stream.
-                * Need to notify virtual stream to work around
-                * headless case. HPD does not fire when system is in
-                * S0i2.
-                */
                /* abusing the fact that the dig and phy are coupled to see if the phy is enabled */
-               if (link->connector_signal == SIGNAL_TYPE_VIRTUAL ||
-                               link->link_enc->funcs->is_dig_enabled(link->link_enc))
+               if (link->link_enc->funcs->is_dig_enabled(link->link_enc))
                        display_count++;
        }
 
index 9a8e66b..991b9c5 100644 (file)
@@ -74,15 +74,8 @@ int vg_get_active_display_cnt_wa(
        for (i = 0; i < dc->link_count; i++) {
                const struct dc_link *link = dc->links[i];
 
-               /*
-                * Only notify active stream or virtual stream.
-                * Need to notify virtual stream to work around
-                * headless case. HPD does not fire when system is in
-                * S0i2.
-                */
                /* abusing the fact that the dig and phy are coupled to see if the phy is enabled */
-               if (link->connector_signal == SIGNAL_TYPE_VIRTUAL ||
-                               link->link_enc->funcs->is_dig_enabled(link->link_enc))
+               if (link->link_enc->funcs->is_dig_enabled(link->link_enc))
                        display_count++;
        }