During the discussion of the DP connectors, it was suggested that USB-C
DisplayPort connectors should have DRM_MODE_CONNECTOR_DisplayPort
connector type. This follows the example provided by other drivers
(AMDGPU, Intel). To distinguish them from native DP ports, they should
have the freshly defined USB as a subconnector type.

Suggested-by: Simon Ser <cont...@emersion.fr>
Cc: Neil Armstrong <neil.armstr...@linaro.org>
Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
---
 drivers/soc/qcom/pmic_glink_altmode.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/soc/qcom/pmic_glink_altmode.c 
b/drivers/soc/qcom/pmic_glink_altmode.c
index 1dedacc52aea..9094944c6cc0 100644
--- a/drivers/soc/qcom/pmic_glink_altmode.c
+++ b/drivers/soc/qcom/pmic_glink_altmode.c
@@ -417,7 +417,8 @@ static int pmic_glink_altmode_probe(struct auxiliary_device 
*adev,
                alt_port->bridge.funcs = &pmic_glink_altmode_bridge_funcs;
                alt_port->bridge.of_node = to_of_node(fwnode);
                alt_port->bridge.ops = DRM_BRIDGE_OP_HPD;
-               alt_port->bridge.type = DRM_MODE_CONNECTOR_USB;
+               alt_port->bridge.type = DRM_MODE_CONNECTOR_DisplayPort;
+               alt_port->bridge.subtype = DRM_MODE_SUBCONNECTOR_USB;
 
                ret = devm_drm_bridge_add(dev, &alt_port->bridge);
                if (ret)
-- 
2.39.2

Reply via email to