mirror of
https://github.com/openwrt/openwrt.git
synced 2024-12-22 06:57:57 +00:00
cf08db988b
Backport initial LEDs hw control support. Currently this is limited to
only rx/tx and link events for the netdev trigger but the API got
accepted and the additional modes are working on and will be backported
later.
Refresh every patch and add the additional config flag for QCA8K new
LEDs support.
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
(cherry picked from commit 0a4b309f41)
39 lines
1.4 KiB
Diff
39 lines
1.4 KiB
Diff
From 4774ad841bef97cc51df90195338c5b2573dd4cb Mon Sep 17 00:00:00 2001
|
|
From: Christian Marangi <ansuelsmth@gmail.com>
|
|
Date: Sun, 23 Apr 2023 19:28:00 +0200
|
|
Subject: [PATCH] net: phy: marvell: Fix inconsistent indenting in
|
|
led_blink_set
|
|
|
|
Fix inconsistent indeinting in m88e1318_led_blink_set reported by kernel
|
|
test robot, probably done by the presence of an if condition dropped in
|
|
later revision of the same code.
|
|
|
|
Reported-by: kernel test robot <lkp@intel.com>
|
|
Link: https://lore.kernel.org/oe-kbuild-all/202304240007.0VEX8QYG-lkp@intel.com/
|
|
Fixes: ea9e86485dec ("net: phy: marvell: Implement led_blink_set()")
|
|
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
|
|
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
|
|
Link: https://lore.kernel.org/r/20230423172800.3470-1-ansuelsmth@gmail.com
|
|
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
|
|
---
|
|
drivers/net/phy/marvell.c | 8 ++++----
|
|
1 file changed, 4 insertions(+), 4 deletions(-)
|
|
|
|
--- a/drivers/net/phy/marvell.c
|
|
+++ b/drivers/net/phy/marvell.c
|
|
@@ -2841,10 +2841,10 @@ static int m88e1318_led_blink_set(struct
|
|
case 1:
|
|
case 2:
|
|
reg &= ~(0xf << (4 * index));
|
|
- reg |= MII_88E1318S_PHY_LED_FUNC_BLINK << (4 * index);
|
|
- /* Reset default is 84ms */
|
|
- *delay_on = 84 / 2;
|
|
- *delay_off = 84 / 2;
|
|
+ reg |= MII_88E1318S_PHY_LED_FUNC_BLINK << (4 * index);
|
|
+ /* Reset default is 84ms */
|
|
+ *delay_on = 84 / 2;
|
|
+ *delay_off = 84 / 2;
|
|
break;
|
|
default:
|
|
return -EINVAL;
|