Re: [PATCH v9 00/12] dt-bindings: display/msm: rework MDSS and DPU bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux