On Mon, Oct 24, 2022 at 11:42 AM Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > > Create separate YAML schema for MDSS devicesd$ (both for MDP5 and DPU > devices). Cleanup DPU schema files, so that they do not contain schema > for both MDSS and DPU nodes. Apply misc small fixes to the DPU schema > afterwards. Add schema for the MDSS and DPU on sm8250 platform. > > Soft dependency on [1] to define qcom,dsi-phy-14nm-2290 binding used in > examples > > [1] https://lore.kernel.org/linux-arm-msm/20220924121900.222711-1-dmitry.baryshkov@xxxxxxxxxx/ > > Changes since v8: > - Dropped DSI/DSI-PHY examples from the first patch. Proper example > generate a pile of warnings because of DSI schema deficiencies. I'll > add these examples back, once DSI schema is fixed. > > Changes since v7: > - Expanded examples to include MDSS child nodes (Krzysztof) > > Changes since v6: > - Removed extra newlines (Krzysztof) > - Added $ref to dpu-common.yaml#/ports/port@foo to enforce schema > for the port nodes (Rob) > - Removed unused allOf's (Rob) > - Fixed repeated interconnects descriptions (Rob) > - Fixed dpu-common.yaml and mdss-common.yaml descriptions (Rob) > - Fixed intentation of examples (Krzysztof) > - Renamed MDSS and DPU schema to follow compat names (Rob) > > Changes since v5: > - Dropped the core clock from mdss.yaml. It will be handled in a > separate patchset together with adding the clock itself. > - Fixed a typo in two commit subjects (mdm -> msm). > > Changes since v4: > - Created separate mdss-common.yaml > - Rather than squashing everything into mdss.yaml, create individual > schema files for MDSS devices. > > Changes since v3: > - Changed mdss->(dpu, dsi, etc.) relationship into the tight binding > depending on the mdss compatible string. > - Added sm8250 dpu schema and added qcom,sm8250-mdss to mdss.yaml > > Changes since v2: > - Added a patch to allow opp-table under the dpu* nodes. > - Removed the c&p issue which allowed the @0 nodes under the MDSS > device node. > > Changes since v1: > - Renamed DPU device nodes from mdp@ to display-controller@ > - Described removal of mistakenly mentioned "lut" clock > - Switched mdss.yaml to use $ref instead of fixing compatible strings > - Dropped mdp-opp-table description (renamed by Krzysztof in his > patchset) > - Reworked DPU's ports definitions. Dropped description of individual > ports, left only /ports $ref and description in dpu-common.yaml. > > Dmitry Baryshkov (12): > dt-bindings: display/msm: split qcom,mdss bindings > dt-bindings: display/msm: add gcc-bus clock to dpu-smd845 > dt-bindings: display/msm: add interconnects property to > qcom,mdss-smd845 > dt-bindings: display/msm: move common DPU properties to > dpu-common.yaml > dt-bindings: display/msm: move common MDSS properties to > mdss-common.yaml > dt-bindings: display/msm: split dpu-sc7180 into DPU and MDSS parts > dt-bindings: display/msm: split dpu-sc7280 into DPU and MDSS parts > dt-bindings: display/msm: split dpu-sdm845 into DPU and MDSS parts > dt-bindings: display/msm: split dpu-msm8998 into DPU and MDSS parts > dt-bindings: display/msm: split dpu-qcm2290 into DPU and MDSS parts > dt-bindings: display/msm: add missing device nodes to mdss-* schemas > dt-bindings: display/msm: add support for the display on SM8250 This is now warning in linux-next: /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/qcom,sc7180-mdss.example.dtb: dsi@ae94000: 'opp-table' does not match any of the regexes: 'pinctrl-[0-9]+' From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/qcom,sm8250-mdss.example.dtb: dsi@ae94000: 'opp-table' does not match any of the regexes: 'pinctrl-[0-9]+' From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.example.dtb: dsi@5e94000: compatible: 'oneOf' conditional failed, one must be fixed: 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+-.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,(sa|sc)8[0-9]+[a-z][a-z]?-.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,[ak]pss-wdt-(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,gcc-(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,mmcc-(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,pcie-(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,rpm-(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+.*$' 'qcom,dsi-ctrl-6g-qcm2290' does not match '^qcom,scm-(apq|ipq|mdm|msm|qcm|qcs|sa|sc|sdm|sdx|sm)[0-9]+.*$' 'qcom,dsi-ctrl-6g-qcm2290' is not one of ['qcom,gpucc-sdm630', 'qcom,gpucc-sdm660', 'qcom,lcc-apq8064', 'qcom,lcc-ipq8064', 'qcom,lcc-mdm9615', 'qcom,lcc-msm8960', 'qcom,lpass-cpu-apq8016', 'qcom,usb-ss-ipq4019-phy', 'qcom,usb-hs-ipq4019-phy', 'qcom,vqmmc-ipq4019-regulator'] 'qcom,dsi-ctrl-6g-qcm2290' is not one of ['qcom,ipq806x-gmac', 'qcom,ipq806x-nand', 'qcom,ipq806x-sata-phy', 'qcom,ipq806x-usb-phy-ss', 'qcom,ipq806x-usb-phy-hs'] From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/arm/qcom-soc.yaml /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/qcom,sc7280-mdss.example.dtb: dsi@ae94000: 'opp-table' does not match any of the regexes: 'pinctrl-[0-9]+' From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
