On Wed, 15 Feb 2023 21:37:09 +0100 Bastian Germann <bage@xxxxxxxxxx> wrote: > Add an allwinner,sun50i-a64-hwspinlock compatible string to the device > tree binding. The A31 and A64 have compatible hwspinlocks. > > Signed-off-by: Bastian Germann <bage@xxxxxxxxxx> > --- > .../bindings/hwlock/allwinner,sun6i-a31-hwspinlock.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/hwlock/allwinner,sun6i-a31-hwspinlock.yaml b/Documentation/devicetree/bindings/hwlock/allwinner,sun6i-a31-hwspinlock.yaml > index 6cdfe22deb3c..281c285282a1 100644 > --- a/Documentation/devicetree/bindings/hwlock/allwinner,sun6i-a31-hwspinlock.yaml > +++ b/Documentation/devicetree/bindings/hwlock/allwinner,sun6i-a31-hwspinlock.yaml > @@ -15,7 +15,9 @@ description: > > properties: > compatible: > - const: allwinner,sun6i-a31-hwspinlock > + items: > + - const: allwinner,sun6i-a31-hwspinlock > + - const: allwinner,sun50i-a64-hwspinlock So this would not allow the previous single compatible string, which I think we should preserve? You can check out other bindings (like allwinner,sun6i-a31-spi.yaml) to get an idea of how this could be written. Cheers, Andre > > reg: > maxItems: 1