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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CA1D6C3DA79 for ; Mon, 26 Dec 2022 15:21:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc: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=BorsIotAeEKFwCZB9cxzDewcoyGI07IOgHa/Sf25/Lk=; b=we65hLFO9/kSv2 5+6GSvkvcGS/fVvcrMCkDo7WCInSaHe0jJBQgKreKFStO77PKrUVV8kVs19C77dThdOsOjrt/q5G0 vYtcZEPfEqoISDvkTSFmk6oLmBAtbr9Ih98LWac42g1JmprgS0tWlrCrbsjFeU6OxGsvsTYm2NICQ ep/XhZvoHSNYEcawG4XUgR3uotUPaS1X4QESPA8EyLPBEctW9a7zODIcS6x6XlD0zAqWTTjX1RB3L TFPjKa6hHI/OoD/SmTq4vu6RhZbXNoK3c9++7vZnHRP62zwKK9yH0vs6W2sEsGPB6xk+NqMXhpMwC bmd07ewfcb003ga1CvEw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p9pGd-0053rT-H6; Mon, 26 Dec 2022 15:19:55 +0000 Received: from vps0.lunn.ch ([156.67.10.101]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p9pGX-0053k0-Ns; Mon, 26 Dec 2022 15:19:51 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=CJ0s4RC8ixikC2btIIdHDJkIJhgIxIMo2x6A5xpALMk=; b=5Y2X3l4ayBSYhKB92xci3QHhza 3+rZitQO+VAVq7GdeQsfaKzvkXLi3YAd5W/LiKV+lsp1KUfeDV6auyENA1c0Dn9bJ3HvJHZsNevHg Gedc5W2EBEAPzoAp15tYOHjSOeFhzkiTzreplDFmPnrLOA8oLtfIpitn+TeH3i8UnUCM=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1p9pG4-000VyF-Uc; Mon, 26 Dec 2022 16:19:20 +0100 Date: Mon, 26 Dec 2022 16:19:20 +0100 From: Andrew Lunn To: Biao Huang =?utf-8?B?KOm7hOW9qik=?= Cc: "linux-mediatek@lists.infradead.org" , "robh+dt@kernel.org" , "linux-kernel@vger.kernel.org" , "peppe.cavallaro@st.com" , "joabreu@synopsys.com" , "devicetree@vger.kernel.org" , "kuba@kernel.org" , "edumazet@google.com" , "alexandre.torgue@foss.st.com" , "mcoquelin.stm32@gmail.com" , "pabeni@redhat.com" , "richardcochran@gmail.com" , "krzysztof.kozlowski+dt@linaro.org" , Macpaul Lin =?utf-8?B?KOael+aZuuaWjCk=?= , "matthias.bgg@gmail.com" , "davem@davemloft.net" , "linux-arm-kernel@lists.infradead.org" , "angelogioacchino.delregno@collabora.com" Subject: Re: [PATCH v5 1/2] stmmac: dwmac-mediatek: enable 2ns delay only for special cases Message-ID: References: <20221223015029.24978-1-biao.huang@mediatek.com> <20221223015029.24978-2-biao.huang@mediatek.com> <4a82aa5437e496e2b65b77ef08d7897a4b23f7e5.camel@mediatek.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <4a82aa5437e496e2b65b77ef08d7897a4b23f7e5.camel@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221226_071949_818249_140B74DA X-CRM114-Status: UNSURE ( 9.31 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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 > We don't want to restrict phy-mode to rgmii-id only, so I don't think > we need mask RXID bit, then, User can use rgmii-rxid/-txid/-id as their > will. The phy-mode is a property of the board. It describes what delays are required, in addition to what the board itself provides via extra long lines. As such, there is no choice, it is set by physics. How you implement those delays is then up to the MAC and PHY. 95% of all MAC drivers simply pass phy-mode to the PHY and let the PHY add the delays. The Renesas MAC driver is one of the exceptions, it often adds the delays, and then passes a different phy-mode to the PHY in order that there are not double delays. So you can be different to everybody else, have MAC do one delay, and the PHY do the other. Just be aware that nobody else does this, no DT developers will expect it, etc. Or you can just do what everybody else does and let the PHY add the delays. What some MACs do is implement rx-internal-delay-ps and tx-internal-delay-ps These allow fine tuning of the delays. These are expected to be small values, adding 10s or 100s of pico seconds. They are not expected to be 2000ps. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel