Skip to content

Commit

Permalink
wm97xx_battery: Use DEFINE_MUTEX() for work_lock
Browse files Browse the repository at this point in the history
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: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Anton Vorontsov <cbouatmailru@gmail.com>
  • Loading branch information
Axel Lin authored and Anton Vorontsov committed Jan 6, 2012
1 parent 6859798 commit daf22c3
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions drivers/power/wm97xx_battery.c
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,8 @@
#include <linux/irq.h>
#include <linux/slab.h>

static DEFINE_MUTEX(bat_lock);
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 enum power_supply_property *prop;

Expand Down Expand Up @@ -181,8 +180,6 @@ static int __devinit wm97xx_bat_probe(struct platform_device *dev)
if (dev->id != -1)
return -EINVAL;

mutex_init(&work_lock);

if (!pdata) {
dev_err(&dev->dev, "No platform_data supplied\n");
return -EINVAL;
Expand Down

0 comments on commit daf22c3

Please sign in to comment.