From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CE707C282C4 for ; Tue, 12 Feb 2019 12:09:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9301320821 for ; Tue, 12 Feb 2019 12:09:28 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="wA0fE0CL" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729344AbfBLMJ1 (ORCPT ); Tue, 12 Feb 2019 07:09:27 -0500 Received: from fllv0016.ext.ti.com ([198.47.19.142]:52166 "EHLO fllv0016.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729317AbfBLMJ0 (ORCPT ); Tue, 12 Feb 2019 07:09:26 -0500 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id x1CC9IhS085501; Tue, 12 Feb 2019 06:09:18 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1549973358; bh=HbW5Cbx8/Z9f6U/PntT2uOF/ma9bF8UfgFHsFeLtLrQ=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=wA0fE0CLtydw2mPSLjzLVC7XQh0/YGwafv//GSOVphhR360VbbvDYpumMQtzgWZLy eTPr3KONefPso7eaBx7BBMtdqoKZth/frCwuXFvBhkkfEsgynorUSGmiClY3vjDqct WEfHuVmhJddEtZzfw27bKNl7tacmJJnEJf4hDTjg= Received: from DFLE111.ent.ti.com (dfle111.ent.ti.com [10.64.6.32]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id x1CC9IND102632 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 12 Feb 2019 06:09:18 -0600 Received: from DFLE114.ent.ti.com (10.64.6.35) by DFLE111.ent.ti.com (10.64.6.32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1591.10; Tue, 12 Feb 2019 06:09:15 -0600 Received: from dflp32.itg.ti.com (10.64.6.15) by DFLE114.ent.ti.com (10.64.6.35) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.1591.10 via Frontend Transport; Tue, 12 Feb 2019 06:09:16 -0600 Received: from [192.168.2.6] (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp32.itg.ti.com (8.14.3/8.13.8) with ESMTP id x1CC9DlP021046; Tue, 12 Feb 2019 06:09:14 -0600 Subject: Re: [v3,4/5] net: phy: at803x: Disable phy delay for RGMII mode To: Vinod Koul CC: David S Miller , Roger Quadros , , , Niklas Cassel , Bjorn Andersson , Andrew Lunn , Florian Fainelli , "Nori, Sekhar" References: <20190121091318.20079-5-vkoul@kernel.org> <147151c4-d162-4ebe-189a-564492d84d18@ti.com> <20190212113114.GZ4296@vkoul-mobl> From: Peter Ujfalusi Message-ID: <06680a9b-70ec-df76-b632-d8c467b3a5f7@ti.com> Date: Tue, 12 Feb 2019 14:09:39 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.0 MIME-Version: 1.0 In-Reply-To: <20190212113114.GZ4296@vkoul-mobl> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 8bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Hi Vinod, On 12/02/2019 13.31, Vinod Koul wrote: > Hi Peter, > > On 12-02-19, 12:55, Peter Ujfalusi wrote: >> Vinod, >> >> On 21/01/2019 11.13, Vinod Koul wrote: >>> For RGMII mode, phy delay should be disabled. Add this case along >>> with disable delay routines. >> >> In next-20190211 I need to revert this patch to get cpsw networking to >> work on am335x-evmsk. The board uses AR8031_AL1A PHY, which is handled >> by the phy/at803x.c > > I see that DTS specifies that you are using phy-mode = "rgmii-txid". > RGMII mode implies that we should not have any delay in the > phy, so this patch does the right thing. Right, on am335x-evmsk: [ 3.393406] at803x_config_init: phydev->interface: 11 So PHY_INTERFACE_MODE_RGMII_TXID. > In the previous version of the patch I did propose to add a DT entry so > that current users who are wrongly using this would not be impacted but > the suggestion was to get them fixed. > > So in you case do you need rgmii-txd mode if so why should the delay be > enabled for this? We can add a patch that enabled delay for your > controller but that cant be rgmii mode. I'm not too familiar with the networking, but the AR8031 datasheet does tell that the debug_5:bit8 is 0 by default (rgmii tx clock delay disable). In the errata document of am335x (http://www.ti.com/lit/pdf/sprz360), 1.0.10 I believe tells that the delay needs to be enabled on the rgmii ethernet PHY because the internal delay is not working (?) and advises that the delay might need to be enabled on the rgmii PHY. The PHY_INTERFACE_MODE_RGMII_TXID delay was enabled in the PHY driver since 2013 (1ca6d1b1aef4628ff0fe458135ddb008d134ad8f) To me it looks like that when AR8031 is used with am335x the 'rgmii tx clock delay' needs to be enabled. Sekhar, Roger: any insights, thoughts? > > Thanks > > >> >> On next-20190211: >> [ 3.374601] net eth0: initializing cpsw version 1.12 (0) >> [ 3.384484] Atheros 8031 ethernet 4a101000.mdio:00: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=4a101000.mdio:00, irq=POLL) >> [ 3.400041] cpsw 4a100000.ethernet eth0: Link is Up - 1Gbps/Full - flow control rx/tx >> [ 3.410813] mmc1: new SDIO card at address 0001 >> [ 3.439362] IP-Config: Complete: >> [ 3.442649] device=eth0, hwaddr=bc:6a:29:7d:2c:a9, ipaddr=10.0.0.90, mask=255.255.255.0, gw=10.0.0.1 >> [ 3.452840] host=10.0.0.90, domain=, nis-domain=(none) >> [ 3.458462] bootserver=10.0.0.30, rootserver=10.0.0.30, rootpath= >> [ 3.466296] vwl1271: disabling >> [ 3.470195] ALSA device list: >> [ 3.473189] #0: AM335x-EVMSK >> >> After reverting this patch: >> [ 3.374636] net eth0: initializing cpsw version 1.12 (0) >> [ 3.384534] Atheros 8031 ethernet 4a101000.mdio:00: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=4a101000.mdio:00, irq=POLL) >> [ 3.400125] cpsw 4a100000.ethernet eth0: Link is Up - 1Gbps/Full - flow control rx/tx >> [ 3.410866] mmc1: new SDIO card at address 0001 >> [ 3.439379] IP-Config: Complete: >> [ 3.442666] device=eth0, hwaddr=bc:6a:29:7d:2c:a9, ipaddr=10.0.0.90, mask=255.255.255.0, gw=10.0.0.1 >> [ 3.452865] host=10.0.0.90, domain=, nis-domain=(none) >> [ 3.458482] bootserver=10.0.0.30, rootserver=10.0.0.30, rootpath= >> [ 3.466334] vwl1271: disabling >> [ 3.470245] ALSA device list: >> [ 3.473241] #0: AM335x-EVMSK >> [ 3.501052] VFS: Mounted root (nfs filesystem) readonly on device 0:15. >> [ 3.508694] devtmpfs: mounted >> [ 3.514546] Freeing unused kernel memory: 1024K >> [ 3.520567] Run /sbin/init as init process >> >> and the board boots to nfsroot fine. >> >> >>> Signed-off-by: Vinod Koul >>> --- >>> drivers/net/phy/at803x.c | 22 ++++++++++++---------- >>> 1 file changed, 12 insertions(+), 10 deletions(-) >>> >>> diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c >>> index f9432d053a22..8ff12938ab47 100644 >>> --- a/drivers/net/phy/at803x.c >>> +++ b/drivers/net/phy/at803x.c >>> @@ -110,16 +110,16 @@ static int at803x_debug_reg_mask(struct phy_device *phydev, u16 reg, >>> return phy_write(phydev, AT803X_DEBUG_DATA, val); >>> } >>> >>> -static inline int at803x_enable_rx_delay(struct phy_device *phydev) >>> +static inline int at803x_disable_rx_delay(struct phy_device *phydev) >>> { >>> - return at803x_debug_reg_mask(phydev, AT803X_DEBUG_REG_0, 0, >>> - AT803X_DEBUG_RX_CLK_DLY_EN); >>> + return at803x_debug_reg_mask(phydev, AT803X_DEBUG_REG_0, >>> + AT803X_DEBUG_RX_CLK_DLY_EN, 0); >>> } >>> >>> -static inline int at803x_enable_tx_delay(struct phy_device *phydev) >>> +static inline int at803x_disable_tx_delay(struct phy_device *phydev) >>> { >>> - return at803x_debug_reg_mask(phydev, AT803X_DEBUG_REG_5, 0, >>> - AT803X_DEBUG_TX_CLK_DLY_EN); >>> + return at803x_debug_reg_mask(phydev, AT803X_DEBUG_REG_5, >>> + AT803X_DEBUG_TX_CLK_DLY_EN, 0); >>> } >>> >>> /* save relevant PHY registers to private copy */ >>> @@ -256,15 +256,17 @@ static int at803x_config_init(struct phy_device *phydev) >>> return ret; >>> >>> if (phydev->interface == PHY_INTERFACE_MODE_RGMII_RXID || >>> - phydev->interface == PHY_INTERFACE_MODE_RGMII_ID) { >>> - ret = at803x_enable_rx_delay(phydev); >>> + phydev->interface == PHY_INTERFACE_MODE_RGMII_ID || >>> + phydev->interface == PHY_INTERFACE_MODE_RGMII) { >>> + ret = at803x_disable_rx_delay(phydev); >>> if (ret < 0) >>> return ret; >>> } >>> >>> if (phydev->interface == PHY_INTERFACE_MODE_RGMII_TXID || >>> - phydev->interface == PHY_INTERFACE_MODE_RGMII_ID) { >>> - ret = at803x_enable_tx_delay(phydev); >>> + phydev->interface == PHY_INTERFACE_MODE_RGMII_ID || >>> + phydev->interface == PHY_INTERFACE_MODE_RGMII) { >>> + ret = at803x_disable_tx_delay(phydev); >>> if (ret < 0) >>> return ret; >>> } >>> >> >> - Péter >> >> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > - Péter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki