* [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee
@ 2022-01-23 15:22 Jisheng Zhang
2022-01-23 15:54 ` Andrew Lunn
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Jisheng Zhang @ 2022-01-23 15:22 UTC (permalink / raw)
To: Florian Fainelli, Andrew Lunn, Vivien Didelot, Vladimir Oltean,
David S . Miller, Jakub Kicinski, Sean Wang, Landen Chao,
DENG Qingfang, Matthias Brugger, Doug Berger, Joakim Zhang,
Thomas Petazzoni, Bryan Whitehead, UNGLinuxDriver, Byungho An
Cc: netdev, linux-kernel, linux-arm-kernel, linux-mediatek,
bcm-kernel-feedback-list
The 2nd param of phy_init_eee(): clk_stop_enable is a bool param, use
true or false instead of 1/0.
Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
---
drivers/net/dsa/b53/b53_common.c | 2 +-
drivers/net/dsa/mt7530.c | 2 +-
drivers/net/ethernet/broadcom/genet/bcmgenet.c | 2 +-
drivers/net/ethernet/freescale/fec_main.c | 2 +-
drivers/net/ethernet/marvell/mvneta.c | 2 +-
drivers/net/ethernet/microchip/lan743x_ethtool.c | 2 +-
drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c
index 3867f3d4545f..a3b98992f180 100644
--- a/drivers/net/dsa/b53/b53_common.c
+++ b/drivers/net/dsa/b53/b53_common.c
@@ -2186,7 +2186,7 @@ int b53_eee_init(struct dsa_switch *ds, int port, struct phy_device *phy)
{
int ret;
- ret = phy_init_eee(phy, 0);
+ ret = phy_init_eee(phy, false);
if (ret)
return 0;
diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c
index b82512e5b33b..bc77a26c825a 100644
--- a/drivers/net/dsa/mt7530.c
+++ b/drivers/net/dsa/mt7530.c
@@ -2846,7 +2846,7 @@ static void mt753x_phylink_mac_link_up(struct dsa_switch *ds, int port,
mcr |= PMCR_RX_FC_EN;
}
- if (mode == MLO_AN_PHY && phydev && phy_init_eee(phydev, 0) >= 0) {
+ if (mode == MLO_AN_PHY && phydev && phy_init_eee(phydev, false) >= 0) {
switch (speed) {
case SPEED_1000:
mcr |= PMCR_FORCE_EEE1G;
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
index 87f1056e29ff..cfe09117fe6c 100644
--- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c
+++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
@@ -1368,7 +1368,7 @@ static int bcmgenet_set_eee(struct net_device *dev, struct ethtool_eee *e)
if (!p->eee_enabled) {
bcmgenet_eee_enable_set(dev, false);
} else {
- ret = phy_init_eee(dev->phydev, 0);
+ ret = phy_init_eee(dev->phydev, false);
if (ret) {
netif_err(priv, hw, dev, "EEE initialization failed\n");
return ret;
diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c
index 796133de527e..11227f51404c 100644
--- a/drivers/net/ethernet/freescale/fec_main.c
+++ b/drivers/net/ethernet/freescale/fec_main.c
@@ -2797,7 +2797,7 @@ static int fec_enet_eee_mode_set(struct net_device *ndev, bool enable)
int ret = 0;
if (enable) {
- ret = phy_init_eee(ndev->phydev, 0);
+ ret = phy_init_eee(ndev->phydev, false);
if (ret)
return ret;
diff --git a/drivers/net/ethernet/marvell/mvneta.c b/drivers/net/ethernet/marvell/mvneta.c
index 83c8908f0cc7..7f44b73024a0 100644
--- a/drivers/net/ethernet/marvell/mvneta.c
+++ b/drivers/net/ethernet/marvell/mvneta.c
@@ -4169,7 +4169,7 @@ static void mvneta_mac_link_up(struct phylink_config *config,
mvneta_port_up(pp);
if (phy && pp->eee_enabled) {
- pp->eee_active = phy_init_eee(phy, 0) >= 0;
+ pp->eee_active = phy_init_eee(phy, false) >= 0;
mvneta_set_eee(pp, pp->eee_active && pp->tx_lpi_enabled);
}
}
diff --git a/drivers/net/ethernet/microchip/lan743x_ethtool.c b/drivers/net/ethernet/microchip/lan743x_ethtool.c
index 91a755efe2e6..5f1e7b8bad4f 100644
--- a/drivers/net/ethernet/microchip/lan743x_ethtool.c
+++ b/drivers/net/ethernet/microchip/lan743x_ethtool.c
@@ -750,7 +750,7 @@ static int lan743x_ethtool_set_eee(struct net_device *netdev,
}
if (eee->eee_enabled) {
- ret = phy_init_eee(phydev, 0);
+ ret = phy_init_eee(phydev, false);
if (ret) {
netif_err(adapter, drv, adapter->netdev,
"EEE initialization failed\n");
diff --git a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
index 32161a56726c..77a0d9d7e65a 100644
--- a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
+++ b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
@@ -127,7 +127,7 @@ bool sxgbe_eee_init(struct sxgbe_priv_data * const priv)
/* MAC core supports the EEE feature. */
if (priv->hw_cap.eee) {
/* Check if the PHY supports EEE */
- if (phy_init_eee(ndev->phydev, 1))
+ if (phy_init_eee(ndev->phydev, true))
return false;
priv->eee_active = 1;
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee
2022-01-23 15:22 [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee Jisheng Zhang
@ 2022-01-23 15:54 ` Andrew Lunn
2022-01-24 17:09 ` Florian Fainelli
2022-01-25 1:10 ` Jakub Kicinski
2 siblings, 0 replies; 4+ messages in thread
From: Andrew Lunn @ 2022-01-23 15:54 UTC (permalink / raw)
To: Jisheng Zhang
Cc: Florian Fainelli, Vivien Didelot, Vladimir Oltean,
David S . Miller, Jakub Kicinski, Sean Wang, Landen Chao,
DENG Qingfang, Matthias Brugger, Doug Berger, Joakim Zhang,
Thomas Petazzoni, Bryan Whitehead, UNGLinuxDriver, Byungho An,
netdev, linux-kernel, linux-arm-kernel, linux-mediatek,
bcm-kernel-feedback-list
On Sun, Jan 23, 2022 at 11:22:41PM +0800, Jisheng Zhang wrote:
> The 2nd param of phy_init_eee(): clk_stop_enable is a bool param, use
> true or false instead of 1/0.
>
> Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Andrew
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee
2022-01-23 15:22 [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee Jisheng Zhang
2022-01-23 15:54 ` Andrew Lunn
@ 2022-01-24 17:09 ` Florian Fainelli
2022-01-25 1:10 ` Jakub Kicinski
2 siblings, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2022-01-24 17:09 UTC (permalink / raw)
To: Jisheng Zhang, Andrew Lunn, Vivien Didelot, Vladimir Oltean,
David S . Miller, Jakub Kicinski, Sean Wang, Landen Chao,
DENG Qingfang, Matthias Brugger, Doug Berger, Joakim Zhang,
Thomas Petazzoni, Bryan Whitehead, UNGLinuxDriver, Byungho An
Cc: netdev, linux-kernel, linux-arm-kernel, linux-mediatek,
bcm-kernel-feedback-list
On 1/23/2022 7:22 AM, Jisheng Zhang wrote:
> The 2nd param of phy_init_eee(): clk_stop_enable is a bool param, use
> true or false instead of 1/0.
>
> Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Nit: for future changes, if your patch spans multiple
subsystems/directories, just go with the top-most subject prefix, for
instance here "net: " would have been sufficient. Thanks!
--
Florian
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee
2022-01-23 15:22 [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee Jisheng Zhang
2022-01-23 15:54 ` Andrew Lunn
2022-01-24 17:09 ` Florian Fainelli
@ 2022-01-25 1:10 ` Jakub Kicinski
2 siblings, 0 replies; 4+ messages in thread
From: Jakub Kicinski @ 2022-01-25 1:10 UTC (permalink / raw)
To: Jisheng Zhang
Cc: Florian Fainelli, Andrew Lunn, Vivien Didelot, Vladimir Oltean,
David S . Miller, Sean Wang, Landen Chao, DENG Qingfang,
Matthias Brugger, Doug Berger, Joakim Zhang, Thomas Petazzoni,
Bryan Whitehead, UNGLinuxDriver, Byungho An, netdev,
linux-kernel, linux-arm-kernel, linux-mediatek,
bcm-kernel-feedback-list
On Sun, 23 Jan 2022 23:22:41 +0800 Jisheng Zhang wrote:
> The 2nd param of phy_init_eee(): clk_stop_enable is a bool param, use
> true or false instead of 1/0.
>
> Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
Applied, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-01-25 3:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-23 15:22 [PATCH] net: dsa|ethernet: use bool values to pass bool param of phy_init_eee Jisheng Zhang
2022-01-23 15:54 ` Andrew Lunn
2022-01-24 17:09 ` Florian Fainelli
2022-01-25 1:10 ` Jakub Kicinski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).