| Message ID | 20210717123249.56505-1-marex@denx.de |
|---|---|
| State | New |
| Headers | show |
| Series | net: phy: Fix data type in DP83822 dp8382x_disable_wol() | expand |
On 7/17/2021 5:32 AM, Marek Vasut wrote: > The last argument of phy_clear_bits_mmd(..., u16 val); is u16 and not > int, just inline the value into the function call arguments. > > No functional change. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Andrew Lunn <andrew@lunn.ch> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: David S. Miller <davem@davemloft.net> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
On Sat, Jul 17, 2021 at 02:32:49PM +0200, Marek Vasut wrote: > The last argument of phy_clear_bits_mmd(..., u16 val); is u16 and not > int, just inline the value into the function call arguments. > > No functional change. I don't think there is an overflow issue here, so this can go into net-next. The subject should reflect this, Subject: [PATCH net-next] ... Thanks for splitting it into a standalone patch. > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Andrew Lunn <andrew@lunn.ch> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: David S. Miller <davem@davemloft.net> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Sat, 17 Jul 2021 14:32:49 +0200 you wrote: > The last argument of phy_clear_bits_mmd(..., u16 val); is u16 and not > int, just inline the value into the function call arguments. > > No functional change. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Andrew Lunn <andrew@lunn.ch> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: David S. Miller <davem@davemloft.net> > > [...] Here is the summary with links: - net: phy: Fix data type in DP83822 dp8382x_disable_wol() https://git.kernel.org/netdev/net-next/c/0d6835ffe50c You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/drivers/net/phy/dp83822.c b/drivers/net/phy/dp83822.c index f7a2ec150e54..211b5476a6f5 100644 --- a/drivers/net/phy/dp83822.c +++ b/drivers/net/phy/dp83822.c @@ -326,11 +326,9 @@ static irqreturn_t dp83822_handle_interrupt(struct phy_device *phydev) static int dp8382x_disable_wol(struct phy_device *phydev) { - int value = DP83822_WOL_EN | DP83822_WOL_MAGIC_EN | - DP83822_WOL_SECURE_ON; - - return phy_clear_bits_mmd(phydev, DP83822_DEVADDR, - MII_DP83822_WOL_CFG, value); + return phy_clear_bits_mmd(phydev, DP83822_DEVADDR, MII_DP83822_WOL_CFG, + DP83822_WOL_EN | DP83822_WOL_MAGIC_EN | + DP83822_WOL_SECURE_ON); } static int dp83822_read_status(struct phy_device *phydev)
The last argument of phy_clear_bits_mmd(..., u16 val); is u16 and not int, just inline the value into the function call arguments. No functional change. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Andrew Lunn <andrew@lunn.ch> Cc: Florian Fainelli <f.fainelli@gmail.com> Cc: David S. Miller <davem@davemloft.net> --- drivers/net/phy/dp83822.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-)