| Message ID | 20241031200350.274945-2-chris.packham@alliedtelesis.co.nz |
|---|---|
| State | Accepted |
| Commit | e7af7d13316dc5e2293c4f777f71bd8331f5d7a5 |
| Headers | show |
| Series | RTL9300 support for reboot and i2c | expand |
On Fri, Nov 01, 2024 at 09:03:44AM +1300, Chris Packham wrote: > The syscon-reboot binding used 'offset' for historical reasons. Having a > reg property is appropriate when these nodes are children of a MMIO bus. > Add a reg property and modify the constraints so that one of 'reg' or > 'offset' is expected. > > Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > > Notes: > Changes in v8: > - None. This has been applied to by Sebastian, I've just kept it in the > series so that others can see the complete picutre No, this only creates unnecessary review / traffic and might lead to duplicated patches. Provide link to commit if you want to give a reference. *Never* send already applied patches. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml b/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml index 75061124d9a8..19d3093e6cd2 100644 --- a/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +++ b/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml @@ -31,6 +31,10 @@ properties: $ref: /schemas/types.yaml#/definitions/uint32 description: Offset in the register map for the reboot register (in bytes). + reg: + maxItems: 1 + description: Base address and size for the reboot register. + regmap: $ref: /schemas/types.yaml#/definitions/phandle deprecated: true @@ -45,9 +49,14 @@ properties: priority: default: 192 +oneOf: + - required: + - offset + - required: + - reg + required: - compatible - - offset additionalProperties: false