Re: [PATCH RFC] ARM: dts: add support for Turris Omnia

Hello Uwe!

On Mon, Nov 14, 2016 at 9:28 PM, Andrew Lunn [off-list ref] wrote:

quoted

 +               i2c@7 {
 +                       #address-cells = <1>;
 +                       #size-cells = <0>;
 +                       reg = <7>;
 +
 +                       pcawan: gpio@71 {
 +                               compatible = "nxp,pca9538";
 +                               reg = <0x71>;
 +
 +                               pinctrl-names = "default";
 +                               pinctrl-0 = <&pcawan_pins>;
 +
 +                               interrupt-parent = <&gpio1>;
 +                               interrupts = <14 
IRQ_TYPE_LEVEL_LOW>;
 +
 +                               gpio-controller;
 +                               #gpio-cells = <2>;
 +
 +                               interrupt-controller;
 +                               #interrupt-cells = <2>;
 +                       };
 +               };

 The interrupt-controller part doesn't seem to work though, at least

 +               interrupt-parent = <&pcawan>;
 +               interrupts = <7 IRQ_TYPE_LEVEL_LOW>;

 in the phy node gives an error.

Interrupts don't seem to work very well with the nxp,pca9538. Which
is probably why it is disabled by default.

I was thinking about this issue and I can remember that there was an 
earlier prototype that had a shared interrupt line from PHY (88E1514) 
and from the PCA9538. In this case we needed to specifically disable 
the interrupt of the PHY to release the interrupt line (which needed a 
hack into PHY driver code). The IRQ from PHY is connected as an 
ordinary input to PCA9538 in later board prototype. And the same holds 
for the production version.

Do you have CZ11NIC13 or older board revision?

Tomas



--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html