diff options
author | David S. Miller <davem@davemloft.net> | 2010-01-23 01:45:46 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-01-23 01:45:46 -0500 |
commit | 6be325719b3e54624397e413efd4b33a997e55a3 (patch) | |
tree | 57f321a56794cab2222e179b16731e0d76a4a68a /drivers/char/sonypi.c | |
parent | 26d92f9276a56d55511a427fb70bd70886af647a (diff) | |
parent | 92dcffb916d309aa01778bf8963a6932e4014d07 (diff) |
Merge branch 'master' of /home/davem/src/GIT/linux-2.6/
Diffstat (limited to 'drivers/char/sonypi.c')
-rw-r--r-- | drivers/char/sonypi.c | 60 |
1 files changed, 30 insertions, 30 deletions
diff --git a/drivers/char/sonypi.c b/drivers/char/sonypi.c index 8c262aaf7c26..bba727c3807e 100644 --- a/drivers/char/sonypi.c +++ b/drivers/char/sonypi.c | |||
@@ -50,7 +50,6 @@ | |||
50 | #include <linux/err.h> | 50 | #include <linux/err.h> |
51 | #include <linux/kfifo.h> | 51 | #include <linux/kfifo.h> |
52 | #include <linux/platform_device.h> | 52 | #include <linux/platform_device.h> |
53 | #include <linux/smp_lock.h> | ||
54 | 53 | ||
55 | #include <asm/uaccess.h> | 54 | #include <asm/uaccess.h> |
56 | #include <asm/io.h> | 55 | #include <asm/io.h> |
@@ -487,7 +486,7 @@ static struct sonypi_device { | |||
487 | int camera_power; | 486 | int camera_power; |
488 | int bluetooth_power; | 487 | int bluetooth_power; |
489 | struct mutex lock; | 488 | struct mutex lock; |
490 | struct kfifo *fifo; | 489 | struct kfifo fifo; |
491 | spinlock_t fifo_lock; | 490 | spinlock_t fifo_lock; |
492 | wait_queue_head_t fifo_proc_list; | 491 | wait_queue_head_t fifo_proc_list; |
493 | struct fasync_struct *fifo_async; | 492 | struct fasync_struct *fifo_async; |
@@ -496,7 +495,7 @@ static struct sonypi_device { | |||
496 | struct input_dev *input_jog_dev; | 495 | struct input_dev *input_jog_dev; |
497 | struct input_dev *input_key_dev; | 496 | struct input_dev *input_key_dev; |
498 | struct work_struct input_work; | 497 | struct work_struct input_work; |
499 | struct kfifo *input_fifo; | 498 | struct kfifo input_fifo; |
500 | spinlock_t input_fifo_lock; | 499 | spinlock_t input_fifo_lock; |
501 | } sonypi_device; | 500 | } sonypi_device; |
502 | 501 | ||
@@ -777,8 +776,9 @@ static void input_keyrelease(struct work_struct *work) | |||
777 | { | 776 | { |
778 | struct sonypi_keypress kp; | 777 | struct sonypi_keypress kp; |
779 | 778 | ||
780 | while (kfifo_get(sonypi_device.input_fifo, (unsigned char *)&kp, | 779 | while (kfifo_out_locked(&sonypi_device.input_fifo, (unsigned char *)&kp, |
781 | sizeof(kp)) == sizeof(kp)) { | 780 | sizeof(kp), &sonypi_device.input_fifo_lock) |
781 | == sizeof(kp)) { | ||
782 | msleep(10); | 782 | msleep(10); |
783 | input_report_key(kp.dev, kp.key, 0); | 783 | input_report_key(kp.dev, kp.key, 0); |
784 | input_sync(kp.dev); | 784 | input_sync(kp.dev); |
@@ -827,8 +827,9 @@ static void sonypi_report_input_event(u8 event) | |||
827 | if (kp.dev) { | 827 | if (kp.dev) { |
828 | input_report_key(kp.dev, kp.key, 1); | 828 | input_report_key(kp.dev, kp.key, 1); |
829 | input_sync(kp.dev); | 829 | input_sync(kp.dev); |
830 | kfifo_put(sonypi_device.input_fifo, | 830 | kfifo_in_locked(&sonypi_device.input_fifo, |
831 | (unsigned char *)&kp, sizeof(kp)); | 831 | (unsigned char *)&kp, sizeof(kp), |
832 | &sonypi_device.input_fifo_lock); | ||
832 | schedule_work(&sonypi_device.input_work); | 833 | schedule_work(&sonypi_device.input_work); |
833 | } | 834 | } |
834 | } | 835 | } |
@@ -880,7 +881,8 @@ found: | |||
880 | acpi_bus_generate_proc_event(sonypi_acpi_device, 1, event); | 881 | acpi_bus_generate_proc_event(sonypi_acpi_device, 1, event); |
881 | #endif | 882 | #endif |
882 | 883 | ||
883 | kfifo_put(sonypi_device.fifo, (unsigned char *)&event, sizeof(event)); | 884 | kfifo_in_locked(&sonypi_device.fifo, (unsigned char *)&event, |
885 | sizeof(event), &sonypi_device.fifo_lock); | ||
884 | kill_fasync(&sonypi_device.fifo_async, SIGIO, POLL_IN); | 886 | kill_fasync(&sonypi_device.fifo_async, SIGIO, POLL_IN); |
885 | wake_up_interruptible(&sonypi_device.fifo_proc_list); | 887 | wake_up_interruptible(&sonypi_device.fifo_proc_list); |
886 | 888 | ||
@@ -902,14 +904,13 @@ static int sonypi_misc_release(struct inode *inode, struct file *file) | |||
902 | 904 | ||
903 | static int sonypi_misc_open(struct inode *inode, struct file *file) | 905 | static int sonypi_misc_open(struct inode *inode, struct file *file) |
904 | { | 906 | { |
905 | lock_kernel(); | ||
906 | mutex_lock(&sonypi_device.lock); | 907 | mutex_lock(&sonypi_device.lock); |
907 | /* Flush input queue on first open */ | 908 | /* Flush input queue on first open */ |
908 | if (!sonypi_device.open_count) | 909 | if (!sonypi_device.open_count) |
909 | kfifo_reset(sonypi_device.fifo); | 910 | kfifo_reset(&sonypi_device.fifo); |
910 | sonypi_device.open_count++; | 911 | sonypi_device.open_count++; |
911 | mutex_unlock(&sonypi_device.lock); | 912 | mutex_unlock(&sonypi_device.lock); |
912 | unlock_kernel(); | 913 | |
913 | return 0; | 914 | return 0; |
914 | } | 915 | } |
915 | 916 | ||
@@ -919,17 +920,18 @@ static ssize_t sonypi_misc_read(struct file *file, char __user *buf, | |||
919 | ssize_t ret; | 920 | ssize_t ret; |
920 | unsigned char c; | 921 | unsigned char c; |
921 | 922 | ||
922 | if ((kfifo_len(sonypi_device.fifo) == 0) && | 923 | if ((kfifo_len(&sonypi_device.fifo) == 0) && |
923 | (file->f_flags & O_NONBLOCK)) | 924 | (file->f_flags & O_NONBLOCK)) |
924 | return -EAGAIN; | 925 | return -EAGAIN; |
925 | 926 | ||
926 | ret = wait_event_interruptible(sonypi_device.fifo_proc_list, | 927 | ret = wait_event_interruptible(sonypi_device.fifo_proc_list, |
927 | kfifo_len(sonypi_device.fifo) != 0); | 928 | kfifo_len(&sonypi_device.fifo) != 0); |
928 | if (ret) | 929 | if (ret) |
929 | return ret; | 930 | return ret; |
930 | 931 | ||
931 | while (ret < count && | 932 | while (ret < count && |
932 | (kfifo_get(sonypi_device.fifo, &c, sizeof(c)) == sizeof(c))) { | 933 | (kfifo_out_locked(&sonypi_device.fifo, &c, sizeof(c), |
934 | &sonypi_device.fifo_lock) == sizeof(c))) { | ||
933 | if (put_user(c, buf++)) | 935 | if (put_user(c, buf++)) |
934 | return -EFAULT; | 936 | return -EFAULT; |
935 | ret++; | 937 | ret++; |
@@ -946,15 +948,15 @@ static ssize_t sonypi_misc_read(struct file *file, char __user *buf, | |||
946 | static unsigned int sonypi_misc_poll(struct file *file, poll_table *wait) | 948 | static unsigned int sonypi_misc_poll(struct file *file, poll_table *wait) |
947 | { | 949 | { |
948 | poll_wait(file, &sonypi_device.fifo_proc_list, wait); | 950 | poll_wait(file, &sonypi_device.fifo_proc_list, wait); |
949 | if (kfifo_len(sonypi_device.fifo)) | 951 | if (kfifo_len(&sonypi_device.fifo)) |
950 | return POLLIN | POLLRDNORM; | 952 | return POLLIN | POLLRDNORM; |
951 | return 0; | 953 | return 0; |
952 | } | 954 | } |
953 | 955 | ||
954 | static int sonypi_misc_ioctl(struct inode *ip, struct file *fp, | 956 | static long sonypi_misc_ioctl(struct file *fp, |
955 | unsigned int cmd, unsigned long arg) | 957 | unsigned int cmd, unsigned long arg) |
956 | { | 958 | { |
957 | int ret = 0; | 959 | long ret = 0; |
958 | void __user *argp = (void __user *)arg; | 960 | void __user *argp = (void __user *)arg; |
959 | u8 val8; | 961 | u8 val8; |
960 | u16 val16; | 962 | u16 val16; |
@@ -1070,7 +1072,8 @@ static const struct file_operations sonypi_misc_fops = { | |||
1070 | .open = sonypi_misc_open, | 1072 | .open = sonypi_misc_open, |
1071 | .release = sonypi_misc_release, | 1073 | .release = sonypi_misc_release, |
1072 | .fasync = sonypi_misc_fasync, | 1074 | .fasync = sonypi_misc_fasync, |
1073 | .ioctl = sonypi_misc_ioctl, | 1075 | .unlocked_ioctl = sonypi_misc_ioctl, |
1076 | .llseek = no_llseek, | ||
1074 | }; | 1077 | }; |
1075 | 1078 | ||
1076 | static struct miscdevice sonypi_misc_device = { | 1079 | static struct miscdevice sonypi_misc_device = { |
@@ -1313,11 +1316,10 @@ static int __devinit sonypi_probe(struct platform_device *dev) | |||
1313 | "http://www.linux.it/~malattia/wiki/index.php/Sony_drivers\n"); | 1316 | "http://www.linux.it/~malattia/wiki/index.php/Sony_drivers\n"); |
1314 | 1317 | ||
1315 | spin_lock_init(&sonypi_device.fifo_lock); | 1318 | spin_lock_init(&sonypi_device.fifo_lock); |
1316 | sonypi_device.fifo = kfifo_alloc(SONYPI_BUF_SIZE, GFP_KERNEL, | 1319 | error = kfifo_alloc(&sonypi_device.fifo, SONYPI_BUF_SIZE, GFP_KERNEL); |
1317 | &sonypi_device.fifo_lock); | 1320 | if (error) { |
1318 | if (IS_ERR(sonypi_device.fifo)) { | ||
1319 | printk(KERN_ERR "sonypi: kfifo_alloc failed\n"); | 1321 | printk(KERN_ERR "sonypi: kfifo_alloc failed\n"); |
1320 | return PTR_ERR(sonypi_device.fifo); | 1322 | return error; |
1321 | } | 1323 | } |
1322 | 1324 | ||
1323 | init_waitqueue_head(&sonypi_device.fifo_proc_list); | 1325 | init_waitqueue_head(&sonypi_device.fifo_proc_list); |
@@ -1393,12 +1395,10 @@ static int __devinit sonypi_probe(struct platform_device *dev) | |||
1393 | } | 1395 | } |
1394 | 1396 | ||
1395 | spin_lock_init(&sonypi_device.input_fifo_lock); | 1397 | spin_lock_init(&sonypi_device.input_fifo_lock); |
1396 | sonypi_device.input_fifo = | 1398 | error = kfifo_alloc(&sonypi_device.input_fifo, SONYPI_BUF_SIZE, |
1397 | kfifo_alloc(SONYPI_BUF_SIZE, GFP_KERNEL, | 1399 | GFP_KERNEL); |
1398 | &sonypi_device.input_fifo_lock); | 1400 | if (error) { |
1399 | if (IS_ERR(sonypi_device.input_fifo)) { | ||
1400 | printk(KERN_ERR "sonypi: kfifo_alloc failed\n"); | 1401 | printk(KERN_ERR "sonypi: kfifo_alloc failed\n"); |
1401 | error = PTR_ERR(sonypi_device.input_fifo); | ||
1402 | goto err_inpdev_unregister; | 1402 | goto err_inpdev_unregister; |
1403 | } | 1403 | } |
1404 | 1404 | ||
@@ -1423,7 +1423,7 @@ static int __devinit sonypi_probe(struct platform_device *dev) | |||
1423 | pci_disable_device(pcidev); | 1423 | pci_disable_device(pcidev); |
1424 | err_put_pcidev: | 1424 | err_put_pcidev: |
1425 | pci_dev_put(pcidev); | 1425 | pci_dev_put(pcidev); |
1426 | kfifo_free(sonypi_device.fifo); | 1426 | kfifo_free(&sonypi_device.fifo); |
1427 | 1427 | ||
1428 | return error; | 1428 | return error; |
1429 | } | 1429 | } |
@@ -1438,7 +1438,7 @@ static int __devexit sonypi_remove(struct platform_device *dev) | |||
1438 | if (useinput) { | 1438 | if (useinput) { |
1439 | input_unregister_device(sonypi_device.input_key_dev); | 1439 | input_unregister_device(sonypi_device.input_key_dev); |
1440 | input_unregister_device(sonypi_device.input_jog_dev); | 1440 | input_unregister_device(sonypi_device.input_jog_dev); |
1441 | kfifo_free(sonypi_device.input_fifo); | 1441 | kfifo_free(&sonypi_device.input_fifo); |
1442 | } | 1442 | } |
1443 | 1443 | ||
1444 | misc_deregister(&sonypi_misc_device); | 1444 | misc_deregister(&sonypi_misc_device); |
@@ -1451,7 +1451,7 @@ static int __devexit sonypi_remove(struct platform_device *dev) | |||
1451 | pci_dev_put(sonypi_device.dev); | 1451 | pci_dev_put(sonypi_device.dev); |
1452 | } | 1452 | } |
1453 | 1453 | ||
1454 | kfifo_free(sonypi_device.fifo); | 1454 | kfifo_free(&sonypi_device.fifo); |
1455 | 1455 | ||
1456 | return 0; | 1456 | return 0; |
1457 | } | 1457 | } |