From: Axel Lin Date: Mon, 28 Nov 2011 14:37:35 +0000 (+0800) Subject: wm97xx_battery: Use DEFINE_MUTEX() for work_lock X-Git-Tag: upstream/snapshot3+hdmi~8366^2~15 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=daf22c3c4444e10a852a2f460f5cb1a43038bd53;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git 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: Axel Lin Signed-off-by: Anton Vorontsov --- diff --git a/drivers/power/wm97xx_battery.c b/drivers/power/wm97xx_battery.c index 199818f..d2d4c08 100644 --- a/drivers/power/wm97xx_battery.c +++ b/drivers/power/wm97xx_battery.c @@ -25,9 +25,8 @@ #include #include -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; @@ -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;