diff options
| author | Christian Marangi <ansuelsmth@gmail.com> | 2023-04-23 19:28:00 +0200 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2023-04-25 04:01:47 +0200 |
| commit | 4774ad841bef97cc51df90195338c5b2573dd4cb (patch) | |
| tree | 7aaa45da5d733420079462bfe24a8280adbacc06 /drivers | |
| parent | lan966x: Don't use xdp_frame when action is XDP_TX (diff) | |
| download | linux-4774ad841bef97cc51df90195338c5b2573dd4cb.tar.xz linux-4774ad841bef97cc51df90195338c5b2573dd4cb.zip | |
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>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/phy/marvell.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c index cd5d0ed9c5e5..43b6cb725551 100644 --- a/drivers/net/phy/marvell.c +++ b/drivers/net/phy/marvell.c @@ -2880,10 +2880,10 @@ static int m88e1318_led_blink_set(struct phy_device *phydev, u8 index, 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; |