| Message ID | 20230719132749.661914-1-dinguyen@kernel.org |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [PATCHv2] dt-bindings: reset: altr,modrst-offset is not required for arm64 SoCFPGA | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Wed, Jul 19, 2023 at 08:27:49AM -0500, Dinh Nguyen wrote: > The property "altr,modrst-offset" is only applicable to arm32 SoCFPGA > platforms, thus it not required for all platforms. > > While at it, update my email address. > > Signed-off-by: Dinh Nguyen <dinguyen@kernel.org> Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor. > --- > v2: remove the need for else statement since "altr,modrst-offset" is > already allowed > --- > .../devicetree/bindings/reset/altr,rst-mgr.yaml | 12 ++++++++++-- > 1 file changed, 10 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml b/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml > index 4379cec6b35a..761c70cf9ddf 100644 > --- a/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml > +++ b/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml > @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml# > title: Altera SOCFPGA Reset Manager > > maintainers: > - - Dinh Nguyen <dinguyen@altera.com> > + - Dinh Nguyen <dinguyen@kernel.org> > > properties: > compatible: > @@ -32,9 +32,17 @@ properties: > required: > - compatible > - reg > - - altr,modrst-offset > - '#reset-cells' > > +if: > + properties: > + compatible: > + contains: > + const: altr,stratix10-rst-mgr > +then: > + properties: > + altr,modrst-offset: false > + > additionalProperties: false > > examples: > -- > 2.25.1 >
On Wed, 19 Jul 2023 08:27:49 -0500, Dinh Nguyen wrote: > The property "altr,modrst-offset" is only applicable to arm32 SoCFPGA > platforms, thus it not required for all platforms. > > While at it, update my email address. > > Signed-off-by: Dinh Nguyen <dinguyen@kernel.org> > --- > v2: remove the need for else statement since "altr,modrst-offset" is > already allowed > --- > .../devicetree/bindings/reset/altr,rst-mgr.yaml | 12 ++++++++++-- > 1 file changed, 10 insertions(+), 2 deletions(-) > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml b/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml index 4379cec6b35a..761c70cf9ddf 100644 --- a/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml +++ b/Documentation/devicetree/bindings/reset/altr,rst-mgr.yaml @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml# title: Altera SOCFPGA Reset Manager maintainers: - - Dinh Nguyen <dinguyen@altera.com> + - Dinh Nguyen <dinguyen@kernel.org> properties: compatible: @@ -32,9 +32,17 @@ properties: required: - compatible - reg - - altr,modrst-offset - '#reset-cells' +if: + properties: + compatible: + contains: + const: altr,stratix10-rst-mgr +then: + properties: + altr,modrst-offset: false + additionalProperties: false examples:
The property "altr,modrst-offset" is only applicable to arm32 SoCFPGA platforms, thus it not required for all platforms. While at it, update my email address. Signed-off-by: Dinh Nguyen <dinguyen@kernel.org> --- v2: remove the need for else statement since "altr,modrst-offset" is already allowed --- .../devicetree/bindings/reset/altr,rst-mgr.yaml | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-)