Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753843AbcJUBcC (ORCPT ); Thu, 20 Oct 2016 21:32:02 -0400 Received: from mail.kernel.org ([198.145.29.136]:50888 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751226AbcJUBb6 (ORCPT ); Thu, 20 Oct 2016 21:31:58 -0400 Date: Fri, 21 Oct 2016 09:31:28 +0800 From: Shawn Guo To: Jagan Teki Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Matteo Lisi , Michael Trimarchi , Jagan Teki , Sascha Hauer , Fabio Estevam Subject: Re: [PATCH v7 5/5] ARM: dts: imx6qdl-icore: Add FEC support Message-ID: <20161021013128.GE30578@tiger> References: <1476437243-13297-1-git-send-email-jteki@openedev.com> <1476437243-13297-6-git-send-email-jteki@openedev.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1476437243-13297-6-git-send-email-jteki@openedev.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3273 Lines: 111 On Fri, Oct 14, 2016 at 02:57:23PM +0530, Jagan Teki wrote: > From: Jagan Teki > > 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 > Cc: Fabio Estevam > Cc: Shawn Guo > Cc: Matteo Lisi > Cc: Michael Trimarchi > Signed-off-by: Jagan Teki > --- > 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 >