* [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema
@ 2022-09-23 20:36 Krzysztof Kozlowski
2022-09-26 23:02 ` Rob Herring
2022-10-26 16:44 ` Krzysztof Kozlowski
0 siblings, 2 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-23 20:36 UTC (permalink / raw)
To: Sebastian Reichel, Rob Herring, Krzysztof Kozlowski, linux-pm,
devicetree, linux-kernel
Cc: Krzysztof Kozlowski
Add common schema for restart and shutdown handlers, so they all use
same meaning of "priority" field. The Linux drivers already have this
property and some systems want to customize it per-board in DTS.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Changes since v1:
1. Use double ':'
v1 was marked as changes-requested in Rob's patchwork, but I think there
was not actual consensus on any other changes to make.
v1:
https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20220907123630.57383-1-krzysztof.kozlowski@linaro.org/
See also:
https://lore.kernel.org/all/8fe93da3-f768-16ae-7025-1cfa97a42b27@linaro.org/
https://lore.kernel.org/all/20220831081715.14673-1-pali@kernel.org/
---
.../bindings/power/reset/gpio-restart.yaml | 13 ++------
.../bindings/power/reset/restart-handler.yaml | 30 +++++++++++++++++++
2 files changed, 33 insertions(+), 10 deletions(-)
create mode 100644 Documentation/devicetree/bindings/power/reset/restart-handler.yaml
diff --git a/Documentation/devicetree/bindings/power/reset/gpio-restart.yaml b/Documentation/devicetree/bindings/power/reset/gpio-restart.yaml
index a72d5c721516..d3d18e0f5db3 100644
--- a/Documentation/devicetree/bindings/power/reset/gpio-restart.yaml
+++ b/Documentation/devicetree/bindings/power/reset/gpio-restart.yaml
@@ -25,6 +25,9 @@ description: >
inactive-delay, the GPIO is driven active again. After a delay specified by wait-delay, the
restart handler completes allowing other restart handlers to be attempted.
+allOf:
+ - $ref: restart-handler.yaml#
+
properties:
compatible:
const: gpio-restart
@@ -41,16 +44,6 @@ properties:
in its inactive state.
priority:
- $ref: /schemas/types.yaml#/definitions/uint32
- description: |
- A priority ranging from 0 to 255 (default 129) according to the following guidelines:
-
- 0: Restart handler of last resort, with limited restart capabilities.
- 128: Default restart handler; use if no other restart handler is expected to be available,
- and/or if restart functionality is sufficient to restart the entire system.
- 255: Highest priority restart handler, will preempt all other restart handlers.
- minimum: 0
- maximum: 255
default: 129
active-delay:
diff --git a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
new file mode 100644
index 000000000000..1f9a2aac53c0
--- /dev/null
+++ b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
@@ -0,0 +1,30 @@
+# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/power/reset/restart-handler.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Restart and shutdown handler generic binding
+
+maintainers:
+ - Sebastian Reichel <sre@kernel.org>
+
+description:
+ Restart and shutdown handler device is responsible for powering off the
+ system, e.g. my cutting off the power. System might have several restart
+ handlers, which usually are tried from most precise to last resort.
+
+properties:
+ priority:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: |
+ A priority ranging from 0 to 255 according to the following guidelines::
+ 0:: Restart handler of last resort, with limited restart capabilities.
+ 128:: Typical, default restart handler; use if no other restart handler
+ is expected to be available, and/or if restart functionality is
+ sufficient to restart the entire system.
+ 255:: Highest priority restart handler, will preempt all other restart handlers.
+ minimum: 0
+ maximum: 255
+
+additionalProperties: true
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema
2022-09-23 20:36 [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema Krzysztof Kozlowski
@ 2022-09-26 23:02 ` Rob Herring
2022-10-26 16:44 ` Krzysztof Kozlowski
1 sibling, 0 replies; 4+ messages in thread
From: Rob Herring @ 2022-09-26 23:02 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: linux-pm, Sebastian Reichel, linux-kernel, Krzysztof Kozlowski,
devicetree, Rob Herring
On Fri, 23 Sep 2022 22:36:03 +0200, Krzysztof Kozlowski wrote:
> Add common schema for restart and shutdown handlers, so they all use
> same meaning of "priority" field. The Linux drivers already have this
> property and some systems want to customize it per-board in DTS.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> ---
>
> Changes since v1:
> 1. Use double ':'
>
> v1 was marked as changes-requested in Rob's patchwork, but I think there
> was not actual consensus on any other changes to make.
>
> v1:
> https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20220907123630.57383-1-krzysztof.kozlowski@linaro.org/
>
> See also:
> https://lore.kernel.org/all/8fe93da3-f768-16ae-7025-1cfa97a42b27@linaro.org/
> https://lore.kernel.org/all/20220831081715.14673-1-pali@kernel.org/
> ---
> .../bindings/power/reset/gpio-restart.yaml | 13 ++------
> .../bindings/power/reset/restart-handler.yaml | 30 +++++++++++++++++++
> 2 files changed, 33 insertions(+), 10 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/power/reset/restart-handler.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema
2022-09-23 20:36 [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema Krzysztof Kozlowski
2022-09-26 23:02 ` Rob Herring
@ 2022-10-26 16:44 ` Krzysztof Kozlowski
2022-10-28 14:14 ` Sebastian Reichel
1 sibling, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2022-10-26 16:44 UTC (permalink / raw)
To: Sebastian Reichel
Cc: devicetree, linux-pm, Rob Herring, linux-kernel, Krzysztof Kozlowski
On 23/09/2022 16:36, Krzysztof Kozlowski wrote:
> Add common schema for restart and shutdown handlers, so they all use
> same meaning of "priority" field. The Linux drivers already have this
> property and some systems want to customize it per-board in DTS.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> ---
Sebastian,
This is still pending. Any comments from your side? Can you pick it up?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema
2022-10-26 16:44 ` Krzysztof Kozlowski
@ 2022-10-28 14:14 ` Sebastian Reichel
0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Reichel @ 2022-10-28 14:14 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: devicetree, linux-pm, Rob Herring, linux-kernel, Krzysztof Kozlowski
[-- Attachment #1: Type: text/plain, Size: 564 bytes --]
Hi,
On Wed, Oct 26, 2022 at 12:44:22PM -0400, Krzysztof Kozlowski wrote:
> On 23/09/2022 16:36, Krzysztof Kozlowski wrote:
> > Add common schema for restart and shutdown handlers, so they all use
> > same meaning of "priority" field. The Linux drivers already have this
> > property and some systems want to customize it per-board in DTS.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >
> > ---
>
> Sebastian,
>
> This is still pending. Any comments from your side? Can you pick it up?
Done.
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-10-28 14:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-23 20:36 [PATCH v2] dt-bindings: power: reset: restart-handler: add common schema Krzysztof Kozlowski
2022-09-26 23:02 ` Rob Herring
2022-10-26 16:44 ` Krzysztof Kozlowski
2022-10-28 14:14 ` Sebastian Reichel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.