On 27/03/2024 09:52, Dmitry Baryshkov wrote: > On Wed, 27 Mar 2024 at 10:45, Krzysztof Kozlowski > <krzysztof.kozlowski@xxxxxxxxxx> wrote: >> >> On 26/03/2024 21:02, Dmitry Baryshkov wrote: >>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8150-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8150-mdss.yaml >>> index c0d6a4fdff97..40b077fb20aa 100644 >>> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm8150-mdss.yaml >>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8150-mdss.yaml >>> @@ -53,6 +53,16 @@ patternProperties: >>> compatible: >>> const: qcom,sm8150-dpu >>> >>> + "^displayport-controller@[0-9a-f]+$": >>> + type: object >>> + additionalProperties: true >>> + >>> + properties: >>> + compatible: >>> + items: >>> + - const: qcom,sm8150-dp >>> + - const: qcom,sm8350-dp >> >> This does not look right. sm8350 has its own mdss binding file. > > So just a single entry here, even though SM8150 uses fallback compat string? Ah, wait, I misread, I thought it is enum. Best regards, Krzysztof