RE: [PATCH net-next 5/5] dt-bindings: net: Add bindings for the ADIN1140

[Regus, Ciprian]

> -----Original Message-----
> From: Andrew Lunn <andrew@xxxxxxx>
> Sent: Sunday, May 3, 2026 4:07 AM
> To: Regus, Ciprian <Ciprian.Regus@xxxxxxxxxx>
> Cc: Parthiban Veerasooran <parthiban.veerasooran@xxxxxxxxxxxxx>;
> Andrew Lunn <andrew+netdev@xxxxxxx>; David S. Miller
> <davem@xxxxxxxxxxxxx>; Eric Dumazet <edumazet@xxxxxxxxxx>; Jakub
> Kicinski <kuba@xxxxxxxxxx>; Paolo Abeni <pabeni@xxxxxxxxxx>; Simon
> Horman <horms@xxxxxxxxxx>; Jonathan Corbet <corbet@xxxxxxx>; Shuah
> Khan <skhan@xxxxxxxxxxxxxxxxxxx>; Heiner Kallweit
> <hkallweit1@xxxxxxxxx>; Russell King <linux@xxxxxxxxxxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor
> Dooley <conor+dt@xxxxxxxxxx>; netdev@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; linux-doc@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH net-next 5/5] dt-bindings: net: Add bindings for the
> ADIN1140
> 
> [External]
> 
> > +  The ADIN1140 (also called AD3306) is a low power single port
> > +  10BASE-T1S MAC-PHY. It integrates an Ethernet PHY with a MAC
> > +  and all the associated analog circuitry.
> > +  The device implements the Open Alliance TC6 10BASE-T1x MAC-PHY
> 
> The device _tries_ to implements the Open Alliance TC6 10BASE-T1x MAC-
> PHY.

Will update in v2.

> 
> > +examples:
> > +  - |
> > +    #include <dt-bindings/interrupt-controller/irq.h>
> > +
> > +    spi {
> > +        #address-cells = <1>;
> > +        #size-cells = <0>;
> > +
> > +        ethernet@0 {
> > +            compatible = "adi,adin1140";
> > +            reg = <0>;
> > +            spi-max-frequency = <23000000>;
> > +
> > +            interrupt-parent = <&gpio>;
> > +            interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
> 
> Table 1: OPEN serial 10BASE-T1x Interface Pin Definition
> 
> IRQn MAC-PHY Interrupt Request (Active Low)
> 
> Or is this something else which the device gets wrong?

The device generates interrupts correctly (the IRQ signal remains
asserted while there are active interrupt conditions that have not
been cleared yet). The oa_tc6 driver requests the interrupt with
the IRQF_TRIGGER_FALLING flag set, so the DT flag will be overridden
and the behavior remains the same.

However, the devicetree shouldn't care about this. I'll update to
IRQ_TYPE_LEVEL_LOW in v2.

> 
> 	Andrew