On Tue, Jul 18, 2023 at 03:42:17PM -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@xxxxxxxxxx> > --- > .../devicetree/bindings/reset/altr,rst-mgr.yaml | 15 +++++++++++++-- > 1 file changed, 13 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..7796408d2545 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@xxxxxxxxxx> > + - Dinh Nguyen <dinguyen@xxxxxxxxxx> > > properties: > compatible: > @@ -32,9 +32,20 @@ properties: > required: > - compatible > - reg > - - altr,modrst-offset > - '#reset-cells' > > +if: > + properties: > + compatible: > + contains: > + const: altr,stratix10-rst-mgr > +then: > + properties: > + altr,modrst-offset: false > +else: > + properties: > + altr,modrst-offset: true You don't need the else clause. "altr,modrst-offset" is already allowed by default. Rob
