Re: [PATCH v2 1/4] dt-bindings: net: Add MTIP L2 switch description
From: Lukasz Majewski
Date: Sun Mar 30 2025 - 02:37:18 EST
Hi Andrew,
> > + ethphy0: ethernet-phy@0 {
> > + reg = <0>;
> > + smsc,disable-energy-detect;
> > + /* Both PHYs (i.e. 0,1) have the same,
> > single GPIO, */
> > + /* line to handle both, their interrupts
> > (AND'ed) */
> > + interrupt-parent = <&gpio4>;
> > + interrupts = <13 IRQ_TYPE_EDGE_FALLING>;
>
> Shared interrupts cannot be edge. They are level, so that either can
> hold the interrupt active until it is cleared.
>
> Also, PHY interrupts in general are level, because there are multiple
> interrupt sources within the PHY, and you need to clear them all
> before the interrupt is released.
Good point, I will update it accordingly. Thanks.
>
> Andrew
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@xxxxxxx
Attachment:
pgpvmqkou27bf.pgp
Description: OpenPGP digital signature