Commit daf22c3c authored by Axel Lin's avatar Axel Lin Committed by Anton Vorontsov

wm97xx_battery: Use DEFINE_MUTEX() for work_lock

work_lock can be initialized automatically with
DEFINE_MUTEX() rather than explicitly calling mutex_init().

This patch also removes an unused bat_lock mutex.
Signed-off-by: default avatarAxel Lin <axel.lin@gmail.com>
Signed-off-by: default avatarAnton Vorontsov <cbouatmailru@gmail.com>
parent 68597986
...@@ -25,9 +25,8 @@ ...@@ -25,9 +25,8 @@
#include <linux/irq.h> #include <linux/irq.h>
#include <linux/slab.h> #include <linux/slab.h>
static DEFINE_MUTEX(bat_lock);
static struct work_struct bat_work; static struct work_struct bat_work;
static struct mutex work_lock; static DEFINE_MUTEX(work_lock);
static int bat_status = POWER_SUPPLY_STATUS_UNKNOWN; static int bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
static enum power_supply_property *prop; static enum power_supply_property *prop;
...@@ -181,8 +180,6 @@ static int __devinit wm97xx_bat_probe(struct platform_device *dev) ...@@ -181,8 +180,6 @@ static int __devinit wm97xx_bat_probe(struct platform_device *dev)
if (dev->id != -1) if (dev->id != -1)
return -EINVAL; return -EINVAL;
mutex_init(&work_lock);
if (!pdata) { if (!pdata) {
dev_err(&dev->dev, "No platform_data supplied\n"); dev_err(&dev->dev, "No platform_data supplied\n");
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment