2020-02-21 23:30:37

by Elliot Berman

[permalink] [raw]
Subject: [PATCH 1/3] dt: psci: Add arm,psci-sys-reset2-type property

Some implementors of PSCI may relax the requirements of the PSCI
architectural warm reset. In order to comply with PSCI specification, a
different reset_type value must be used. The alternate PSCI
SYSTEM_RESET2 may be used in all warm/soft reboot scenarios, replacing
the architectural warm reset.

Signed-off-by: Elliot Berman <[email protected]>
---
Documentation/devicetree/bindings/arm/psci.yaml | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/psci.yaml b/Documentation/devicetree/bindings/arm/psci.yaml
index 8ef8542..a790e5a 100644
--- a/Documentation/devicetree/bindings/arm/psci.yaml
+++ b/Documentation/devicetree/bindings/arm/psci.yaml
@@ -102,6 +102,11 @@ properties:
[1] Kernel documentation - ARM idle states bindings
Documentation/devicetree/bindings/arm/idle-states.txt

+ arm,psci-sys-reset2-type:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: |
+ reset_type parameter value to use during a warm or soft reboot.
+
"#power-domain-cells":
description:
The number of cells in a PM domain specifier as per binding in [3].
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project


2020-02-24 09:52:57

by Sudeep Holla

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt: psci: Add arm,psci-sys-reset2-type property

On Fri, Feb 21, 2020 at 03:28:03PM -0800, Elliot Berman wrote:
> Some implementors of PSCI may relax the requirements of the PSCI
> architectural warm reset. In order to comply with PSCI specification, a
> different reset_type value must be used. The alternate PSCI
> SYSTEM_RESET2 may be used in all warm/soft reboot scenarios, replacing
> the architectural warm reset.
>
> Signed-off-by: Elliot Berman <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/psci.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/psci.yaml b/Documentation/devicetree/bindings/arm/psci.yaml
> index 8ef8542..a790e5a 100644
> --- a/Documentation/devicetree/bindings/arm/psci.yaml
> +++ b/Documentation/devicetree/bindings/arm/psci.yaml
> @@ -102,6 +102,11 @@ properties:
> [1] Kernel documentation - ARM idle states bindings
> Documentation/devicetree/bindings/arm/idle-states.txt
>
> + arm,psci-sys-reset2-type:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: |
> + reset_type parameter value to use during a warm or soft reboot.
> +

I would rather use param instead of type in the name.

--
Regards,
Sudeep