devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: arm: cpus: Document pu-supply
@ 2025-09-07 15:25 Fabio Estevam
  2025-09-08  7:58 ` Krzysztof Kozlowski
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Fabio Estevam @ 2025-09-07 15:25 UTC (permalink / raw)
  To: robh; +Cc: krzk+dt, conor+dt, devicetree, Frank.Li, Fabio Estevam

The i.MX6Q Reference Manual describes the three digital LDO regulators
as follows:

"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)."

imx6dl.dtsi uses the correct names to describe these supplies:

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

'arm-supply' and 'soc-supply' are already documented, but 'pu-supply' is not.

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>
---
Changes since v1:
- Explain that PU is a valid name. (Krzysztof)

 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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] dt-bindings: arm: cpus: Document pu-supply
  2025-09-07 15:25 [PATCH v2] dt-bindings: arm: cpus: Document pu-supply Fabio Estevam
@ 2025-09-08  7:58 ` Krzysztof Kozlowski
  2025-09-08 20:48 ` Frank Li
  2025-09-09  0:24 ` Rob Herring (Arm)
  2 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-09-08  7:58 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: robh, krzk+dt, conor+dt, devicetree, Frank.Li

On Sun, Sep 07, 2025 at 12:25:13PM -0300, Fabio Estevam wrote:
> The i.MX6Q Reference Manual describes the three digital LDO regulators
> as follows:
> 
> "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)."
> 
> imx6dl.dtsi uses the correct names to describe these supplies:
> 
> arm-supply = <&reg_arm>;
> pu-supply = <&reg_pu>;
> soc-supply = <&reg_soc>;
> 
> 'arm-supply' and 'soc-supply' are already documented, but 'pu-supply' is not.
> 
> 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>
> ---
> Changes since v1:
> - Explain that PU is a valid name. (Krzysztof)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] dt-bindings: arm: cpus: Document pu-supply
  2025-09-07 15:25 [PATCH v2] dt-bindings: arm: cpus: Document pu-supply Fabio Estevam
  2025-09-08  7:58 ` Krzysztof Kozlowski
@ 2025-09-08 20:48 ` Frank Li
  2025-09-09  0:24 ` Rob Herring (Arm)
  2 siblings, 0 replies; 4+ messages in thread
From: Frank Li @ 2025-09-08 20:48 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: robh, krzk+dt, conor+dt, devicetree

On Sun, Sep 07, 2025 at 12:25:13PM -0300, Fabio Estevam wrote:
> The i.MX6Q Reference Manual describes the three digital LDO regulators
> as follows:
>
> "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)."
>
> imx6dl.dtsi uses the correct names to describe these supplies:
>
> arm-supply = <&reg_arm>;
> pu-supply = <&reg_pu>;
> soc-supply = <&reg_soc>;
>
> 'arm-supply' and 'soc-supply' are already documented, but 'pu-supply' is not.
>
> 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>

Reviewed-by: Frank Li <Frank.Li@nxp.com>

> ---
> Changes since v1:
> - Explain that PU is a valid name. (Krzysztof)
>
>  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
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] dt-bindings: arm: cpus: Document pu-supply
  2025-09-07 15:25 [PATCH v2] dt-bindings: arm: cpus: Document pu-supply Fabio Estevam
  2025-09-08  7:58 ` Krzysztof Kozlowski
  2025-09-08 20:48 ` Frank Li
@ 2025-09-09  0:24 ` Rob Herring (Arm)
  2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-09-09  0:24 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: krzk+dt, devicetree, Frank.Li, conor+dt


On Sun, 07 Sep 2025 12:25:13 -0300, Fabio Estevam wrote:
> The i.MX6Q Reference Manual describes the three digital LDO regulators
> as follows:
> 
> "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)."
> 
> imx6dl.dtsi uses the correct names to describe these supplies:
> 
> arm-supply = <&reg_arm>;
> pu-supply = <&reg_pu>;
> soc-supply = <&reg_soc>;
> 
> 'arm-supply' and 'soc-supply' are already documented, but 'pu-supply' is not.
> 
> 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>
> ---
> Changes since v1:
> - Explain that PU is a valid name. (Krzysztof)
> 
>  Documentation/devicetree/bindings/arm/cpus.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 

Applied, thanks!


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-09-09  0:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-07 15:25 [PATCH v2] dt-bindings: arm: cpus: Document pu-supply Fabio Estevam
2025-09-08  7:58 ` Krzysztof Kozlowski
2025-09-08 20:48 ` Frank Li
2025-09-09  0:24 ` Rob Herring (Arm)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).