| Message ID | 20230619220332.4038924-1-andrew@lunn.ch (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | b7c31ccd60d1df4634670e3eb9902baa19b9517b |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net-next,v1] net: phy-c45: Fix genphy_c45_ethtool_set_eee description | expand |
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski <kuba@kernel.org>: On Tue, 20 Jun 2023 00:03:32 +0200 you wrote: > The text has been cut/paste from genphy_c45_ethtool_get_eee but not > changed to reflect it performs set. > > Additionally, extend the comment. This function implements the logic > that eee_enabled has global control over EEE. When eee_enabled is > false, no link modes will be advertised, and as a result, the MAC > should not transmit LPI. > > [...] Here is the summary with links: - [net-next,v1] net: phy-c45: Fix genphy_c45_ethtool_set_eee description https://git.kernel.org/netdev/net-next/c/b7c31ccd60d1 You are awesome, thank you!
diff --git a/drivers/net/phy/phy-c45.c b/drivers/net/phy/phy-c45.c index fee514b96ab1..93ed07223377 100644 --- a/drivers/net/phy/phy-c45.c +++ b/drivers/net/phy/phy-c45.c @@ -1425,12 +1425,15 @@ int genphy_c45_ethtool_get_eee(struct phy_device *phydev, EXPORT_SYMBOL(genphy_c45_ethtool_get_eee); /** - * genphy_c45_ethtool_set_eee - get EEE supported and status + * genphy_c45_ethtool_set_eee - set EEE supported and status * @phydev: target phy_device struct * @data: ethtool_eee data * - * Description: it reportes the Supported/Advertisement/LP Advertisement - * capabilities. + * Description: sets the Supported/Advertisement/LP Advertisement + * capabilities. If eee_enabled is false, no links modes are + * advertised, but the previously advertised link modes are + * retained. This allows EEE to be enabled/disabled in a + * non-destructive way. */ int genphy_c45_ethtool_set_eee(struct phy_device *phydev, struct ethtool_eee *data)
The text has been cut/paste from genphy_c45_ethtool_get_eee but not changed to reflect it performs set. Additionally, extend the comment. This function implements the logic that eee_enabled has global control over EEE. When eee_enabled is false, no link modes will be advertised, and as a result, the MAC should not transmit LPI. Signed-off-by: Andrew Lunn <andrew@lunn.ch> --- Since v0: Rework documentation as suggested by Russell King drivers/net/phy/phy-c45.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)