On 11/11/2022 11:05, AngeloGioacchino Del Regno wrote: > Il 04/11/22 18:54, Rob Herring ha scritto: >> >> On Fri, 04 Nov 2022 15:22:03 +0100, AngeloGioacchino Del Regno wrote: >>> Document bindings for the Qualcomm Ramp Controller, found on various >>> legacy Qualcomm SoCs. >>> >>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx> >>> --- >>> .../qcom/qcom,msm8976-ramp-controller.yaml | 37 +++++++++++++++++++ >>> 1 file changed, 37 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,msm8976-ramp-controller.yaml >>> >> >> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' >> on your patch (DT_CHECKER_FLAGS is new in v5.13): >> >> yamllint warnings/errors: >> >> dtschema/dtc warnings/errors: >> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/soc/qcom/qcom,msm8976-ramp-controller.example.dtb: power-controller@b014000: '#power-domain-cells' is a required property >> From schema: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/power/power-domain.yaml >> >> doc reference errors (make refcheckdocs): >> >> See https://patchwork.ozlabs.org/patch/ >> >> This check can fail if there are any dependencies. The base for a patch >> series is generally the most recent rc1. >> >> 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. >> > > I'm unsure about what I should do about this one. > This is a power-controller, but does *not* need any #power-domain-cells, as it is > standalone and doesn't require being attached to anything. power-domain-cells are for power domain providers, not consumers. The generic binding expect that nodes called power-controller are exactly like that. Solutions could be: 1. Rename the node to something else. I cannot deduct the type of the device based on description. What is "sequence ID" and how is it even closely related to power control? 2. Narrow the node name in power-domain.yaml which would require changes in multiple DTS and bindings. 3. Do not require power-domain-cells for power-controllers, only for power-domains. Best regards, Krzysztof
