On Wed, Jul 11, 2018 at 7:57 AM Andrew Lunn <andrew@xxxxxxx> wrote: > > On Tue, Jul 10, 2018 at 10:07:04PM -0700, Andrey Smirnov wrote: > > + switch@0 { > > + compatible = "marvell,mv88e6085"; > > + reg = <0>; > > + dsa,member = <0 0>; > > + eeprom-length = <512>; > > + interrupt-parent = <&gpio4>; > > + interrupts = <20 IRQ_TYPE_LEVEL_HIGH>; > > + interrupt-controller; > > + #interrupt-cells = <2>; > > Hi Andrey > > I don't see a pinmux for this GPIO. Is one needed? I got lucky and this pin is configured in GPIO mode out of reset. But strictly speaking, yeah, I think there should be a pinmux entry for this. Since things are working as is, I'll send a separate patch adding that instead of re-spinning this one. Thanks Andrey Smirnov -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html