diff options
| author | Alexey Dobriyan <adobriyan@gmail.com> | 2009-09-23 18:57:19 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-24 10:21:04 -0400 |
| commit | 8d65af789f3e2cf4cfbdbf71a0f7a61ebcd41d38 (patch) | |
| tree | 121df3bfffc7853ac6d2c514ad514d4a748a0933 /drivers | |
| parent | c0d0787b6d47d9f4d5e8bd321921104e854a9135 (diff) | |
sysctl: remove "struct file *" argument of ->proc_handler
It's unused.
It isn't needed -- read or write flag is already passed and sysctl
shouldn't care about the rest.
It _was_ used in two places at arch/frv for some reason.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Cc: David Howells <dhowells@redhat.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: James Morris <jmorris@namei.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/cdrom/cdrom.c | 8 | ||||
| -rw-r--r-- | drivers/char/random.c | 4 | ||||
| -rw-r--r-- | drivers/net/wireless/arlan-proc.c | 28 | ||||
| -rw-r--r-- | drivers/parport/procfs.c | 12 |
4 files changed, 26 insertions, 26 deletions
diff --git a/drivers/cdrom/cdrom.c b/drivers/cdrom/cdrom.c index 71d1b9bab70b..614da5b8613a 100644 --- a/drivers/cdrom/cdrom.c +++ b/drivers/cdrom/cdrom.c | |||
| @@ -3412,7 +3412,7 @@ static int cdrom_print_info(const char *header, int val, char *info, | |||
| 3412 | return 0; | 3412 | return 0; |
| 3413 | } | 3413 | } |
| 3414 | 3414 | ||
| 3415 | static int cdrom_sysctl_info(ctl_table *ctl, int write, struct file * filp, | 3415 | static int cdrom_sysctl_info(ctl_table *ctl, int write, |
| 3416 | void __user *buffer, size_t *lenp, loff_t *ppos) | 3416 | void __user *buffer, size_t *lenp, loff_t *ppos) |
| 3417 | { | 3417 | { |
| 3418 | int pos; | 3418 | int pos; |
| @@ -3489,7 +3489,7 @@ static int cdrom_sysctl_info(ctl_table *ctl, int write, struct file * filp, | |||
| 3489 | goto done; | 3489 | goto done; |
| 3490 | doit: | 3490 | doit: |
| 3491 | mutex_unlock(&cdrom_mutex); | 3491 | mutex_unlock(&cdrom_mutex); |
| 3492 | return proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 3492 | return proc_dostring(ctl, write, buffer, lenp, ppos); |
| 3493 | done: | 3493 | done: |
| 3494 | printk(KERN_INFO "cdrom: info buffer too small\n"); | 3494 | printk(KERN_INFO "cdrom: info buffer too small\n"); |
| 3495 | goto doit; | 3495 | goto doit; |
| @@ -3525,12 +3525,12 @@ static void cdrom_update_settings(void) | |||
| 3525 | mutex_unlock(&cdrom_mutex); | 3525 | mutex_unlock(&cdrom_mutex); |
| 3526 | } | 3526 | } |
| 3527 | 3527 | ||
| 3528 | static int cdrom_sysctl_handler(ctl_table *ctl, int write, struct file * filp, | 3528 | static int cdrom_sysctl_handler(ctl_table *ctl, int write, |
| 3529 | void __user *buffer, size_t *lenp, loff_t *ppos) | 3529 | void __user *buffer, size_t *lenp, loff_t *ppos) |
| 3530 | { | 3530 | { |
| 3531 | int ret; | 3531 | int ret; |
| 3532 | 3532 | ||
| 3533 | ret = proc_dointvec(ctl, write, filp, buffer, lenp, ppos); | 3533 | ret = proc_dointvec(ctl, write, buffer, lenp, ppos); |
| 3534 | 3534 | ||
| 3535 | if (write) { | 3535 | if (write) { |
| 3536 | 3536 | ||
diff --git a/drivers/char/random.c b/drivers/char/random.c index d8a9255e1a3f..04b505e5a5e2 100644 --- a/drivers/char/random.c +++ b/drivers/char/random.c | |||
| @@ -1231,7 +1231,7 @@ static char sysctl_bootid[16]; | |||
| 1231 | * as an ASCII string in the standard UUID format. If accesses via the | 1231 | * as an ASCII string in the standard UUID format. If accesses via the |
| 1232 | * sysctl system call, it is returned as 16 bytes of binary data. | 1232 | * sysctl system call, it is returned as 16 bytes of binary data. |
| 1233 | */ | 1233 | */ |
| 1234 | static int proc_do_uuid(ctl_table *table, int write, struct file *filp, | 1234 | static int proc_do_uuid(ctl_table *table, int write, |
| 1235 | void __user *buffer, size_t *lenp, loff_t *ppos) | 1235 | void __user *buffer, size_t *lenp, loff_t *ppos) |
| 1236 | { | 1236 | { |
| 1237 | ctl_table fake_table; | 1237 | ctl_table fake_table; |
| @@ -1254,7 +1254,7 @@ static int proc_do_uuid(ctl_table *table, int write, struct file *filp, | |||
| 1254 | fake_table.data = buf; | 1254 | fake_table.data = buf; |
| 1255 | fake_table.maxlen = sizeof(buf); | 1255 | fake_table.maxlen = sizeof(buf); |
| 1256 | 1256 | ||
| 1257 | return proc_dostring(&fake_table, write, filp, buffer, lenp, ppos); | 1257 | return proc_dostring(&fake_table, write, buffer, lenp, ppos); |
| 1258 | } | 1258 | } |
| 1259 | 1259 | ||
| 1260 | static int uuid_strategy(ctl_table *table, | 1260 | static int uuid_strategy(ctl_table *table, |
diff --git a/drivers/net/wireless/arlan-proc.c b/drivers/net/wireless/arlan-proc.c index 2ab1d59870f4..a8b689635a3b 100644 --- a/drivers/net/wireless/arlan-proc.c +++ b/drivers/net/wireless/arlan-proc.c | |||
| @@ -402,7 +402,7 @@ static int arlan_setup_card_by_book(struct net_device *dev) | |||
| 402 | 402 | ||
| 403 | static char arlan_drive_info[ARLAN_STR_SIZE] = "A655\n\0"; | 403 | static char arlan_drive_info[ARLAN_STR_SIZE] = "A655\n\0"; |
| 404 | 404 | ||
| 405 | static int arlan_sysctl_info(ctl_table * ctl, int write, struct file *filp, | 405 | static int arlan_sysctl_info(ctl_table * ctl, int write, |
| 406 | void __user *buffer, size_t * lenp, loff_t *ppos) | 406 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 407 | { | 407 | { |
| 408 | int i; | 408 | int i; |
| @@ -629,7 +629,7 @@ final: | |||
| 629 | *lenp = pos; | 629 | *lenp = pos; |
| 630 | 630 | ||
| 631 | if (!write) | 631 | if (!write) |
| 632 | retv = proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 632 | retv = proc_dostring(ctl, write, buffer, lenp, ppos); |
| 633 | else | 633 | else |
| 634 | { | 634 | { |
| 635 | *lenp = 0; | 635 | *lenp = 0; |
| @@ -639,7 +639,7 @@ final: | |||
| 639 | } | 639 | } |
| 640 | 640 | ||
| 641 | 641 | ||
| 642 | static int arlan_sysctl_info161719(ctl_table * ctl, int write, struct file *filp, | 642 | static int arlan_sysctl_info161719(ctl_table * ctl, int write, |
| 643 | void __user *buffer, size_t * lenp, loff_t *ppos) | 643 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 644 | { | 644 | { |
| 645 | int i; | 645 | int i; |
| @@ -669,11 +669,11 @@ static int arlan_sysctl_info161719(ctl_table * ctl, int write, struct file *filp | |||
| 669 | 669 | ||
| 670 | final: | 670 | final: |
| 671 | *lenp = pos; | 671 | *lenp = pos; |
| 672 | retv = proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 672 | retv = proc_dostring(ctl, write, buffer, lenp, ppos); |
| 673 | return retv; | 673 | return retv; |
| 674 | } | 674 | } |
| 675 | 675 | ||
| 676 | static int arlan_sysctl_infotxRing(ctl_table * ctl, int write, struct file *filp, | 676 | static int arlan_sysctl_infotxRing(ctl_table * ctl, int write, |
| 677 | void __user *buffer, size_t * lenp, loff_t *ppos) | 677 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 678 | { | 678 | { |
| 679 | int i; | 679 | int i; |
| @@ -698,11 +698,11 @@ static int arlan_sysctl_infotxRing(ctl_table * ctl, int write, struct file *filp | |||
| 698 | SARLBNpln(u_char, txBuffer, 0x800); | 698 | SARLBNpln(u_char, txBuffer, 0x800); |
| 699 | final: | 699 | final: |
| 700 | *lenp = pos; | 700 | *lenp = pos; |
| 701 | retv = proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 701 | retv = proc_dostring(ctl, write, buffer, lenp, ppos); |
| 702 | return retv; | 702 | return retv; |
| 703 | } | 703 | } |
| 704 | 704 | ||
| 705 | static int arlan_sysctl_inforxRing(ctl_table * ctl, int write, struct file *filp, | 705 | static int arlan_sysctl_inforxRing(ctl_table * ctl, int write, |
| 706 | void __user *buffer, size_t * lenp, loff_t *ppos) | 706 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 707 | { | 707 | { |
| 708 | int i; | 708 | int i; |
| @@ -726,11 +726,11 @@ static int arlan_sysctl_inforxRing(ctl_table * ctl, int write, struct file *filp | |||
| 726 | SARLBNpln(u_char, rxBuffer, 0x800); | 726 | SARLBNpln(u_char, rxBuffer, 0x800); |
| 727 | final: | 727 | final: |
| 728 | *lenp = pos; | 728 | *lenp = pos; |
| 729 | retv = proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 729 | retv = proc_dostring(ctl, write, buffer, lenp, ppos); |
| 730 | return retv; | 730 | return retv; |
| 731 | } | 731 | } |
| 732 | 732 | ||
| 733 | static int arlan_sysctl_info18(ctl_table * ctl, int write, struct file *filp, | 733 | static int arlan_sysctl_info18(ctl_table * ctl, int write, |
| 734 | void __user *buffer, size_t * lenp, loff_t *ppos) | 734 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 735 | { | 735 | { |
| 736 | int i; | 736 | int i; |
| @@ -756,7 +756,7 @@ static int arlan_sysctl_info18(ctl_table * ctl, int write, struct file *filp, | |||
| 756 | 756 | ||
| 757 | final: | 757 | final: |
| 758 | *lenp = pos; | 758 | *lenp = pos; |
| 759 | retv = proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 759 | retv = proc_dostring(ctl, write, buffer, lenp, ppos); |
| 760 | return retv; | 760 | return retv; |
| 761 | } | 761 | } |
| 762 | 762 | ||
| @@ -766,7 +766,7 @@ final: | |||
| 766 | 766 | ||
| 767 | static char conf_reset_result[200]; | 767 | static char conf_reset_result[200]; |
| 768 | 768 | ||
| 769 | static int arlan_configure(ctl_table * ctl, int write, struct file *filp, | 769 | static int arlan_configure(ctl_table * ctl, int write, |
| 770 | void __user *buffer, size_t * lenp, loff_t *ppos) | 770 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 771 | { | 771 | { |
| 772 | int pos = 0; | 772 | int pos = 0; |
| @@ -788,10 +788,10 @@ static int arlan_configure(ctl_table * ctl, int write, struct file *filp, | |||
| 788 | return -1; | 788 | return -1; |
| 789 | 789 | ||
| 790 | *lenp = pos; | 790 | *lenp = pos; |
| 791 | return proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 791 | return proc_dostring(ctl, write, buffer, lenp, ppos); |
| 792 | } | 792 | } |
| 793 | 793 | ||
| 794 | static int arlan_sysctl_reset(ctl_table * ctl, int write, struct file *filp, | 794 | static int arlan_sysctl_reset(ctl_table * ctl, int write, |
| 795 | void __user *buffer, size_t * lenp, loff_t *ppos) | 795 | void __user *buffer, size_t * lenp, loff_t *ppos) |
| 796 | { | 796 | { |
| 797 | int pos = 0; | 797 | int pos = 0; |
| @@ -811,7 +811,7 @@ static int arlan_sysctl_reset(ctl_table * ctl, int write, struct file *filp, | |||
| 811 | } else | 811 | } else |
| 812 | return -1; | 812 | return -1; |
| 813 | *lenp = pos + 3; | 813 | *lenp = pos + 3; |
| 814 | return proc_dostring(ctl, write, filp, buffer, lenp, ppos); | 814 | return proc_dostring(ctl, write, buffer, lenp, ppos); |
| 815 | } | 815 | } |
| 816 | 816 | ||
| 817 | 817 | ||
diff --git a/drivers/parport/procfs.c b/drivers/parport/procfs.c index 554e11f9e1ce..8eefe56f1cbe 100644 --- a/drivers/parport/procfs.c +++ b/drivers/parport/procfs.c | |||
| @@ -31,7 +31,7 @@ | |||
| 31 | #define PARPORT_MIN_SPINTIME_VALUE 1 | 31 | #define PARPORT_MIN_SPINTIME_VALUE 1 |
| 32 | #define PARPORT_MAX_SPINTIME_VALUE 1000 | 32 | #define PARPORT_MAX_SPINTIME_VALUE 1000 |
| 33 | 33 | ||
| 34 | static int do_active_device(ctl_table *table, int write, struct file *filp, | 34 | static int do_active_device(ctl_table *table, int write, |
| 35 | void __user *result, size_t *lenp, loff_t *ppos) | 35 | void __user *result, size_t *lenp, loff_t *ppos) |
| 36 | { | 36 | { |
| 37 | struct parport *port = (struct parport *)table->extra1; | 37 | struct parport *port = (struct parport *)table->extra1; |
| @@ -68,7 +68,7 @@ static int do_active_device(ctl_table *table, int write, struct file *filp, | |||
| 68 | } | 68 | } |
| 69 | 69 | ||
| 70 | #ifdef CONFIG_PARPORT_1284 | 70 | #ifdef CONFIG_PARPORT_1284 |
| 71 | static int do_autoprobe(ctl_table *table, int write, struct file *filp, | 71 | static int do_autoprobe(ctl_table *table, int write, |
| 72 | void __user *result, size_t *lenp, loff_t *ppos) | 72 | void __user *result, size_t *lenp, loff_t *ppos) |
| 73 | { | 73 | { |
| 74 | struct parport_device_info *info = table->extra2; | 74 | struct parport_device_info *info = table->extra2; |
| @@ -111,7 +111,7 @@ static int do_autoprobe(ctl_table *table, int write, struct file *filp, | |||
| 111 | #endif /* IEEE1284.3 support. */ | 111 | #endif /* IEEE1284.3 support. */ |
| 112 | 112 | ||
| 113 | static int do_hardware_base_addr (ctl_table *table, int write, | 113 | static int do_hardware_base_addr (ctl_table *table, int write, |
| 114 | struct file *filp, void __user *result, | 114 | void __user *result, |
| 115 | size_t *lenp, loff_t *ppos) | 115 | size_t *lenp, loff_t *ppos) |
| 116 | { | 116 | { |
| 117 | struct parport *port = (struct parport *)table->extra1; | 117 | struct parport *port = (struct parport *)table->extra1; |
| @@ -139,7 +139,7 @@ static int do_hardware_base_addr (ctl_table *table, int write, | |||
| 139 | } | 139 | } |
| 140 | 140 | ||
| 141 | static int do_hardware_irq (ctl_table *table, int write, | 141 | static int do_hardware_irq (ctl_table *table, int write, |
| 142 | struct file *filp, void __user *result, | 142 | void __user *result, |
| 143 | size_t *lenp, loff_t *ppos) | 143 | size_t *lenp, loff_t *ppos) |
| 144 | { | 144 | { |
| 145 | struct parport *port = (struct parport *)table->extra1; | 145 | struct parport *port = (struct parport *)table->extra1; |
| @@ -167,7 +167,7 @@ static int do_hardware_irq (ctl_table *table, int write, | |||
| 167 | } | 167 | } |
| 168 | 168 | ||
| 169 | static int do_hardware_dma (ctl_table *table, int write, | 169 | static int do_hardware_dma (ctl_table *table, int write, |
| 170 | struct file *filp, void __user *result, | 170 | void __user *result, |
| 171 | size_t *lenp, loff_t *ppos) | 171 | size_t *lenp, loff_t *ppos) |
| 172 | { | 172 | { |
| 173 | struct parport *port = (struct parport *)table->extra1; | 173 | struct parport *port = (struct parport *)table->extra1; |
| @@ -195,7 +195,7 @@ static int do_hardware_dma (ctl_table *table, int write, | |||
| 195 | } | 195 | } |
| 196 | 196 | ||
| 197 | static int do_hardware_modes (ctl_table *table, int write, | 197 | static int do_hardware_modes (ctl_table *table, int write, |
| 198 | struct file *filp, void __user *result, | 198 | void __user *result, |
| 199 | size_t *lenp, loff_t *ppos) | 199 | size_t *lenp, loff_t *ppos) |
| 200 | { | 200 | { |
| 201 | struct parport *port = (struct parport *)table->extra1; | 201 | struct parport *port = (struct parport *)table->extra1; |
