diff options
| author | Jonas Jelonek | 2025-06-18 21:40:03 +0000 |
|---|---|---|
| committer | Hauke Mehrtens | 2025-07-13 14:47:42 +0000 |
| commit | 6432b411805fad2e35ad2f3726bcb9d4cd58ce89 (patch) | |
| tree | edb7e692753d3345df33b0e5864135dd97f12f05 | |
| parent | 655eb05699975fff2e1df96bf80cb284d3215ec3 (diff) | |
| download | openwrt-6432b411805fad2e35ad2f3726bcb9d4cd58ce89.tar.gz | |
realtek: rtl931x: fix setting number of leds per port
In rtl931x_led_init, the number of leds per port is not properly set. It
currently uses a hardcoded value of 1 which seems to be taken initially
from a specific device. This hardcoded value assumes any port always has
exactly two leds.
The RTL930x variant - rtl930x_led_init - does a better job at this. So
take it and use it for RTL931x too with the corresponding register.
While at it, rename the function to a proper naming scheme.
Signed-off-by: Jonas Jelonek <jelonek.jonas@gmail.com>
Link: https://github.com/openwrt/openwrt/pull/19241
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
| -rw-r--r-- | target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/rtl931x.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/rtl931x.c b/target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/rtl931x.c index 5ce5913ad5..a3277992ae 100644 --- a/target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/rtl931x.c +++ b/target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/rtl931x.c @@ -1554,7 +1554,7 @@ static void rtl931x_set_distribution_algorithm(int group, int algoidx, u32 algom sw_w32(newmask << l3shift, RTL931X_TRK_HASH_CTRL + (algoidx << 2)); } -static void rtl931x_led_init(struct rtl838x_switch_priv *priv) +static void rtldsa_931x_led_init(struct rtl838x_switch_priv *priv) { u64 pm_copper = 0, pm_fiber = 0; struct device_node *node; @@ -1569,7 +1569,6 @@ static void rtl931x_led_init(struct rtl838x_switch_priv *priv) for (int i = 0; i < priv->cpu_port; i++) { int pos = (i << 1) % 32; u32 set; - u32 v; sw_w32_mask(0x3 << pos, 0, RTL931X_LED_PORT_FIB_SET_SEL_CTRL(i)); sw_w32_mask(0x3 << pos, 0, RTL931X_LED_PORT_COPR_SET_SEL_CTRL(i)); @@ -1577,8 +1576,9 @@ static void rtl931x_led_init(struct rtl838x_switch_priv *priv) if (!priv->ports[i].phy) continue; - v = 0x1; /* Found on the EdgeCore, but we do not have any HW description */ - sw_w32_mask(0x3 << pos, v << pos, RTL931X_LED_PORT_NUM_CTRL(i)); + /* 0x0 = 1 led, 0x1 = 2 leds, 0x2 = 3 leds, 0x3 = 4 leds per port */ + sw_w32_mask(0x3 << pos, (priv->ports[i].leds_on_this_port - 1) << pos, + RTL931X_LED_PORT_NUM_CTRL(i)); if (priv->ports[i].phy_is_integrated) pm_fiber |= BIT_ULL(i); @@ -1686,5 +1686,5 @@ const struct rtl838x_reg rtl931x_reg = { .pie_rule_rm = rtl931x_pie_rule_rm, .l2_learning_setup = rtl931x_l2_learning_setup, .l3_setup = rtl931x_l3_setup, - .led_init = rtl931x_led_init, + .led_init = rtldsa_931x_led_init, }; |