2021-02-09 01:04:28

by Michael Walle

Subject: [PATCH net-next] net: phy: drop explicit genphy_read_status() op

genphy_read_status() is already the default for the .read_status() op.
Drop the unnecessary references.

Signed-off-by: Michael Walle <[email protected]>
---
drivers/net/phy/marvell.c | 1 -
drivers/net/phy/micrel.c | 1 -
2 files changed, 2 deletions(-)

diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index 620052c023a5..b523aa37ebf0 100644
--- a/drivers/net/phy/marvell.c
+++ b/drivers/net/phy/marvell.c
@@ -2852,7 +2852,6 @@ static struct phy_driver marvell_drivers[] = {
.probe = marvell_probe,
.config_init = m88e1145_config_init,
.config_aneg = m88e1101_config_aneg,
- .read_status = genphy_read_status,
.config_intr = marvell_config_intr,
.handle_interrupt = marvell_handle_interrupt,
.resume = genphy_resume,
diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c
index 39c7c786a912..494abf608b8f 100644
--- a/drivers/net/phy/micrel.c
+++ b/drivers/net/phy/micrel.c
@@ -1368,7 +1368,6 @@ static struct phy_driver ksphy_driver[] = {
.driver_data = &ksz9021_type,
.probe = kszphy_probe,
.config_init = ksz9131_config_init,
- .read_status = genphy_read_status,
.config_intr = kszphy_config_intr,
.handle_interrupt = kszphy_handle_interrupt,
.get_sset_count = kszphy_get_sset_count,
--
2.20.1

2021-02-09 01:25:36

by Andrew Lunn

Subject: Re: [PATCH net-next] net: phy: drop explicit genphy_read_status() op

On Tue, Feb 09, 2021 at 02:00:18AM +0100, Michael Walle wrote:
> genphy_read_status() is already the default for the .read_status() op.
> Drop the unnecessary references.
>
> Signed-off-by: Michael Walle <[email protected]>

Reviewed-by: Andrew Lunn <[email protected]>

Andrew

2021-02-09 23:07:16

by patchwork-bot+netdevbpf

Subject: Re: [PATCH net-next] net: phy: drop explicit genphy_read_status() op

Hello:

This patch was applied to netdev/net-next.git (refs/heads/master):

On Tue, 9 Feb 2021 02:00:18 +0100 you wrote:
> genphy_read_status() is already the default for the .read_status() op.
> Drop the unnecessary references.
>
> Signed-off-by: Michael Walle <[email protected]>
> ---
> drivers/net/phy/marvell.c | 1 -
> drivers/net/phy/micrel.c | 1 -
> 2 files changed, 2 deletions(-)

Here is the summary with links:
- [net-next] net: phy: drop explicit genphy_read_status() op
https://git.kernel.org/netdev/net-next/c/f15008fbaa33

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html