Commit 342d4821 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Dmitry Baryshkov
Browse files

dt-bindings: display/msm: merge SM8550 DPU into SC7280 



Split of the bindings was artificial and not helping - we end up with
multiple binding files for very similar devices thus increasing the
chances of using different order of reg and clocks entries.

Unify DPU bindings of SC7280 and SM8550, because they are the same.

Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: default avatarRob Herring (Arm) <robh@kernel.org>
Patchwork: https://patchwork.freedesktop.org/patch/617874/
Link: https://lore.kernel.org/r/20241003-dt-binding-display-msm-merge-v1-5-91ab08fc76a2@linaro.org


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent abf0a19c

Documentation/devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml

+2 −0
Original line number Diff line number Diff line
@@ -8,6 +8,7 @@ title: Qualcomm Display DPU on SC7280 


maintainers:

  - Bjorn Andersson <andersson@kernel.org>

  - Neil Armstrong <neil.armstrong@linaro.org>

  - Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

  - Krishna Manikandan <quic_mkrishn@quicinc.com>
@@ -20,6 +21,7 @@ properties: 
      - qcom,sc8280xp-dpu

      - qcom,sm8350-dpu

      - qcom,sm8450-dpu

      - qcom,sm8550-dpu



  reg:

    items:

Documentation/devicetree/bindings/display/msm/qcom,sm8550-dpu.yaml

deleted100644 → 0
+0 −133
Original line number Diff line number Diff line 
# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause

%YAML 1.2

---

$id: http://devicetree.org/schemas/display/msm/qcom,sm8550-dpu.yaml#

$schema: http://devicetree.org/meta-schemas/core.yaml#



title: Qualcomm SM8550 Display DPU



maintainers:

  - Neil Armstrong <neil.armstrong@linaro.org>



$ref: /schemas/display/msm/dpu-common.yaml#



properties:

  compatible:

    const: qcom,sm8550-dpu



  reg:

    items:

      - description: Address offset and size for mdp register set

      - description: Address offset and size for vbif register set



  reg-names:

    items:

      - const: mdp

      - const: vbif



  clocks:

    items:

      - description: Display AHB

      - description: Display hf axi

      - description: Display MDSS ahb

      - description: Display lut

      - description: Display core

      - description: Display vsync



  clock-names:

    items:

      - const: bus

      - const: nrt_bus

      - const: iface

      - const: lut

      - const: core

      - const: vsync



required:

  - compatible

  - reg

  - reg-names

  - clocks

  - clock-names



unevaluatedProperties: false



examples:

  - |

    #include <dt-bindings/clock/qcom,sm8550-dispcc.h>

    #include <dt-bindings/clock/qcom,sm8550-gcc.h>

    #include <dt-bindings/interrupt-controller/arm-gic.h>

    #include <dt-bindings/power/qcom,rpmhpd.h>



    display-controller@ae01000 {

        compatible = "qcom,sm8550-dpu";

        reg = <0x0ae01000 0x8f000>,

              <0x0aeb0000 0x2008>;

        reg-names = "mdp", "vbif";



        clocks = <&gcc GCC_DISP_AHB_CLK>,

                <&gcc GCC_DISP_HF_AXI_CLK>,

                <&dispcc DISP_CC_MDSS_AHB_CLK>,

                <&dispcc DISP_CC_MDSS_MDP_LUT_CLK>,

                <&dispcc DISP_CC_MDSS_MDP_CLK>,

                <&dispcc DISP_CC_MDSS_VSYNC_CLK>;

        clock-names = "bus",

                      "nrt_bus",

                      "iface",

                      "lut",

                      "core",

                      "vsync";



        assigned-clocks = <&dispcc DISP_CC_MDSS_VSYNC_CLK>;

        assigned-clock-rates = <19200000>;



        operating-points-v2 = <&mdp_opp_table>;

        power-domains = <&rpmhpd RPMHPD_MMCX>;



        interrupt-parent = <&mdss>;

        interrupts = <0>;



        ports {

            #address-cells = <1>;

            #size-cells = <0>;



            port@0 {

                reg = <0>;

                dpu_intf1_out: endpoint {

                    remote-endpoint = <&dsi0_in>;

                };

            };



            port@1 {

                reg = <1>;

                dpu_intf2_out: endpoint {

                    remote-endpoint = <&dsi1_in>;

                };

            };

        };



        mdp_opp_table: opp-table {

            compatible = "operating-points-v2";



            opp-200000000 {

                opp-hz = /bits/ 64 <200000000>;

                required-opps = <&rpmhpd_opp_low_svs>;

            };



            opp-325000000 {

                opp-hz = /bits/ 64 <325000000>;

                required-opps = <&rpmhpd_opp_svs>;

            };



            opp-375000000 {

                opp-hz = /bits/ 64 <375000000>;

                required-opps = <&rpmhpd_opp_svs_l1>;

            };



            opp-514000000 {

                opp-hz = /bits/ 64 <514000000>;

                required-opps = <&rpmhpd_opp_nom>;

            };

        };

    };

...