Re: [PATCH v5 3/7] dt-bindings: arm: cpus: Document 'qcom,freq-domain' property

From: Rob Herring
Date: Fri Feb 05 2021 - 16:47:50 EST


On Thu, Jan 21, 2021 at 08:52:46PM +0100, AngeloGioacchino Del Regno wrote:
> From: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
>
> Add devicetree documentation for 'qcom,freq-domain' property specific
> to Qualcomm CPUs. This property is used to reference the CPUFREQ node
> along with Domain ID (0/1).
>
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/arm/cpus.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
> index 14cd727d3c4b..1d60975df23a 100644
> --- a/Documentation/devicetree/bindings/arm/cpus.yaml
> +++ b/Documentation/devicetree/bindings/arm/cpus.yaml
> @@ -290,6 +290,12 @@ properties:
>
> * arm/msm/qcom,kpss-acc.txt
>
> + qcom,freq-domain:

What happened to having a standard property here? There were 1 or 2
other cases of doing the same thing.

> + $ref: '/schemas/types.yaml#/definitions/phandle-array'
> + description: |
> + CPUs supporting freq-domain must set their "qcom,freq-domain" property
> + with phandle to a cpufreq_hw node followed by the Domain ID(0/1).
> +
> rockchip,pmu:
> $ref: '/schemas/types.yaml#/definitions/phandle'
> description: |
> --
> 2.30.0
>