| Message ID | 20250401-syscon-reboot-reset-mode-v5-1-5b9357442363@linaro.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | support Linux reboot modes in syscon-reboot on gs101 (Google Pixel) | expand |
On Tue, Apr 01, 2025 at 11:11:02AM +0100, André Draszik wrote: > GS101 supports a couple different reset types via certain registers in > the SYSCON register map. > > Add a compatible for it. When in effect, all register values and offsets > are implied, hence they shall not be specified in that case. > > Signed-off-by: André Draszik <andre.draszik@linaro.org> > --- > .../bindings/power/reset/syscon-reboot.yaml | 42 +++++++++++++++------- > 1 file changed, 30 insertions(+), 12 deletions(-) Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml b/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml index 19d3093e6cd2f7e39d94c56636dc202a4427ffc3..ccd5558700943ef56f5e1c866400bcc21c0115f0 100644 --- a/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +++ b/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml @@ -21,7 +21,9 @@ description: |+ properties: compatible: - const: syscon-reboot + enum: + - syscon-reboot + - google,gs101-reboot mask: $ref: /schemas/types.yaml#/definitions/uint32 @@ -49,12 +51,6 @@ properties: priority: default: 192 -oneOf: - - required: - - offset - - required: - - reg - required: - compatible @@ -63,12 +59,29 @@ additionalProperties: false allOf: - $ref: restart-handler.yaml# - if: - not: - required: - - mask + properties: + compatible: + contains: + const: google,gs101-reboot then: - required: - - value + properties: + mask: false + offset: false + reg: false + value: false + + else: + if: + not: + required: + - mask + then: + required: + - value + + oneOf: + - required: [offset] + - required: [reg] examples: - | @@ -78,3 +91,8 @@ examples: offset = <0x0>; mask = <0x1>; }; + + - | + reboot { + compatible = "google,gs101-reboot"; + };
GS101 supports a couple different reset types via certain registers in the SYSCON register map. Add a compatible for it. When in effect, all register values and offsets are implied, hence they shall not be specified in that case. Signed-off-by: André Draszik <andre.draszik@linaro.org> --- .../bindings/power/reset/syscon-reboot.yaml | 42 +++++++++++++++------- 1 file changed, 30 insertions(+), 12 deletions(-)