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=-5.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no 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 33BAEC433E0 for ; Mon, 8 Feb 2021 14:27:47 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B226F64E5A for ; Mon, 8 Feb 2021 14:27:46 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B226F64E5A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=i2uKWa9XKk6dijcC9sOXWgQHM09bQjuwBiQ1C7WakyM=; b=eGpnQG8xPKgOwuYn7Ud4NhWEF M6tS4H1MDr3qKDMCCWLVag7lCMSWw99w6PrJgYQtvehFLZI6GZ9OF0bXw7msCFIV1Yirt6yMoLT6j uFpY+9sa0ji4BUN/6Gca5Vc4oWX1MNdrKImAM/lk4AHIEty7DudFBYxhzCTbt80OSTd97GWOY3zlZ DAPzEKHTyZmQuGU/EWxdDL4Ijjr9OZz9rKBRMocfwjoo+JmUQsjS0VIHM5nI1eHq9AkGjctcD8zpg WqwKFvzdPTAni+ZQTzM0OhueUvAW3BT7WMkT9Okt+VeF1zK4hMpd0IIKv03e0f1Vo25emwudpoRSN 9jy7UgrWA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l97UR-0005o2-7y; Mon, 08 Feb 2021 14:26:11 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1l97Sp-0003iW-6C for linux-arm-kernel@lists.infradead.org; Mon, 08 Feb 2021 14:24:55 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=PpJHE1jhhcZJ5XSixE7dqR0H/LdzaFtWtbMNUpSjYyI=; b=d/q/AV2/ZOstIsnIKGshZup5T WYMEXB69o9g4nyrY7XzyckkSDJUkmuahEY1R658pffr8zLzaDog7a0ds0JS9d5sSJutOKxLFytdUZ 8GS4K8f138sO3i1A62YkursMf0n4Dkb2fdPVJbX8KeYiZb8lvz1ht4FhF2l6lm3OHM1ax/dJ7Jvsm TPGODMjOZx1+EJU8obQg4BJN/qlhMc+10fc3ZQCa+yjX40F3znJxUvcA8hBgY0znObDAlObI2LNgN 7sPOzBxJG4kk4hTciED4AhGpXagli02AFZLmNGWG4hhGlYtaglPzqPg3nKo7Q8wNTwuziFkgL1I6n fmiJC2sRw==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:40794) by pandora.armlinux.org.uk with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1l97Pa-00023v-Jb; Mon, 08 Feb 2021 14:21:10 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.92) (envelope-from ) id 1l97PZ-00037D-7x; Mon, 08 Feb 2021 14:21:09 +0000 Date: Mon, 8 Feb 2021 14:21:09 +0000 From: Russell King - ARM Linux admin To: Oleksij Rempel Subject: Re: [PATCH v1 5/7] ARM i.MX6q: remove Atheros AR8035 SmartEEE fixup Message-ID: <20210208142107.GJ1463@shell.armlinux.org.uk> References: <20210203091857.16936-1-o.rempel@pengutronix.de> <20210203091857.16936-6-o.rempel@pengutronix.de> <20210203095628.GP1463@shell.armlinux.org.uk> <20210208092038.cjmnycyctsapgy7w@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210208092038.cjmnycyctsapgy7w@pengutronix.de> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210208_092431_327737_4D1E582F X-CRM114-Status: GOOD ( 18.15 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Andrew Lunn , Florian Fainelli , netdev@vger.kernel.org, Sascha Hauer , linux-kernel@vger.kernel.org, Fabio Estevam , Philippe Schenker , linux-imx@nxp.com, kernel@pengutronix.de, David Jander , Shawn Guo , linux-arm-kernel@lists.infradead.org, Heiner Kallweit Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Feb 08, 2021 at 10:20:38AM +0100, Oleksij Rempel wrote: > On Wed, Feb 03, 2021 at 09:56:28AM +0000, Russell King - ARM Linux admin wrote: > > That is the historical fix for this problem, but there is a better > > solution now in net-next - configuring the Tw parameter for gigabit > > connections. That solves the random link drop issue when EEE is > > enabled. > > Do you mean this properties? > qca,smarteee-tw-us-1g > qca,smarteee-tw-us-100m > > Do you have some recommendations, which values can be here used? Are > they same for all MACs? Or, can we calculate this values automatically? I don't think there's a way to "calculate" them. The AR8035 default is 17us for 1G and 23us for 100M. Increasing the 1G Tw to 26us or 27us fixes it on several different Solidrun platforms (iMX6 Hummingboards and Cubox-i, and LX2160A based). The boards all have differing layouts, so I don't think it's layout or SoC specific (which is good news.) These figures have been arrived at by repetitive long-term testing and observing whether there are sporadic link drops over these platforms. > Beside, I have seen this patch: "ARM: dts: imx6qdl-sr-som: fix some > cubox-i platforms" That's for a different problem: moving these settings to DT broke some Cubox-i platforms because of the weird ways that the AR8035 configures the address bits, using the LED pin. Tying a LED to the LED pin is not sufficient to guarantee that a board always configures the PHY to a particular address, so it can appear on address 0 or 4 depending on noise, temperature, supply voltage, and PHY chip thresholds. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel