[PATCH] dt-bindings: arm: cpus: Document pu-supply

[PATCH] dt-bindings: arm: cpus: Document pu-supply

[Fabio Estevam]

The i.MX6Q/DL/SL devicetrees use the 'pu-supply' property to describe the
regulator connected to the PU domain.

Document the 'pu-supply' property and set it to deprecated.
    
This fixes the following dt-schema warning:
 
cpu@1 (arm,cortex-a9): Unevaluated properties are not allowed ('pu-supply' was unexpected)

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 Documentation/devicetree/bindings/arm/cpus.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index 4f1906d9fa68..c8a72fd02530 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -350,6 +350,10 @@ properties:
     deprecated: true
     description: Use 'cpu-supply' instead
 
+  pu-supply:
+    deprecated: true
+    description: Only for i.MX6Q/DL/SL SoCs.
+
   soc-supply:
     deprecated: true
     description: Only for i.MX6/7 Soc.
-- 
2.34.1

Re: [PATCH] dt-bindings: arm: cpus: Document pu-supply

[Krzysztof Kozlowski]

On Sat, Sep 06, 2025 at 08:19:30PM -0300, Fabio Estevam wrote:
> The i.MX6Q/DL/SL devicetrees use the 'pu-supply' property to describe the
> regulator connected to the PU domain.
> 
> Document the 'pu-supply' property and set it to deprecated.

This awfully looks like typo of cpu-supply, so I don't get why you just
cannot fix the DTS?

Best regards,
Krzysztof

Re: [PATCH] dt-bindings: arm: cpus: Document pu-supply

[Fabio Estevam]

On Sun, Sep 7, 2025 at 6:57 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:

> This awfully looks like typo of cpu-supply, so I don't get why you just
> cannot fix the DTS?

It is not a typo.

The i.MX6Q Reference Manual does reference it as PU:

"10.4.1.1.1
Digital LDO Regulators
The integrated PMU includes three digital LDO regulators: LDO_ARM, LDO_PU, and
LDO_SOC. These regulators provide power to the ARM_Core power domain, the
combined VPU, IPU and GPU power domain, and the rest of the SoC logic (except
always-ON SNVS domain)."

The imx6dl.dtsi uses the correct names:

arm-supply = <&reg_arm>;
pu-supply = <&reg_pu>;
soc-supply = <&reg_soc>;

Re: [PATCH] dt-bindings: arm: cpus: Document pu-supply

[Krzysztof Kozlowski]

On 07/09/2025 14:08, Fabio Estevam wrote:
> On Sun, Sep 7, 2025 at 6:57 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
>> This awfully looks like typo of cpu-supply, so I don't get why you just
>> cannot fix the DTS?
> 
> It is not a typo.
> 
> The i.MX6Q Reference Manual does reference it as PU:
> 
> "10.4.1.1.1
> Digital LDO Regulators
> The integrated PMU includes three digital LDO regulators: LDO_ARM, LDO_PU, and
> LDO_SOC. These regulators provide power to the ARM_Core power domain, the
> combined VPU, IPU and GPU power domain, and the rest of the SoC logic (except
> always-ON SNVS domain)."


Then please mention this in the commit msg.

Best regards,
Krzysztof