diff options
| author | Pekka Enberg <penberg@kernel.org> | 2010-10-24 12:57:05 -0400 |
|---|---|---|
| committer | Pekka Enberg <penberg@kernel.org> | 2010-10-24 12:57:05 -0400 |
| commit | 6d4121f6c20a0e86231d52f535f1c82423b3326f (patch) | |
| tree | 5c235cac699ca86b504850aa663ddadde0455a61 /drivers/rtc | |
| parent | 92a5bbc11ff2442a54b2f1d313088c245828ef4e (diff) | |
| parent | 35da7a307c535f9c2929cae277f3df425c9f9b1e (diff) | |
Merge branch 'master' into for-linus
Conflicts:
include/linux/percpu.h
mm/percpu.c
Diffstat (limited to 'drivers/rtc')
| -rw-r--r-- | drivers/rtc/rtc-ds3232.c | 2 | ||||
| -rw-r--r-- | drivers/rtc/rtc-m41t80.c | 14 |
2 files changed, 8 insertions, 8 deletions
diff --git a/drivers/rtc/rtc-ds3232.c b/drivers/rtc/rtc-ds3232.c index 9daed8db83d3..9de8516e3531 100644 --- a/drivers/rtc/rtc-ds3232.c +++ b/drivers/rtc/rtc-ds3232.c | |||
| @@ -268,7 +268,6 @@ out_irq: | |||
| 268 | free_irq(client->irq, client); | 268 | free_irq(client->irq, client); |
| 269 | 269 | ||
| 270 | out_free: | 270 | out_free: |
| 271 | i2c_set_clientdata(client, NULL); | ||
| 272 | kfree(ds3232); | 271 | kfree(ds3232); |
| 273 | return ret; | 272 | return ret; |
| 274 | } | 273 | } |
| @@ -287,7 +286,6 @@ static int __devexit ds3232_remove(struct i2c_client *client) | |||
| 287 | } | 286 | } |
| 288 | 287 | ||
| 289 | rtc_device_unregister(ds3232->rtc); | 288 | rtc_device_unregister(ds3232->rtc); |
| 290 | i2c_set_clientdata(client, NULL); | ||
| 291 | kfree(ds3232); | 289 | kfree(ds3232); |
| 292 | return 0; | 290 | return 0; |
| 293 | } | 291 | } |
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c index d60557cae8ef..5a8daa358066 100644 --- a/drivers/rtc/rtc-m41t80.c +++ b/drivers/rtc/rtc-m41t80.c | |||
| @@ -20,7 +20,7 @@ | |||
| 20 | #include <linux/module.h> | 20 | #include <linux/module.h> |
| 21 | #include <linux/rtc.h> | 21 | #include <linux/rtc.h> |
| 22 | #include <linux/slab.h> | 22 | #include <linux/slab.h> |
| 23 | #include <linux/smp_lock.h> | 23 | #include <linux/mutex.h> |
| 24 | #include <linux/string.h> | 24 | #include <linux/string.h> |
| 25 | #ifdef CONFIG_RTC_DRV_M41T80_WDT | 25 | #ifdef CONFIG_RTC_DRV_M41T80_WDT |
| 26 | #include <linux/fs.h> | 26 | #include <linux/fs.h> |
| @@ -68,6 +68,7 @@ | |||
| 68 | 68 | ||
| 69 | #define DRV_VERSION "0.05" | 69 | #define DRV_VERSION "0.05" |
| 70 | 70 | ||
| 71 | static DEFINE_MUTEX(m41t80_rtc_mutex); | ||
| 71 | static const struct i2c_device_id m41t80_id[] = { | 72 | static const struct i2c_device_id m41t80_id[] = { |
| 72 | { "m41t62", M41T80_FEATURE_SQ | M41T80_FEATURE_SQ_ALT }, | 73 | { "m41t62", M41T80_FEATURE_SQ | M41T80_FEATURE_SQ_ALT }, |
| 73 | { "m41t65", M41T80_FEATURE_HT | M41T80_FEATURE_WD }, | 74 | { "m41t65", M41T80_FEATURE_HT | M41T80_FEATURE_WD }, |
| @@ -677,9 +678,9 @@ static long wdt_unlocked_ioctl(struct file *file, unsigned int cmd, | |||
| 677 | { | 678 | { |
| 678 | int ret; | 679 | int ret; |
| 679 | 680 | ||
| 680 | lock_kernel(); | 681 | mutex_lock(&m41t80_rtc_mutex); |
| 681 | ret = wdt_ioctl(file, cmd, arg); | 682 | ret = wdt_ioctl(file, cmd, arg); |
| 682 | unlock_kernel(); | 683 | mutex_unlock(&m41t80_rtc_mutex); |
| 683 | 684 | ||
| 684 | return ret; | 685 | return ret; |
| 685 | } | 686 | } |
| @@ -693,16 +694,16 @@ static long wdt_unlocked_ioctl(struct file *file, unsigned int cmd, | |||
| 693 | static int wdt_open(struct inode *inode, struct file *file) | 694 | static int wdt_open(struct inode *inode, struct file *file) |
| 694 | { | 695 | { |
| 695 | if (MINOR(inode->i_rdev) == WATCHDOG_MINOR) { | 696 | if (MINOR(inode->i_rdev) == WATCHDOG_MINOR) { |
| 696 | lock_kernel(); | 697 | mutex_lock(&m41t80_rtc_mutex); |
| 697 | if (test_and_set_bit(0, &wdt_is_open)) { | 698 | if (test_and_set_bit(0, &wdt_is_open)) { |
| 698 | unlock_kernel(); | 699 | mutex_unlock(&m41t80_rtc_mutex); |
| 699 | return -EBUSY; | 700 | return -EBUSY; |
| 700 | } | 701 | } |
| 701 | /* | 702 | /* |
| 702 | * Activate | 703 | * Activate |
| 703 | */ | 704 | */ |
| 704 | wdt_is_open = 1; | 705 | wdt_is_open = 1; |
| 705 | unlock_kernel(); | 706 | mutex_unlock(&m41t80_rtc_mutex); |
| 706 | return nonseekable_open(inode, file); | 707 | return nonseekable_open(inode, file); |
| 707 | } | 708 | } |
| 708 | return -ENODEV; | 709 | return -ENODEV; |
| @@ -748,6 +749,7 @@ static const struct file_operations wdt_fops = { | |||
| 748 | .write = wdt_write, | 749 | .write = wdt_write, |
| 749 | .open = wdt_open, | 750 | .open = wdt_open, |
| 750 | .release = wdt_release, | 751 | .release = wdt_release, |
| 752 | .llseek = no_llseek, | ||
| 751 | }; | 753 | }; |
| 752 | 754 | ||
| 753 | static struct miscdevice wdt_dev = { | 755 | static struct miscdevice wdt_dev = { |
