[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <99a4fbb3-0204-4a96-bc38-9a76817a326a@gmail.com>
Date: Fri, 1 Mar 2024 23:38:24 +0100
From: Heiner Kallweit <hkallweit1@...il.com>
To: Oleksij Rempel <o.rempel@...gutronix.de>, Wei Fang <wei.fang@....com>,
"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Andrew Lunn <andrew@...n.ch>, Russell King <linux@...linux.org.uk>
Cc: Russell King <rmk+kernel@...linux.org.uk>,
Florian Fainelli <florian.fainelli@...adcom.com>, kernel@...gutronix.de,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
Shenwei Wang <shenwei.wang@....com>, Clark Wang <xiaoning.wang@....com>,
NXP Linux Team <linux-imx@....com>
Subject: Re: [PATCH net-next v8 1/8] net: add helpers for EEE configuration
On 01.03.2024 11:01, Oleksij Rempel wrote:
> From: Russell King <rmk+kernel@...linux.org.uk>
>
> Add helpers that phylib and phylink can use to manage EEE configuration
> and determine whether the MAC should be permitted to use LPI based on
> that configuration.
>
> Signed-off-by: Russell King (Oracle) <rmk+kernel@...linux.org.uk>
> Signed-off-by: Andrew Lunn <andrew@...n.ch>
> Reviewed-by: Florian Fainelli <florian.fainelli@...adcom.com>
> Signed-off-by: Oleksij Rempel <o.rempel@...gutronix.de>
> ---
> include/net/eee.h | 38 ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 38 insertions(+)
> create mode 100644 include/net/eee.h
>
> diff --git a/include/net/eee.h b/include/net/eee.h
> new file mode 100644
> index 0000000000000..1232658b32f40
> --- /dev/null
> +++ b/include/net/eee.h
> @@ -0,0 +1,38 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +#ifndef _EEE_H
> +#define _EEE_H
> +
> +#include <linux/types.h>
> +
> +struct eee_config {
> + u32 tx_lpi_timer;
> + bool tx_lpi_enabled;
> + bool eee_enabled;
> +};
> +
> +static inline bool eeecfg_mac_can_tx_lpi(const struct eee_config *eeecfg)
> +{
> + /* eee_enabled is the master on/off */
> + if (!eeecfg->eee_enabled || !eeecfg->tx_lpi_enabled)
> + return false;
> +
> + return true;
> +}
> +
> +static inline void eeecfg_to_eee(const struct eee_config *eeecfg,
> + struct ethtool_keee *eee)
> +{
Typically the argument order is f(dst, src), like for string functions.
Any specific reason handle it differently here?
> + eee->tx_lpi_timer = eeecfg->tx_lpi_timer;
> + eee->tx_lpi_enabled = eeecfg->tx_lpi_enabled;
> + eee->eee_enabled = eeecfg->eee_enabled;
> +}
> +
> +static inline void eee_to_eeecfg(const struct ethtool_keee *eee,
> + struct eee_config *eeecfg)
> +{
> + eeecfg->tx_lpi_timer = eee->tx_lpi_timer;
> + eeecfg->tx_lpi_enabled = eee->tx_lpi_enabled;
> + eeecfg->eee_enabled = eee->eee_enabled;
> +}
> +
> +#endif
Powered by blists - more mailing lists