Hi Russell, On lun., janv. 02 2017, Russell King <rmk+kernel@xxxxxxxxxxxxxxx> wrote: It would be nice to have some word here about this patch. Especially why we need it now. I guess it is for being less dependent on the initialization done by the bootloader but maybe you have other reasons. Thanks, Gregory > Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxx> > --- > arch/arm/boot/dts/armada-388-clearfog-base.dts | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm/boot/dts/armada-388-clearfog-base.dts b/arch/arm/boot/dts/armada-388-clearfog-base.dts > index f86e1876fb38..da788ea40717 100644 > --- a/arch/arm/boot/dts/armada-388-clearfog-base.dts > +++ b/arch/arm/boot/dts/armada-388-clearfog-base.dts > @@ -74,7 +74,17 @@ > phy = <&phy1>; > }; > > +&gpio0 { > + phy1_reset { > + gpio-hog; > + gpios = <19 GPIO_ACTIVE_LOW>; > + output-low; > + line-name = "phy1-reset"; > + }; > +}; > + > &mdio { > + pinctrl-0 = <&mdio_pins µsom_phy_clk_pins &clearfog_phy_pins>; > phy1: ethernet-phy@1 { > /* > * Annoyingly, the marvell phy driver configures the LED > @@ -87,6 +97,11 @@ > }; > > &pinctrl { > + /* phy1 reset */ > + clearfog_phy_pins: clearfog-phy-pins { > + marvell,pins = "mpp19"; > + marvell,function = "gpio"; > + }; > rear_button_pins: rear-button-pins { > marvell,pins = "mpp44"; > marvell,function = "gpio"; > -- > 2.7.4 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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