On Fri, Oct 14, 2016 at 02:57:23PM +0530, Jagan Teki wrote: > From: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> > > Add FEC support for Engicam i.CoreM6 dql modules. > > Observed similar 'eth0: link is not ready' issue which was > discussed in [1] due rmii mode with external ref_clk, so added > clock node along with the properties mentioned by Shawn in [2] > > FEC link log: > ------------ > $ ifconfig eth0 up > [ 27.905187] SMSC LAN8710/LAN8720 2188000.ethernet:00: attached PHY driver > [SMSC LAN8710/LAN8720] (mii_bus:phy_addr=2188000.ethernet:00, irq=-1) > [ 27.918982] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready > > [1] https://patchwork.kernel.org/patch/3491061/ > [2] https://patchwork.kernel.org/patch/3490511/ > > Cc: Sascha Hauer <kernel@xxxxxxxxxxxxxx> > Cc: Fabio Estevam <fabio.estevam@xxxxxxx> > Cc: Shawn Guo <shawnguo@xxxxxxxxxx> > Cc: Matteo Lisi <matteo.lisi@xxxxxxxxxxx> > Cc: Michael Trimarchi <michael@xxxxxxxxxxxxxxxxxxxx> > Signed-off-by: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> > --- > Changes for v7: > - none > Changes for v6: > - none > Changes for v5: > - new patch > > arch/arm/boot/dts/imx6qdl-icore.dtsi | 37 ++++++++++++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6qdl-icore.dtsi b/arch/arm/boot/dts/imx6qdl-icore.dtsi > index 4e79858..972f48f 100644 > --- a/arch/arm/boot/dts/imx6qdl-icore.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-icore.dtsi > @@ -48,6 +48,18 @@ > reg = <0x10000000 0x80000000>; > }; > > + clocks { > + #address-cells = <1>; > + #size-cells = <0>; DT maintainers do not like this container node. So please, just like fix regulator node, put the fixed clock directly under root and give the node an unique name like clock-xxx. > + > + rmii_clk: clock@0 { > + compatible = "fixed-clock"; > + reg = <0>; > + #clock-cells = <0>; > + clock-frequency = <25000000>; /* 25MHz for example */ > + }; > + }; > + > reg_3p3v: regulator-3p3v { > compatible = "regulator-fixed"; > regulator-name = "3P3V"; > @@ -93,6 +105,15 @@ > assigned-clock-parents = <&clks IMX6QDL_CLK_OSC>; > }; > > +&fec { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_enet>; > + phy-reset-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>; > + clocks = <&clks 117>, <&clks 117>, <&rmii_clk>; s/117/IMX6QDL_CLK_ENET Shawn > + phy-mode = "rmii"; > + status = "okay"; > +}; > + > &gpmi { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_gpmi_nand>; > @@ -150,6 +171,22 @@ > }; > > &iomuxc { > + pinctrl_enet: enetgrp { > + fsl,pins = < > + MX6QDL_PAD_ENET_CRS_DV__ENET_RX_EN 0x1b0b0 > + MX6QDL_PAD_GPIO_16__ENET_REF_CLK 0x1b0b1 > + MX6QDL_PAD_ENET_TX_EN__ENET_TX_EN 0x1b0b0 > + MX6QDL_PAD_ENET_RXD1__ENET_RX_DATA1 0x1b0b0 > + MX6QDL_PAD_ENET_RXD0__ENET_RX_DATA0 0x1b0b0 > + MX6QDL_PAD_ENET_TXD1__ENET_TX_DATA1 0x1b0b0 > + MX6QDL_PAD_ENET_TXD0__ENET_TX_DATA0 0x1b0b0 > + MX6QDL_PAD_ENET_MDC__ENET_MDC 0x1b0b0 > + MX6QDL_PAD_ENET_MDIO__ENET_MDIO 0x1b0b0 > + MX6QDL_PAD_ENET_REF_CLK__GPIO1_IO23 0x1b0b0 > + MX6QDL_PAD_GPIO_17__GPIO7_IO12 0x1b0b0 > + >; > + }; > + > pinctrl_flexcan1: flexcan1grp { > fsl,pins = < > MX6QDL_PAD_KEY_ROW2__FLEXCAN1_RX 0x1b020 > -- > 2.7.4 > -- 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
