From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Andrew Lunn <andrew@lunn.ch>
Cc: netdev <netdev@vger.kernel.org>,
Heiner Kallweit <hkallweit1@gmail.com>,
Florian Fainelli <f.fainelli@gmail.com>,
Oleksij Rempel <linux@rempel-privat.de>
Subject: Re: [PATCH v4 net-next 5/9] net: phy: Keep track of EEE configuration
Date: Sun, 18 Jun 2023 20:20:51 +0100 [thread overview]
Message-ID: <ZI9ZEy82KX0orkTi@shell.armlinux.org.uk> (raw)
In-Reply-To: <20230618184119.4017149-6-andrew@lunn.ch>
On Sun, Jun 18, 2023 at 08:41:15PM +0200, Andrew Lunn wrote:
> Have phylib keep track of the EEE configuration. This simplifies the
> MAC drivers, in that they don't need to store it.
>
> Future patches to phylib will also make use of this information to
> further simplify the MAC drivers.
>
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> ---
> drivers/net/phy/phy.c | 5 ++++-
> include/linux/phy.h | 3 +++
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
> index 629499e5aff0..48150d5626d8 100644
> --- a/drivers/net/phy/phy.c
> +++ b/drivers/net/phy/phy.c
> @@ -1588,7 +1588,7 @@ EXPORT_SYMBOL(phy_get_eee_err);
> * @data: ethtool_eee data
> *
> * Description: it reportes the Supported/Advertisement/LP Advertisement
> - * capabilities.
> + * capabilities, etc.
Any scope to drop the "it" and fix "reports" ?
Apart from that, looks good.
Reviewed-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Thanks!
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
next prev parent reply other threads:[~2023-06-18 19:20 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-18 18:41 [PATCH v4 net-next 0/9] net: ethernet: Rework EEE Andrew Lunn
2023-06-18 18:41 ` [PATCH v4 net-next 1/9] net: phy-c45: Fix genphy_c45_ethtool_set_eee description Andrew Lunn
2023-06-18 19:18 ` Russell King (Oracle)
2023-06-18 20:30 ` Andrew Lunn
2023-06-18 18:41 ` [PATCH v4 net-next 2/9] net: add helpers for EEE configuration Andrew Lunn
2023-06-19 15:26 ` Florian Fainelli
2023-06-18 18:41 ` [PATCH v4 net-next 3/9] net: phy: Add phydev->enable_tx_lpi to simplify adjust link callbacks Andrew Lunn
2023-06-18 18:41 ` [PATCH v4 net-next 4/9] net: phy: Add helper to set EEE Clock stop enable bit Andrew Lunn
2023-06-18 18:41 ` [PATCH v4 net-next 5/9] net: phy: Keep track of EEE configuration Andrew Lunn
2023-06-18 19:20 ` Russell King (Oracle) [this message]
2023-06-19 14:20 ` Simon Horman
2023-06-19 15:27 ` Florian Fainelli
2023-06-18 18:41 ` [PATCH v4 net-next 6/9] net: phy: Immediately call adjust_link if only tx_lpi_enabled changes Andrew Lunn
2023-06-18 18:41 ` [PATCH v4 net-next 7/9] net: phy: Add phy_support_eee() indicating MAC support EEE Andrew Lunn
2023-06-19 15:21 ` Florian Fainelli
2023-06-22 15:20 ` Russell King (Oracle)
2023-06-22 15:43 ` Andrew Lunn
2023-06-18 18:41 ` [PATCH v4 net-next 8/9] net: fec: Move fec_enet_eee_mode_set() and helper earlier Andrew Lunn
2023-06-19 15:22 ` Florian Fainelli
2023-06-18 18:41 ` [PATCH v4 net-next 9/9] net: fec: Fixup EEE Andrew Lunn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZI9ZEy82KX0orkTi@shell.armlinux.org.uk \
--to=linux@armlinux.org.uk \
--cc=andrew@lunn.ch \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--cc=linux@rempel-privat.de \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.