From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="ykNWUMcT" Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [IPv6:2001:4d48:ad52:32c8:5054:ff:fe00:142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8DA5F8E; Thu, 14 Dec 2023 08:49:17 -0800 (PST) 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=jpEgcy8NN3HTStEB0Wx38X+kWFgy54/SSxE7weirF/E=; b=ykNWUMcT36CU8ETNnUBdAfU5Di DdbbhgKydxhHANMjCciNJr98L8meROJk0TARR4eSpvc0QjNauXChKS9k/5RgLtTgmReaGbOzh/gQs UB+HHTwT2uI5MnX919RNy3rtpS0rtfqnAU1OWGoy4D+JdqYWp4AFF+7z3jiTAXaCa8/IkY2WaZ5Kv 9a/3mNXm+y5GwXuFJVgTJvqJmi/S+p9PB0YwIa2mPnjLD5rdHkQB9TLOJAeT5d9m5Vqzk6aLmANKh ANoL90HggfZf6QWBokdy6F/zsFD5mNiv6PdbmO9yG96k/zN3kwqFVaMdCwIvB5vFjK2kGNmGWOoEY Ew0OAhsA==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:49794) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1rDotX-0001hF-1F; Thu, 14 Dec 2023 16:49:07 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1rDotX-0002h8-FN; Thu, 14 Dec 2023 16:49:07 +0000 Date: Thu, 14 Dec 2023 16:49:07 +0000 From: "Russell King (Oracle)" To: Romain Gantois Cc: Oleksij Rempel , Wei Fang , Marek Vasut , "netdev@vger.kernel.org" , "David S. Miller" , Andrew Lunn , Eric Dumazet , Heiner Kallweit , Jakub Kicinski , Paolo Abeni , kernel@pengutronix.de, linux-clk@vger.kernel.org, Stephen Boyd , Michael Turquette Subject: Re: [PATCH] net: phy: at803x: Improve hibernation support on start up Message-ID: References: <20230804175842.209537-1-marex@denx.de> <45b1ee70-8330-0b18-2de1-c94ddd35d817@denx.de> <20230809043626.GG5736@pengutronix.de> <20230809060836.GA13300@pengutronix.de> <7aabc198-9df5-5bce-2968-90d4cda3c244@bootlin.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7aabc198-9df5-5bce-2968-90d4cda3c244@bootlin.com> Sender: Russell King (Oracle) On Thu, Dec 14, 2023 at 09:13:58AM +0100, Romain Gantois wrote: > Hello Russell, > > I've implemented and tested the general-case solution you proposed to this > receive clock issue with stmmac drivers. The core of your suggestion is pretty > much unchanged, I just added a phylink_pcs flag for standalone PCS drivers that > also need to provide the receive clock. So this affects the ability of PCS to operate correctly as well as MACs? Would you enlighten which PCS are affected, and what PCS <--> PHY link modes this is required for? > I'd like to send a series for this upstream, which would allow solving this > issue for both the DWMAC RZN1 case and the AT803x PHY suspend/hibernate case > (and also potentially other cases with a similar bug). > > I wanted to ask you how you would prefer to be credited in my patch series. I > was considering putting you as author and first signer of the initial patch > adding the phy_dev flag. Would that be okay or would you prefer something else? It depends how big the changes are from my patches. If more than 50% of the patch remains my work, please retain my authorship. If you wish to also indicate your authorship, then there is a mechanism to do that - Co-developed-by: from submitting-patches.rst: Co-developed-by: states that the patch was co-created by multiple developers; it is used to give attribution to co-authors (in addition to the author attributed by the From: tag) when several people work on a single patch. Since Co-developed-by: denotes authorship, every Co-developed-by: must be immediately followed by a Signed-off-by: of the associated co-author. See submitting-patches.rst for examples of the ordering of the attributations. Thanks for asking. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!