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 311C4C4828D for ; Tue, 6 Feb 2024 13:13:24 +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=uzf+d0SAvMGCUOhHxb1DKs7IpvBDkwTXl3zaDqfK/4Q=; b=TEN4g5brXHtzIE a2zCHX6L0GXW49OO3wHbaqv7tb7rHn1zVNMowfaHKRFQYqUlT2Gr47p8Et1TG7hR8WtiK59QJOYYB s0eL4dkhz3WsWZuLtutSJZ+FHT/i4ZsIibO0NtEE1qlpiRRriRWSepZZemasSEGzKyJAVHh2L7bqM PJgiY+H4k3JBqUoPJ16WCAxIeeLLVcHgZ8bNqVbNgnpgu64haPss29ObIjQ35c18Ip8lScGgIYhYU 7Zg8VvOhDBs3ltmpSt0dVfHzMbGarhU9eUHkrI0z2qewNRbdKFQMkSQq15cNoGirmk9AjkcyOrFKT ayEHYXVkqw5h5OWdEgOg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rXLGC-00000007fvK-2UVU; Tue, 06 Feb 2024 13:13:12 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rXLG9-00000007fqf-0L5V for linux-arm-kernel@lists.infradead.org; Tue, 06 Feb 2024 13:13:11 +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=UjKogbJoVYwz4/4zIRPiHZA8DeUtMRsDWvm2PZop3Tc=; b=1naGhjY7kYT8LNxUUIUOowYWeX eKwBs27naUc6/NxZ6U/hWTiIEXpVXwM6TFkHBBzOETzr0T2kl8nAOdru3pAOKyabsFXeL7livY0kf OD3F5iyMvCtxKORhmUACtDY0BoSx242QEo+yw8OiIHoVlXgbhR9X5Nn02hPMtAyuE5vaTuJRcWVon CAjx21TdK+VHLsaK7sfo1PVuJ+xHQtc6AXlDN23pRm1ObkEfVef33aY3BZcB9ht4M6C9coPYiaCFG AF75hBHggI3Ug/weORRMZpw+EMmfo35N0wmIodL9XBj4A661QS1p3FcnpNa/8Agc8SSU/Op6FcAPp 2qslTbKw==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:39738) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1rXLFh-00021Y-0h; Tue, 06 Feb 2024 13:12:41 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1rXLFZ-0003f6-I9; Tue, 06 Feb 2024 13:12:33 +0000 Date: Tue, 6 Feb 2024 13:12:33 +0000 From: "Russell King (Oracle)" To: Vladimir Oltean Cc: Florian Fainelli , Doug Berger , Andrew Lunn , Heiner Kallweit , Alexandre Torgue , bcm-kernel-feedback-list@broadcom.com, Byungho An , Clark Wang , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Jose Abreu , Justin Chen , linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, Maxime Coquelin , netdev@vger.kernel.org, NXP Linux Team , Paolo Abeni , Shenwei Wang , Wei Fang Subject: Re: [PATCH net-next v2 6/6] net: dsa: b53: remove eee_enabled/eee_active in b53_get_mac_eee() Message-ID: References: <20240206112024.3jxtcru3dupeirnj@skbuf> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240206112024.3jxtcru3dupeirnj@skbuf> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240206_051309_453167_A1A2D5EC X-CRM114-Status: GOOD ( 20.45 ) 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 On Tue, Feb 06, 2024 at 01:20:24PM +0200, Vladimir Oltean wrote: > On Sun, Feb 04, 2024 at 12:13:28PM +0000, Russell King (Oracle) wrote: > > b53_get_mac_eee() sets both eee_enabled and eee_active, and then > > returns zero. > > > > dsa_slave_get_eee(), which calls this function, will then continue to > > call phylink_ethtool_get_eee(), which will return -EOPNOTSUPP if there > > is no PHY present, otherwise calling phy_ethtool_get_eee() which in > > turn will call genphy_c45_ethtool_get_eee(). > > Nitpick: If you need to resend, the function name changed to > dsa_user_get_eee(). Thanks. > > @@ -2227,16 +2227,10 @@ EXPORT_SYMBOL(b53_eee_init); > > int b53_get_mac_eee(struct dsa_switch *ds, int port, struct ethtool_keee *e) > > { > > struct b53_device *dev = ds->priv; > > - struct ethtool_keee *p = &dev->ports[port].eee; > > - u16 reg; > > > > if (is5325(dev) || is5365(dev)) > > return -EOPNOTSUPP; > > > > - b53_read16(dev, B53_EEE_PAGE, B53_EEE_LPI_INDICATE, ®); > > - e->eee_enabled = p->eee_enabled; > > - e->eee_active = !!(reg & BIT(port)); > > - > > I know next to nothing about EEE and especially the implementation on > Broadcom switches. But is the information brought by B53_EEE_LPI_INDICATE > completely redundant? Is it actually in the system's best interest to > ignore it? That's a review comment that should have been made when the original change to phylib was done, because it's already ignored in kernels today since the commit changing phylib that I've referenced in this series - since e->eee_enabled and e->eee_active will be overwritten by phylib. If we need B53_EEE_LPI_INDICATE to do something, then we need to have a discussion about it, and decide how that fits in with the EEE interface, and how to work around phylib's implementation. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps 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