netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 net] net: phy: meson-gxl: use MMD access dummy stubs for GXL, internal PHY
@ 2023-02-02 20:45 Heiner Kallweit
  2023-02-04  4:00 ` patchwork-bot+netdevbpf
  0 siblings, 1 reply; 2+ messages in thread
From: Heiner Kallweit @ 2023-02-02 20:45 UTC (permalink / raw)
  To: Andrew Lunn, Russell King - ARM Linux, David Miller,
	Jakub Kicinski, Paolo Abeni, Eric Dumazet, Neil Armstrong,
	Kevin Hilman, Jerome Brunet, Martin Blumenstingl
  Cc: netdev, linux-arm-kernel, open list:ARM/Amlogic Meson..., Chris Healy

Jerome provided the information that also the GXL internal PHY doesn't
support MMD register access and EEE. MMD reads return 0xffff, what
results in e.g. completely wrong ethtool --show-eee output.
Therefore use the MMD dummy stubs.

v2:
- Change Fixes tag to the actually offending commit. As 4.9 is EOL
  this fix will apply on all stable versions.

Fixes: d853d145ea3e ("net: phy: add an option to disable EEE advertisement")
Suggested-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
---
 drivers/net/phy/meson-gxl.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
index c49062ad7..fbf5f2416 100644
--- a/drivers/net/phy/meson-gxl.c
+++ b/drivers/net/phy/meson-gxl.c
@@ -261,6 +261,8 @@ static struct phy_driver meson_gxl_phy[] = {
 		.handle_interrupt = meson_gxl_handle_interrupt,
 		.suspend        = genphy_suspend,
 		.resume         = genphy_resume,
+		.read_mmd	= genphy_read_mmd_unsupported,
+		.write_mmd	= genphy_write_mmd_unsupported,
 	}, {
 		PHY_ID_MATCH_EXACT(0x01803301),
 		.name		= "Meson G12A Internal PHY",
-- 
2.39.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2 net] net: phy: meson-gxl: use MMD access dummy stubs for GXL, internal PHY
  2023-02-02 20:45 [PATCH v2 net] net: phy: meson-gxl: use MMD access dummy stubs for GXL, internal PHY Heiner Kallweit
@ 2023-02-04  4:00 ` patchwork-bot+netdevbpf
  0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2023-02-04  4:00 UTC (permalink / raw)
  To: Heiner Kallweit
  Cc: andrew, linux, davem, kuba, pabeni, edumazet, neil.armstrong,
	khilman, jbrunet, martin.blumenstingl, netdev, linux-arm-kernel,
	linux-amlogic, cphealy

Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Thu, 2 Feb 2023 21:45:36 +0100 you wrote:
> Jerome provided the information that also the GXL internal PHY doesn't
> support MMD register access and EEE. MMD reads return 0xffff, what
> results in e.g. completely wrong ethtool --show-eee output.
> Therefore use the MMD dummy stubs.
> 
> v2:
> - Change Fixes tag to the actually offending commit. As 4.9 is EOL
>   this fix will apply on all stable versions.
> 
> [...]

Here is the summary with links:
  - [v2,net] net: phy: meson-gxl: use MMD access dummy stubs for GXL, internal PHY
    https://git.kernel.org/netdev/net/c/69ff53e4a4c9

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



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-02-04  4:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-02 20:45 [PATCH v2 net] net: phy: meson-gxl: use MMD access dummy stubs for GXL, internal PHY Heiner Kallweit
2023-02-04  4:00 ` patchwork-bot+netdevbpf

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).