From: "Mario Limonciello (AMD)" <[email protected]>

[WHY]
There is already a switch/case looking at link->link_id.id.

[How]
Move the case of `CONNECTOR_ID_PCIE` into switch case.

Reviewed-by: Alex Hung <[email protected]>
Signed-off-by: Mario Limonciello (AMD) <[email protected]>
Signed-off-by: Alex Hung <[email protected]>
---
 drivers/gpu/drm/amd/display/dc/link/link_detection.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/link/link_detection.c 
b/drivers/gpu/drm/amd/display/dc/link/link_detection.c
index e0402a04f25f..778049b45c8f 100644
--- a/drivers/gpu/drm/amd/display/dc/link/link_detection.c
+++ b/drivers/gpu/drm/amd/display/dc/link/link_detection.c
@@ -195,12 +195,6 @@ static enum signal_type 
link_detect_sink_signal_type(struct dc_link *link,
         * eDP connector supports only eDP signal;
         * HDMI should check straps for audio
         */
-
-       /* PCIE detects the actual connector on add-on board */
-       if (link->link_id.id == CONNECTOR_ID_PCIE) {
-               /* ZAZTODO implement PCIE add-on card detection */
-       }
-
        switch (link->link_id.id) {
        case CONNECTOR_ID_HDMI_TYPE_A:
                /* check audio support:
@@ -225,6 +219,9 @@ static enum signal_type link_detect_sink_signal_type(struct 
dc_link *link,
                                result = SIGNAL_TYPE_DVI_SINGLE_LINK;
                }
                break;
+       case CONNECTOR_ID_PCIE:
+               /* ZAZTODO implement PCIE add-on card detection */
+               break;
        default:
                break;
        }
-- 
2.43.0

Reply via email to