On 2025/8/19 15:58, Krzysztof Kozlowski wrote:
On Tue, Aug 19, 2025 at 11:33:29AM +0800, Yongxing Mou wrote:
Add compatible string for the DisplayPort controller found on the
Qualcomm QCS8300 SoC.

The Qualcomm QCS8300 platform comes with one DisplayPort controller
with same base offset as SM8650. But it requires new compatible string
because QCS8300 controller supports 4 MST streams. 4 MST streams will
be enabled as part of MST feature support. Currently, using SM8650 as
fallback to enable SST on QCS8300.

I don't think last sentence is true. Where in current code SM8650 is
used as fallback?

Sorry, this more like a description error. As Dmitry pointed out, we are not using SM8650 as a fallback; we are only using SM8650's data structure in the driver. I will correct the commit message in the next version. Thanks.

Signed-off-by: Yongxing Mou <yongxing....@oss.qualcomm.com>
---
  Documentation/devicetree/bindings/display/msm/dp-controller.yaml | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml 
b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
index 
68a6fd27506fda004e53174db5bcc88a29e8d2a6..ac44abfdd2853393ae199387c9ae2c37e1c48f52
 100644
--- a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
@@ -18,6 +18,7 @@ properties:
    compatible:
      oneOf:
        - enum:
+          - qcom,qcs8300-dp

...here not.
Emm, if we need new compatable, can we add qcom,qcs8300-dp here?
            - qcom,sa8775p-dp
            - qcom,sc7180-dp
            - qcom,sc7280-dp

Best regards,
Krzysztof


Reply via email to