2022-03-03 18:27:48

by Rob Herring (Arm)

Subject: [PATCH] dt-bindings: arm: Allow 32-bit 'cpu-release-addr' values

While the DT Spec says 'cpu-release-addr' is always 64-bit, some 32-bit Arm
DTs used a 32-bit value. We're now stuck with those cases, so add uint32 as
a valid type.

Signed-off-by: Rob Herring <[email protected]>
---
Documentation/devicetree/bindings/arm/cpus.yaml | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
index 85a31ca862d0..ed04650291a8 100644
--- a/Documentation/devicetree/bindings/arm/cpus.yaml
+++ b/Documentation/devicetree/bindings/arm/cpus.yaml
@@ -233,14 +233,14 @@ properties:
- ti,am4372

cpu-release-addr:
- $ref: '/schemas/types.yaml#/definitions/uint64'
-
+ oneOf:
+ - $ref: '/schemas/types.yaml#/definitions/uint32'
+ - $ref: '/schemas/types.yaml#/definitions/uint64'
description:
+ The DT specification defines this as 64-bit always, but some 32-bit Arm
+ systems have used a 32-bit value which must be supported.
Required for systems that have an "enable-method"
property value of "spin-table".
- On ARM v8 64-bit systems must be a two cell
- property identifying a 64-bit zero-initialised
- memory location.

cpu-idle-states:
$ref: '/schemas/types.yaml#/definitions/phandle-array'
--
2.32.0

2022-03-03 19:40:36

by Krzysztof Kozlowski

Subject: Re: [PATCH] dt-bindings: arm: Allow 32-bit 'cpu-release-addr' values

On 03/03/2022 17:57, Rob Herring wrote:
> While the DT Spec says 'cpu-release-addr' is always 64-bit, some 32-bit Arm
> DTs used a 32-bit value. We're now stuck with those cases, so add uint32 as
> a valid type.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/cpus.yaml | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>

Acked-by: Krzysztof Kozlowski <[email protected]>


Best regards,
Krzysztof