diff options
| author |   Dmitry Baryshkov <dmitry.baryshkov@linaro.org> | 2022-08-22 21:49:00 +0300 |
|---|---|---|
| committer |   Rob Clark <robdclark@chromium.org> | 2022-09-18 09:38:04 -0700 |
| commit | 122b17f32e35b9e7cae8feca258658d6ed5de4c3 (patch) | |
| tree | 18e0f8e319b5595728c6c87ecfaa03ba8d6cdc0b | |
| parent | dt-bindings: msm/dp: add missing properties (diff) | |
| download | wireguard-linux-122b17f32e35b9e7cae8feca258658d6ed5de4c3.tar.xz wireguard-linux-122b17f32e35b9e7cae8feca258658d6ed5de4c3.zip | |
dt-bindings: msm/dp: handle DP vs eDP difference
The #sound-dai-cells property should be used only for DP controllers. It
doesn't make sense for eDP, there is no support for audio output. The
aux-bus should not be used for DP controllers. Also p1 MMIO region
should be used only for DP controllers.
Take care of these differences.
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Patchwork: https://patchwork.freedesktop.org/patch/498751/
Link: https://lore.kernel.org/r/20220822184900.307160-4-dmitry.baryshkov@linaro.org
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Rob Clark <robdclark@chromium.org>
| -rw-r--r-- | Documentation/devicetree/bindings/display/msm/dp-controller.yaml | 26 |
1 files changed, 25 insertions, 1 deletions
diff --git a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml index 52cbf00df0ba..f2515af8256f 100644 --- a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml +++ b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml @@ -24,6 +24,7 @@ properties: - qcom,sm8350-dp reg: + minItems: 4 items: - description: ahb register block - description: aux register block @@ -112,10 +113,33 @@ required: - clock-names - phys - phy-names - - "#sound-dai-cells" - power-domains - ports +allOf: + # AUX BUS does not exist on DP controllers + # Audio output also is present only on DP output + # p1 regions is present on DP, but not on eDP + - if: + properties: + compatible: + contains: + enum: + - qcom,sc7280-edp + - qcom,sc8180x-edp + then: + properties: + "#sound-dai-cells": false + reg: + maxItems: 4 + else: + properties: + aux-bus: false + reg: + minItems: 5 + required: + - "#sound-dai-cells" + additionalProperties: false examples: |