aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Kubelun <be.dissent@gmail.com>2016-11-28 15:11:14 +0300
committerJohn Crispin <john@phrozen.org>2016-12-01 15:47:43 +0100
commit9aa734f8f5f3a2804a562f14091b684614941249 (patch)
tree2a3a6c575572269ada213d3423c6109a30d38c91
parent7de8d5322e8a90cf073a92e2c9c479d4667c94b9 (diff)
downloadupstream-9aa734f8f5f3a2804a562f14091b684614941249.tar.gz
upstream-9aa734f8f5f3a2804a562f14091b684614941249.tar.bz2
upstream-9aa734f8f5f3a2804a562f14091b684614941249.zip
net: ar8327: remove unnecessary spinlocks
Import from https://chromium.googlesource.com/chromiumos/third_party/kernel/+/541c15f8dd02ddce07e0a462a0c2572d8398bccc%5E%21/#F0 Signed-off-by: Pavel Kubelun <be.dissent@gmail.com> CHROMIUM: drivers: ar8216: remove unnecessary spinlocks BUG=chrome-os-partner:33096 TEST=none Change-Id: Ia1b51258504501863fd3298717cc923a1baf34ca Signed-off-by: Mathieu Olivari <mathieu@codeaurora.org> Reviewed-on: https://chromium-review.googlesource.com/232828 Reviewed-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-by: Toshi Kikuchi <toshik@chromium.org> Tested-by: Toshi Kikuchi <toshik@chromium.org>
-rw-r--r--target/linux/generic/files/drivers/net/phy/ar8327.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/target/linux/generic/files/drivers/net/phy/ar8327.c b/target/linux/generic/files/drivers/net/phy/ar8327.c
index e7e62e7e0d..d5cc97bc33 100644
--- a/target/linux/generic/files/drivers/net/phy/ar8327.c
+++ b/target/linux/generic/files/drivers/net/phy/ar8327.c
@@ -223,9 +223,7 @@ ar8327_led_work_func(struct work_struct *work)
aled = container_of(work, struct ar8327_led, led_work);
- spin_lock(&aled->lock);
pattern = aled->pattern;
- spin_unlock(&aled->lock);
ar8327_set_led_pattern(aled->sw_priv, aled->led_num,
pattern);
@@ -308,9 +306,7 @@ ar8327_led_enable_hw_mode_show(struct device *dev,
struct ar8327_led *aled = led_cdev_to_ar8327_led(led_cdev);
ssize_t ret = 0;
- spin_lock(&aled->lock);
ret += sprintf(buf, "%d\n", aled->enable_hw_mode);
- spin_unlock(&aled->lock);
return ret;
}