forked from Openwrt-EcoNet/openwrt
d8375034b8
83d962316128 net: phy: realtek: add RTL8127-internal PHY 708686132ba0 net: phy: realtek: Add support for PHY LEDs on RTL8211E be1cc96ddf82 net: phy: realtek: use __set_bit() in rtl8211f_led_hw_control_get() 8c4d0172657c net: phy: realtek: Group RTL82* macro definitions 12d40df259e3 net: phy: realtek: add RTL8211F register defines 7c6fa3ffd265 net: phy: realtek: Clean up RTL821x ExtPage access f3b265358b91 net: phy: realtek: remove unsed RTL821x_PHYSR* macros 7840e4d6f48a net: phy: realtek: Add support for WOL magic packet on RTL8211F Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
36 lines
1.3 KiB
Diff
36 lines
1.3 KiB
Diff
From 4dd2cc9b91ecb25f278a2c55e07e6455e9000e6b Mon Sep 17 00:00:00 2001
|
|
From: Daniel Golle <daniel@makrotopia.org>
|
|
Date: Sat, 22 Apr 2023 01:21:14 +0100
|
|
Subject: [PATCH] net: phy: realtek: make sure paged read is protected by mutex
|
|
|
|
As we cannot rely on phy_read_paged function before the PHY is
|
|
identified, the paged read in rtlgen_supports_2_5gbps needs to be open
|
|
coded as it is being called by the match_phy_device function, ie. before
|
|
.read_page and .write_page have been populated.
|
|
|
|
Make sure it is also protected by the MDIO bus mutex and use
|
|
rtl821x_write_page instead of 3 individually locked MDIO bus operations.
|
|
|
|
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
|
|
---
|
|
drivers/net/phy/realtek/realtek_main.c | 8 +++++---
|
|
1 file changed, 5 insertions(+), 3 deletions(-)
|
|
|
|
--- a/drivers/net/phy/realtek/realtek_main.c
|
|
+++ b/drivers/net/phy/realtek/realtek_main.c
|
|
@@ -1313,9 +1313,11 @@ static bool rtlgen_supports_2_5gbps(stru
|
|
{
|
|
int val;
|
|
|
|
- phy_write(phydev, RTL821x_PAGE_SELECT, 0xa61);
|
|
- val = phy_read(phydev, 0x13);
|
|
- phy_write(phydev, RTL821x_PAGE_SELECT, 0);
|
|
+ mutex_lock(&phydev->mdio.bus->mdio_lock);
|
|
+ rtl821x_write_page(phydev, 0xa61);
|
|
+ val = __phy_read(phydev, 0x13);
|
|
+ rtl821x_write_page(phydev, 0);
|
|
+ mutex_unlock(&phydev->mdio.bus->mdio_lock);
|
|
|
|
return val >= 0 && val & MDIO_PMA_SPEED_2_5G;
|
|
}
|