Document the MDSS hardware found on the Qualcomm QCS8300 platform.
Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com>
---
.../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++
1 file changed, 284 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml
new file mode 100644
index 0000000000000000000000000000000000000000..ae4bc16395326bffd6c9eff92778d9f207209526
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml
@@ -0,0 +1,284 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/msm/qcom,qcs8300-mdss.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Qualcomm Technologies, Inc. QCS8300 Display MDSS
+
+maintainers:
+ - Yongxing Mou <quic_yongmou@quicinc.com>
+
+description:
+ QCS8300 MSM Mobile Display Subsystem(MDSS), which encapsulates sub-blocks like
+ DPU display controller, DP interfaces and EDP etc.
+
+$ref: /schemas/display/msm/mdss-common.yaml#
+
+properties:
+ compatible:
+ const: qcom,qcs8300-mdss
+
+ clocks:
+ items:
+ - description: Display AHB
+ - description: Display hf AXI
+ - description: Display core
+
+ iommus:
+ maxItems: 1
+
+ interconnects:
+ maxItems: 3
+
+ interconnect-names:
+ maxItems: 3
+
+patternProperties:
+ "^display-controller@[0-9a-f]+$":
+ type: object
+ additionalProperties: true
+
+ properties:
+ compatible:
+ items:
+ - const: qcom,qcs8300-dpu
+ - const: qcom,sa8775p-dpu
+
+ "^displayport-controller@[0-9a-f]+$":
+ type: object
+ additionalProperties: true
+
+ properties:
+ compatible:
+ items:
+ - const: qcom,qcs8300-dp
+
+ "^phy@[0-9a-f]+$":
+ type: object
+ additionalProperties: true
+ properties:
+ compatible:
+ items:
+ - const: qcom,qcs8300-edp-phy
+ - const: qcom,sa8775p-edp-phy
+
+required:
+ - compatible
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interconnect/qcom,icc.h>
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+ #include <dt-bindings/clock/qcom,qcs8300-gcc.h>
+ #include <dt-bindings/clock/qcom,sa8775p-dispcc.h>
+ #include <dt-bindings/interconnect/qcom,qcs8300-rpmh.h>
+ #include <dt-bindings/power/qcom,rpmhpd.h>
+ #include <dt-bindings/power/qcom-rpmpd.h>
+
+ mdss: display-subsystem@ae00000 {
+ compatible = "qcom,qcs8300-mdss";
+ reg = <0x0ae00000 0x1000>;
+ reg-names = "mdss";
+
+ interconnects = <&mmss_noc MASTER_MDP0 QCOM_ICC_TAG_ACTIVE_ONLY
+ &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ACTIVE_ONLY>,
+ <&mmss_noc MASTER_MDP1 QCOM_ICC_TAG_ACTIVE_ONLY
+ &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ACTIVE_ONLY>,
+ <&gem_noc MASTER_APPSS_PROC QCOM_ICC_TAG_ACTIVE_ONLY
+ &config_noc SLAVE_DISPLAY_CFG QCOM_ICC_TAG_ACTIVE_ONLY>;
+ interconnect-names = "mdp0-mem",
+ "mdp1-mem",
+ "cpu-cfg";
+
+ resets = <&dispcc_core_bcr>;
+ power-domains = <&dispcc_gdsc>;
+
+ clocks = <&dispcc_ahb_clk>,
+ <&gcc GCC_DISP_HF_AXI_CLK>,
+ <&dispcc_mdp_clk>;
+
+ interrupts = <GIC_SPI 92 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-controller;
+ #interrupt-cells = <1>;
+
+ iommus = <&apps_smmu 0x1000 0x402>;
+
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ display-controller@ae01000 {
+ compatible = "qcom,qcs8300-dpu", "qcom,sa8775p-dpu";
+ reg = <0x0ae01000 0x8f000>,
+ <0x0aeb0000 0x2008>;
+ reg-names = "mdp", "vbif";
+
+ clocks = <&gcc GCC_DISP_HF_AXI_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_AHB_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_MDP_LUT_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_MDP_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_VSYNC_CLK>;
+ clock-names = "nrt_bus",
+ "iface",
+ "lut",
+ "core",
+ "vsync";
+
+ assigned-clocks = <&dispcc0 MDSS_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_intf0_out: endpoint {
+ remote-endpoint = <&mdss_dp0_in>;
+ };
+ };
+ };
+
+ mdp_opp_table: opp-table {
+ compatible = "operating-points-v2";
+
+ opp-375000000 {
+ opp-hz = /bits/ 64 <375000000>;
+ required-opps = <&rpmhpd_opp_svs_l1>;
+ };
+
+ opp-500000000 {
+ opp-hz = /bits/ 64 <500000000>;
+ required-opps = <&rpmhpd_opp_nom>;
+ };
+
+ opp-575000000 {
+ opp-hz = /bits/ 64 <575000000>;
+ required-opps = <&rpmhpd_opp_turbo>;
+ };
+
+ opp-650000000 {
+ opp-hz = /bits/ 64 <650000000>;
+ required-opps = <&rpmhpd_opp_turbo_l1>;
+ };
+ };
+ };
+
+ mdss_dp0_phy: phy@aec2a00 {
+ compatible = "qcom,qcs8300-edp-phy", "qcom,sa8775p-edp-phy";
+
+ reg = <0x0aec2a00 0x200>,
+ <0x0aec2200 0xd0>,
+ <0x0aec2600 0xd0>,
+ <0x0aec2000 0x1c8>;
+
+ clocks = <&dispcc MDSS_DISP_CC_MDSS_DPTX0_AUX_CLK>,
+ <&dispcc MDSS_DISP_CC_MDSS_AHB_CLK>;
+ clock-names = "aux",
+ "cfg_ahb";
+
+ #clock-cells = <1>;
+ #phy-cells = <0>;
+
+ vdda-phy-supply = <&vreg_l1c>;
+ vdda-pll-supply = <&vreg_l4a>;
+ };
+
+ displayport-controller@af54000 {
+ compatible = "qcom,qcs8300-dp";
+
+ pinctrl-0 = <&dp_hot_plug_det>;
+ pinctrl-names = "default";
+
+ reg = <0xaf54000 0x104>,
+ <0xaf54200 0x0c0>,
+ <0xaf55000 0x770>,
+ <0xaf56000 0x09c>,
+ <0xaf57000 0x09c>;
+
+ interrupt-parent = <&mdss>;
+ interrupts = <12>;
+ clocks = <&dispcc0 MDSS_DISP_CC_MDSS_AHB_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_AUX_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_LINK_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_LINK_INTF_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL0_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL1_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL2_CLK>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL3_CLK>;
+ clock-names = "core_iface",
+ "core_aux",
+ "ctrl_link",
+ "ctrl_link_iface",
+ "stream_pixel",
+ "stream_1_pixel",
+ "stream_2_pixel",
+ "stream_3_pixel";
+ assigned-clocks = <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_LINK_CLK_SRC>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL0_CLK_SRC>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL1_CLK_SRC>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL2_CLK_SRC>,
+ <&dispcc0 MDSS_DISP_CC_MDSS_DPTX0_PIXEL3_CLK_SRC>;
+ assigned-clock-parents = <&mdss_dp0_phy 0>,
+ <&mdss_dp0_phy 1>,
+ <&mdss_dp0_phy 1>,
+ <&mdss_dp0_phy 1>;
+ phys = <&mdss_dp0_phy>;
+ phy-names = "dp";
+ operating-points-v2 = <&dp_opp_table>;
+ power-domains = <&rpmhpd RPMHPD_MMCX>;
+
+ #sound-dai-cells = <0>;
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+
+ mdss_dp0_in: endpoint {
+ remote-endpoint = <&dpu_intf0_out>;
+ };
+ };
+
+ port@1 {
+ reg = <1>;
+
+ mdss_dp_out: endpoint { };
+ };
+ };
+
+ dp_opp_table: opp-table {
+ compatible = "operating-points-v2";
+
+ opp-160000000 {
+ opp-hz = /bits/ 64 <160000000>;
+ required-opps = <&rpmhpd_opp_low_svs>;
+ };
+
+ opp-270000000 {
+ opp-hz = /bits/ 64 <270000000>;
+ required-opps = <&rpmhpd_opp_svs>;
+ };
+
+ opp-540000000 {
+ opp-hz = /bits/ 64 <540000000>;
+ required-opps = <&rpmhpd_opp_svs_l1>;
+ };
+
+ opp-810000000 {
+ opp-hz = /bits/ 64 <810000000>;
+ required-opps = <&rpmhpd_opp_nom>;
+ };
+ };
+ };
+ };
+...
--
2.34.1On Wed, Jul 30, 2025 at 05:42:28PM +0800, Yongxing Mou wrote: > Document the MDSS hardware found on the Qualcomm QCS8300 platform. > > Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> > --- > .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ > 1 file changed, 284 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml > new file mode 100644 > index 0000000000000000000000000000000000000000..ae4bc16395326bffd6c9eff92778d9f207209526 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml > @@ -0,0 +1,284 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/msm/qcom,qcs8300-mdss.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm Technologies, Inc. QCS8300 Display MDSS > + > +maintainers: > + - Yongxing Mou <quic_yongmou@quicinc.com> > + > +description: > + QCS8300 MSM Mobile Display Subsystem(MDSS), which encapsulates sub-blocks like > + DPU display controller, DP interfaces and EDP etc. > + > +$ref: /schemas/display/msm/mdss-common.yaml# > + > +properties: > + compatible: > + const: qcom,qcs8300-mdss > + > + clocks: > + items: > + - description: Display AHB > + - description: Display hf AXI > + - description: Display core > + > + iommus: > + maxItems: 1 > + > + interconnects: > + maxItems: 3 > + > + interconnect-names: > + maxItems: 3 > + > +patternProperties: > + "^display-controller@[0-9a-f]+$": > + type: object > + additionalProperties: true > + > + properties: > + compatible: > + items: > + - const: qcom,qcs8300-dpu > + - const: qcom,sa8775p-dpu Use contains: instead of listing both of them > + > + "^displayport-controller@[0-9a-f]+$": > + type: object > + additionalProperties: true > + > + properties: > + compatible: > + items: > + - const: qcom,qcs8300-dp > + > + "^phy@[0-9a-f]+$": > + type: object > + additionalProperties: true > + properties: > + compatible: > + items: > + - const: qcom,qcs8300-edp-phy > + - const: qcom,sa8775p-edp-phy Use contains: instead of listing both of them > + > +required: > + - compatible > + > +unevaluatedProperties: false > + -- With best wishes Dmitry
On 2025/8/2 17:56, Dmitry Baryshkov wrote: > On Wed, Jul 30, 2025 at 05:42:28PM +0800, Yongxing Mou wrote: >> Document the MDSS hardware found on the Qualcomm QCS8300 platform. >> >> Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> >> --- >> .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ >> 1 file changed, 284 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml >> new file mode 100644 >> index 0000000000000000000000000000000000000000..ae4bc16395326bffd6c9eff92778d9f207209526 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml >> @@ -0,0 +1,284 @@ >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/display/msm/qcom,qcs8300-mdss.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Qualcomm Technologies, Inc. QCS8300 Display MDSS >> + >> +maintainers: >> + - Yongxing Mou <quic_yongmou@quicinc.com> >> + >> +description: >> + QCS8300 MSM Mobile Display Subsystem(MDSS), which encapsulates sub-blocks like >> + DPU display controller, DP interfaces and EDP etc. >> + >> +$ref: /schemas/display/msm/mdss-common.yaml# >> + >> +properties: >> + compatible: >> + const: qcom,qcs8300-mdss >> + >> + clocks: >> + items: >> + - description: Display AHB >> + - description: Display hf AXI >> + - description: Display core >> + >> + iommus: >> + maxItems: 1 >> + >> + interconnects: >> + maxItems: 3 >> + >> + interconnect-names: >> + maxItems: 3 >> + >> +patternProperties: >> + "^display-controller@[0-9a-f]+$": >> + type: object >> + additionalProperties: true >> + >> + properties: >> + compatible: >> + items: >> + - const: qcom,qcs8300-dpu >> + - const: qcom,sa8775p-dpu > > Use contains: instead of listing both of them > Got it, thanks~ here we want to fallback qcs8300 to sa8775p. Should we update it to : + contains: + enum: + - qcom,qcs8300-dpu + - qcom,sa8775p-dpu above method can be validated using dt_binding_check. >> + >> + "^displayport-controller@[0-9a-f]+$": >> + type: object >> + additionalProperties: true >> + >> + properties: >> + compatible: >> + items: >> + - const: qcom,qcs8300-dp >> + >> + "^phy@[0-9a-f]+$": >> + type: object >> + additionalProperties: true >> + properties: >> + compatible: >> + items: >> + - const: qcom,qcs8300-edp-phy >> + - const: qcom,sa8775p-edp-phy > > Use contains: instead of listing both of them > Same as above >> + >> +required: >> + - compatible >> + >> +unevaluatedProperties: false >> + >
On Mon, Aug 04, 2025 at 12:31:31PM +0800, Yongxing Mou wrote:
>
>
> On 2025/8/2 17:56, Dmitry Baryshkov wrote:
> > On Wed, Jul 30, 2025 at 05:42:28PM +0800, Yongxing Mou wrote:
> > > Document the MDSS hardware found on the Qualcomm QCS8300 platform.
> > >
> > > Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com>
> > > ---
> > > .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++
> > > 1 file changed, 284 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml
> > > new file mode 100644
> > > index 0000000000000000000000000000000000000000..ae4bc16395326bffd6c9eff92778d9f207209526
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml
> > > @@ -0,0 +1,284 @@
> > > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> > > +%YAML 1.2
> > > +---
> > > +$id: http://devicetree.org/schemas/display/msm/qcom,qcs8300-mdss.yaml#
> > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > +
> > > +title: Qualcomm Technologies, Inc. QCS8300 Display MDSS
> > > +
> > > +maintainers:
> > > + - Yongxing Mou <quic_yongmou@quicinc.com>
> > > +
> > > +description:
> > > + QCS8300 MSM Mobile Display Subsystem(MDSS), which encapsulates sub-blocks like
> > > + DPU display controller, DP interfaces and EDP etc.
> > > +
> > > +$ref: /schemas/display/msm/mdss-common.yaml#
> > > +
> > > +properties:
> > > + compatible:
> > > + const: qcom,qcs8300-mdss
> > > +
> > > + clocks:
> > > + items:
> > > + - description: Display AHB
> > > + - description: Display hf AXI
> > > + - description: Display core
> > > +
> > > + iommus:
> > > + maxItems: 1
> > > +
> > > + interconnects:
> > > + maxItems: 3
> > > +
> > > + interconnect-names:
> > > + maxItems: 3
> > > +
> > > +patternProperties:
> > > + "^display-controller@[0-9a-f]+$":
> > > + type: object
> > > + additionalProperties: true
> > > +
> > > + properties:
> > > + compatible:
> > > + items:
> > > + - const: qcom,qcs8300-dpu
> > > + - const: qcom,sa8775p-dpu
> >
> > Use contains: instead of listing both of them
> >
> Got it, thanks~ here we want to fallback qcs8300 to sa8775p.
> Should we update it to :
> + contains:
> + enum:
> + - qcom,qcs8300-dpu
> + - qcom,sa8775p-dpu
>
> above method can be validated using dt_binding_check.
_instead_ of listing both of them
So:
contains:
const: qcom,qcs8300-dpu
> > > +
> > > + "^displayport-controller@[0-9a-f]+$":
> > > + type: object
> > > + additionalProperties: true
> > > +
> > > + properties:
> > > + compatible:
> > > + items:
> > > + - const: qcom,qcs8300-dp
> > > +
> > > + "^phy@[0-9a-f]+$":
> > > + type: object
> > > + additionalProperties: true
> > > + properties:
> > > + compatible:
> > > + items:
> > > + - const: qcom,qcs8300-edp-phy
> > > + - const: qcom,sa8775p-edp-phy
> >
> > Use contains: instead of listing both of them
> >
> Same as above
> > > +
> > > +required:
> > > + - compatible
> > > +
> > > +unevaluatedProperties: false
> > > +
> >
>
--
With best wishes
Dmitry
On 2025/8/4 12:52, Dmitry Baryshkov wrote: > On Mon, Aug 04, 2025 at 12:31:31PM +0800, Yongxing Mou wrote: >> >> >> On 2025/8/2 17:56, Dmitry Baryshkov wrote: >>> On Wed, Jul 30, 2025 at 05:42:28PM +0800, Yongxing Mou wrote: >>>> Document the MDSS hardware found on the Qualcomm QCS8300 platform. >>>> >>>> Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> >>>> --- >>>> .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ >>>> 1 file changed, 284 insertions(+) >>>> >>>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml >>>> new file mode 100644 >>>> index 0000000000000000000000000000000000000000..ae4bc16395326bffd6c9eff92778d9f207209526 >>>> --- /dev/null >>>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.yaml >>>> @@ -0,0 +1,284 @@ >>>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >>>> +%YAML 1.2 >>>> +--- >>>> +$id: http://devicetree.org/schemas/display/msm/qcom,qcs8300-mdss.yaml# >>>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>>> + >>>> +title: Qualcomm Technologies, Inc. QCS8300 Display MDSS >>>> + >>>> +maintainers: >>>> + - Yongxing Mou <quic_yongmou@quicinc.com> >>>> + >>>> +description: >>>> + QCS8300 MSM Mobile Display Subsystem(MDSS), which encapsulates sub-blocks like >>>> + DPU display controller, DP interfaces and EDP etc. >>>> + >>>> +$ref: /schemas/display/msm/mdss-common.yaml# >>>> + >>>> +properties: >>>> + compatible: >>>> + const: qcom,qcs8300-mdss >>>> + >>>> + clocks: >>>> + items: >>>> + - description: Display AHB >>>> + - description: Display hf AXI >>>> + - description: Display core >>>> + >>>> + iommus: >>>> + maxItems: 1 >>>> + >>>> + interconnects: >>>> + maxItems: 3 >>>> + >>>> + interconnect-names: >>>> + maxItems: 3 >>>> + >>>> +patternProperties: >>>> + "^display-controller@[0-9a-f]+$": >>>> + type: object >>>> + additionalProperties: true >>>> + >>>> + properties: >>>> + compatible: >>>> + items: >>>> + - const: qcom,qcs8300-dpu >>>> + - const: qcom,sa8775p-dpu >>> >>> Use contains: instead of listing both of them >>> >> Got it, thanks~ here we want to fallback qcs8300 to sa8775p. >> Should we update it to : >> + contains: >> + enum: >> + - qcom,qcs8300-dpu >> + - qcom,sa8775p-dpu >> >> above method can be validated using dt_binding_check. > > > _instead_ of listing both of them > > So: > > contains: > const: qcom,qcs8300-dpu > Got it.. thanks~! >>>> + >>>> + "^displayport-controller@[0-9a-f]+$": >>>> + type: object >>>> + additionalProperties: true >>>> + >>>> + properties: >>>> + compatible: >>>> + items: >>>> + - const: qcom,qcs8300-dp >>>> + >>>> + "^phy@[0-9a-f]+$": >>>> + type: object >>>> + additionalProperties: true >>>> + properties: >>>> + compatible: >>>> + items: >>>> + - const: qcom,qcs8300-edp-phy >>>> + - const: qcom,sa8775p-edp-phy >>> >>> Use contains: instead of listing both of them >>> >> Same as above >>>> + >>>> +required: >>>> + - compatible >>>> + >>>> +unevaluatedProperties: false >>>> + >>> >> >
On Wed, 30 Jul 2025 17:42:28 +0800, Yongxing Mou wrote: > Document the MDSS hardware found on the Qualcomm QCS8300 platform. > > Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> > --- > .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ > 1 file changed, 284 insertions(+) > My bot found errors running 'make dt_binding_check' on your patch: yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible:0: 'qcom,qcs8300-edp-phy' is not one of ['qcom,sa8775p-edp-phy', 'qcom,sc7280-edp-phy', 'qcom,sc8180x-edp-phy', 'qcom,sc8280xp-dp-phy', 'qcom,sc8280xp-edp-phy', 'qcom,x1e80100-dp-phy'] from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible: ['qcom,qcs8300-edp-phy', 'qcom,sa8775p-edp-phy'] is too long from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250730-mdssdt_qcs8300-v5-3-bc8ea35bbed6@quicinc.com The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
On 2025/7/31 4:31, Rob Herring (Arm) wrote: > WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros. > > On Wed, 30 Jul 2025 17:42:28 +0800, Yongxing Mou wrote: >> Document the MDSS hardware found on the Qualcomm QCS8300 platform. >> >> Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> >> --- >> .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ >> 1 file changed, 284 insertions(+) >> > > My bot found errors running 'make dt_binding_check' on your patch: > > yamllint warnings/errors: > > dtschema/dtc warnings/errors: > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible:0: 'qcom,qcs8300-edp-phy' is not one of ['qcom,sa8775p-edp-phy', 'qcom,sc7280-edp-phy', 'qcom,sc8180x-edp-phy', 'qcom,sc8280xp-dp-phy', 'qcom,sc8280xp-edp-phy', 'qcom,x1e80100-dp-phy'] > from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible: ['qcom,qcs8300-edp-phy', 'qcom,sa8775p-edp-phy'] is too long > from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# > > doc reference errors (make refcheckdocs): > > See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250730-mdssdt_qcs8300-v5-3-bc8ea35bbed6@quicinc.com > > The base for the series is generally the latest rc1. A different dependency > should be noted in *this* patch. > > If you already ran 'make dt_binding_check' and didn't see the above > error(s), then make sure 'yamllint' is installed and dt-schema is up to > date: > > pip3 install dtschema --upgrade > > Please check and re-submit after running the above command yourself. Note > that DT_SCHEMA_FILES can be set to your schema file to speed up checking > your schema. However, it must be unset to test all examples with your schema. > This warning need to apply this patch.. https://lore.kernel.org/all/20250730072725.1433360-1-quic_yongmou@quicinc.com/ thanks, link in the cover-letter seem wrong. and there is an unnecessary ~
On Mon, Aug 04, 2025 at 12:00:39PM +0800, Yongxing Mou wrote: > > > On 2025/7/31 4:31, Rob Herring (Arm) wrote: > > WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros. > > > > On Wed, 30 Jul 2025 17:42:28 +0800, Yongxing Mou wrote: > > > Document the MDSS hardware found on the Qualcomm QCS8300 platform. > > > > > > Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> > > > --- > > > .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ > > > 1 file changed, 284 insertions(+) > > > > > > > My bot found errors running 'make dt_binding_check' on your patch: > > > > yamllint warnings/errors: > > > > dtschema/dtc warnings/errors: > > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible:0: 'qcom,qcs8300-edp-phy' is not one of ['qcom,sa8775p-edp-phy', 'qcom,sc7280-edp-phy', 'qcom,sc8180x-edp-phy', 'qcom,sc8280xp-dp-phy', 'qcom,sc8280xp-edp-phy', 'qcom,x1e80100-dp-phy'] > > from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# > > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible: ['qcom,qcs8300-edp-phy', 'qcom,sa8775p-edp-phy'] is too long > > from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# > > > > doc reference errors (make refcheckdocs): > > > > See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250730-mdssdt_qcs8300-v5-3-bc8ea35bbed6@quicinc.com > > > > The base for the series is generally the latest rc1. A different dependency > > should be noted in *this* patch. > > > > If you already ran 'make dt_binding_check' and didn't see the above > > error(s), then make sure 'yamllint' is installed and dt-schema is up to > > date: > > > > pip3 install dtschema --upgrade > > > > Please check and re-submit after running the above command yourself. Note > > that DT_SCHEMA_FILES can be set to your schema file to speed up checking > > your schema. However, it must be unset to test all examples with your schema. > > > This warning need to apply this patch.. > https://lore.kernel.org/all/20250730072725.1433360-1-quic_yongmou@quicinc.com/ > thanks, link in the cover-letter seem wrong. and there is an unnecessary ~ It's not listed in b4 dependencies. -- With best wishes Dmitry
On 2025/8/4 12:49, Dmitry Baryshkov wrote: > WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros. > > On Mon, Aug 04, 2025 at 12:00:39PM +0800, Yongxing Mou wrote: >> >> >> On 2025/7/31 4:31, Rob Herring (Arm) wrote: >>> WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros. >>> >>> On Wed, 30 Jul 2025 17:42:28 +0800, Yongxing Mou wrote: >>>> Document the MDSS hardware found on the Qualcomm QCS8300 platform. >>>> >>>> Signed-off-by: Yongxing Mou <quic_yongmou@quicinc.com> >>>> --- >>>> .../bindings/display/msm/qcom,qcs8300-mdss.yaml | 284 +++++++++++++++++++++ >>>> 1 file changed, 284 insertions(+) >>>> >>> >>> My bot found errors running 'make dt_binding_check' on your patch: >>> >>> yamllint warnings/errors: >>> >>> dtschema/dtc warnings/errors: >>> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible:0: 'qcom,qcs8300-edp-phy' is not one of ['qcom,sa8775p-edp-phy', 'qcom,sc7280-edp-phy', 'qcom,sc8180x-edp-phy', 'qcom,sc8280xp-dp-phy', 'qcom,sc8280xp-edp-phy', 'qcom,x1e80100-dp-phy'] >>> from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# >>> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcs8300-mdss.example.dtb: phy@aec2a00 (qcom,qcs8300-edp-phy): compatible: ['qcom,qcs8300-edp-phy', 'qcom,sa8775p-edp-phy'] is too long >>> from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml# >>> >>> doc reference errors (make refcheckdocs): >>> >>> See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250730-mdssdt_qcs8300-v5-3-bc8ea35bbed6@quicinc.com >>> >>> The base for the series is generally the latest rc1. A different dependency >>> should be noted in *this* patch. >>> >>> If you already ran 'make dt_binding_check' and didn't see the above >>> error(s), then make sure 'yamllint' is installed and dt-schema is up to >>> date: >>> >>> pip3 install dtschema --upgrade >>> >>> Please check and re-submit after running the above command yourself. Note >>> that DT_SCHEMA_FILES can be set to your schema file to speed up checking >>> your schema. However, it must be unset to test all examples with your schema. >>> >> This warning need to apply this patch.. >> https://lore.kernel.org/all/20250730072725.1433360-1-quic_yongmou@quicinc.com/ >> thanks, link in the cover-letter seem wrong. and there is an unnecessary ~ > > It's not listed in b4 dependencies. > > -- > With best wishes > Dmitry Ohh, sorry.. will update it in next version...
© 2016 - 2025 Red Hat, Inc.