Messages in this thread | | | Date | Tue, 12 Nov 2019 15:01:09 +0100 | From | Andrew Lunn <> | Subject | Re: [PATCH 2/2] ARM: dts: ls1021a-tsn: Use interrupts for the SGMII PHYs |
| |
> > >> + /* SGMII2_PHY_INT_B: connected to IRQ2, active low > > >> */ > > >> + interrupts-extended = <&extirq 2 > > >> IRQ_TYPE_EDGE_FALLING>;
> The interrupt specifier certainly works. So that points to an issue > with the description. What do you mean, exactly? Does "active low" > mean "level-triggered"? How would you have described this?
I would expect IRQ_TYPE_ACTIVE_LOW, or whatever it is called. Since this is a shared interrupt, going on the edge i think opens up a race condition and interrupts can be missed.
Andrew
| |