From: Anthony Koo <[email protected]>

[Why]
Current implementation that maps link_rate_set value to
actual link rate is incorrect.

[How]
Fix this implementation, such that link_rate_set indexes into
the supported_link_rate table.

Change-Id: I5ff7f7e25951c442240698835526de1ff5400b89
Signed-off-by: Anthony Koo <[email protected]>
Reviewed-by: Harry Wentland <[email protected]>
Acked-by: Aric Cyr <[email protected]>
Acked-by: Bhawanpreet Lakha <[email protected]>
---
 .../gpu/drm/amd/display/dc/core/dc_link_dp.c  | 35 +------------------
 1 file changed, 1 insertion(+), 34 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 92f565ca1260..8ead8c4d575c 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
@@ -2538,7 +2538,6 @@ void detect_edp_sink_caps(struct dc_link *link)
        uint32_t entry;
        uint32_t link_rate_in_khz;
        enum dc_link_rate link_rate = LINK_RATE_UNKNOWN;
-       uint8_t link_rate_set = 0;
 
        retrieve_link_cap(link);
 
@@ -2558,39 +2557,7 @@ void detect_edp_sink_caps(struct dc_link *link)
                                link_rate = 
linkRateInKHzToLinkRateMultiplier(link_rate_in_khz);
                                if (link->reported_link_cap.link_rate < 
link_rate) {
                                        link->reported_link_cap.link_rate = 
link_rate;
-
-                                       switch (link_rate) {
-                                       case LINK_RATE_LOW:
-                                               link_rate_set = 1;
-                                               break;
-                                       case LINK_RATE_RATE_2:
-                                               link_rate_set = 2;
-                                               break;
-                                       case LINK_RATE_RATE_3:
-                                               link_rate_set = 3;
-                                               break;
-                                       case LINK_RATE_HIGH:
-                                               link_rate_set = 4;
-                                               break;
-                                       case LINK_RATE_RBR2:
-                                               link_rate_set = 5;
-                                               break;
-                                       case LINK_RATE_RATE_6:
-                                               link_rate_set = 6;
-                                               break;
-                                       case LINK_RATE_HIGH2:
-                                               link_rate_set = 7;
-                                               break;
-                                       case LINK_RATE_HIGH3:
-                                               link_rate_set = 8;
-                                               break;
-                                       default:
-                                               link_rate_set = 0;
-                                               break;
-                                       }
-
-                                       if (link->dpcd_caps.link_rate_set < 
link_rate_set)
-                                               link->dpcd_caps.link_rate_set = 
link_rate_set;
+                                       link->dpcd_caps.link_rate_set = entry;
                                }
                        }
                }
-- 
2.17.1

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to