On Sun, 14 Apr 2024 12:04:23 -0500 Chris Morgan <macroalpha82@xxxxxxxxx> wrote: > From: Chris Morgan <macromorgan@xxxxxxxxxxx> > > Add binding for the H616 NMI controller. This is necessary to handle > interrupts from devices such as the internal PMIC. I think "internal PMIC" is confusing, and I guess it's not really relevant anyway and you can remove the whole sentence. Instead just mention that this controller turns out to be compatible to the one described in the A80 manual. > > Signed-off-by: Chris Morgan <macromorgan@xxxxxxxxxxx> > --- > .../interrupt-controller/allwinner,sun7i-a20-sc-nmi.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun7i-a20-sc-nmi.yaml b/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun7i-a20-sc-nmi.yaml > index 83603180d8d9..4fc8b0320a31 100644 > --- a/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun7i-a20-sc-nmi.yaml > +++ b/Documentation/devicetree/bindings/interrupt-controller/allwinner,sun7i-a20-sc-nmi.yaml > @@ -32,6 +32,9 @@ properties: > - items: > - const: allwinner,sun50i-a100-nmi > - const: allwinner,sun9i-a80-nmi > + - items: > + - const: allwinner,sun50i-h616-nmi > + - const: allwinner,sun9i-a80-nmi It looks like this is the third chip that uses an A80 compatible version. So I'd say it's time to use an enum to group the A100, H616 and V3s together. Check allwinner,sun6i-a31-r-intc.yaml for an example. Otherwise looks good. Cheers, Andre > > reg: > maxItems: 1
