diff options
author | Arjan van de Ven <arjan@infradead.org> | 2006-03-28 04:56:41 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-28 12:16:05 -0500 |
commit | 99ac48f54a91d02140c497edc31dc57d4bc5c85d (patch) | |
tree | 68719391694a6914191bdf73d2071875f7653f6f /drivers | |
parent | ec1b9466cb4f6ae6d950bd67055d9410d1056d2a (diff) |
[PATCH] mark f_ops const in the inode
Mark the f_ops members of inodes as const, as well as fix the
ripple-through this causes by places that copy this f_ops and then "do
stuff" with it.
Signed-off-by: Arjan van de Ven <arjan@infradead.org>
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/char/drm/drm_fops.c | 2 | ||||
-rw-r--r-- | drivers/char/drm/i810_dma.c | 2 | ||||
-rw-r--r-- | drivers/char/drm/i830_dma.c | 2 | ||||
-rw-r--r-- | drivers/char/mem.c | 2 | ||||
-rw-r--r-- | drivers/char/misc.c | 2 | ||||
-rw-r--r-- | drivers/input/input.c | 2 | ||||
-rw-r--r-- | drivers/isdn/capi/kcapi_proc.c | 2 | ||||
-rw-r--r-- | drivers/media/dvb/dvb-core/dvbdev.c | 2 | ||||
-rw-r--r-- | drivers/media/video/videodev.c | 2 | ||||
-rw-r--r-- | drivers/message/i2o/i2o_proc.c | 2 | ||||
-rw-r--r-- | drivers/oprofile/oprofilefs.c | 6 | ||||
-rw-r--r-- | drivers/telephony/phonedev.c | 2 | ||||
-rw-r--r-- | drivers/usb/core/file.c | 6 | ||||
-rw-r--r-- | drivers/usb/gadget/inode.c | 6 |
14 files changed, 20 insertions, 20 deletions
diff --git a/drivers/char/drm/drm_fops.c b/drivers/char/drm/drm_fops.c index 641f7633878c..b7f7951c4587 100644 --- a/drivers/char/drm/drm_fops.c +++ b/drivers/char/drm/drm_fops.c | |||
@@ -175,7 +175,7 @@ int drm_stub_open(struct inode *inode, struct file *filp) | |||
175 | drm_device_t *dev = NULL; | 175 | drm_device_t *dev = NULL; |
176 | int minor = iminor(inode); | 176 | int minor = iminor(inode); |
177 | int err = -ENODEV; | 177 | int err = -ENODEV; |
178 | struct file_operations *old_fops; | 178 | const struct file_operations *old_fops; |
179 | 179 | ||
180 | DRM_DEBUG("\n"); | 180 | DRM_DEBUG("\n"); |
181 | 181 | ||
diff --git a/drivers/char/drm/i810_dma.c b/drivers/char/drm/i810_dma.c index ae0aa6d7e0bb..c658dde3633b 100644 --- a/drivers/char/drm/i810_dma.c +++ b/drivers/char/drm/i810_dma.c | |||
@@ -126,7 +126,7 @@ static int i810_map_buffer(drm_buf_t * buf, struct file *filp) | |||
126 | drm_device_t *dev = priv->head->dev; | 126 | drm_device_t *dev = priv->head->dev; |
127 | drm_i810_buf_priv_t *buf_priv = buf->dev_private; | 127 | drm_i810_buf_priv_t *buf_priv = buf->dev_private; |
128 | drm_i810_private_t *dev_priv = dev->dev_private; | 128 | drm_i810_private_t *dev_priv = dev->dev_private; |
129 | struct file_operations *old_fops; | 129 | const struct file_operations *old_fops; |
130 | int retcode = 0; | 130 | int retcode = 0; |
131 | 131 | ||
132 | if (buf_priv->currently_mapped == I810_BUF_MAPPED) | 132 | if (buf_priv->currently_mapped == I810_BUF_MAPPED) |
diff --git a/drivers/char/drm/i830_dma.c b/drivers/char/drm/i830_dma.c index 163f2cbfe60d..b0f815d8cea8 100644 --- a/drivers/char/drm/i830_dma.c +++ b/drivers/char/drm/i830_dma.c | |||
@@ -128,7 +128,7 @@ static int i830_map_buffer(drm_buf_t * buf, struct file *filp) | |||
128 | drm_device_t *dev = priv->head->dev; | 128 | drm_device_t *dev = priv->head->dev; |
129 | drm_i830_buf_priv_t *buf_priv = buf->dev_private; | 129 | drm_i830_buf_priv_t *buf_priv = buf->dev_private; |
130 | drm_i830_private_t *dev_priv = dev->dev_private; | 130 | drm_i830_private_t *dev_priv = dev->dev_private; |
131 | struct file_operations *old_fops; | 131 | const struct file_operations *old_fops; |
132 | unsigned long virtual; | 132 | unsigned long virtual; |
133 | int retcode = 0; | 133 | int retcode = 0; |
134 | 134 | ||
diff --git a/drivers/char/mem.c b/drivers/char/mem.c index 5245ba1649ed..66719f9d294c 100644 --- a/drivers/char/mem.c +++ b/drivers/char/mem.c | |||
@@ -899,7 +899,7 @@ static const struct { | |||
899 | unsigned int minor; | 899 | unsigned int minor; |
900 | char *name; | 900 | char *name; |
901 | umode_t mode; | 901 | umode_t mode; |
902 | struct file_operations *fops; | 902 | const struct file_operations *fops; |
903 | } devlist[] = { /* list of minor devices */ | 903 | } devlist[] = { /* list of minor devices */ |
904 | {1, "mem", S_IRUSR | S_IWUSR | S_IRGRP, &mem_fops}, | 904 | {1, "mem", S_IRUSR | S_IWUSR | S_IRGRP, &mem_fops}, |
905 | {2, "kmem", S_IRUSR | S_IWUSR | S_IRGRP, &kmem_fops}, | 905 | {2, "kmem", S_IRUSR | S_IWUSR | S_IRGRP, &kmem_fops}, |
diff --git a/drivers/char/misc.c b/drivers/char/misc.c index 3e4c0414a01a..96eb2a709e21 100644 --- a/drivers/char/misc.c +++ b/drivers/char/misc.c | |||
@@ -129,7 +129,7 @@ static int misc_open(struct inode * inode, struct file * file) | |||
129 | int minor = iminor(inode); | 129 | int minor = iminor(inode); |
130 | struct miscdevice *c; | 130 | struct miscdevice *c; |
131 | int err = -ENODEV; | 131 | int err = -ENODEV; |
132 | struct file_operations *old_fops, *new_fops = NULL; | 132 | const struct file_operations *old_fops, *new_fops = NULL; |
133 | 133 | ||
134 | down(&misc_sem); | 134 | down(&misc_sem); |
135 | 135 | ||
diff --git a/drivers/input/input.c b/drivers/input/input.c index 4fe3da3c667a..f8af0945964e 100644 --- a/drivers/input/input.c +++ b/drivers/input/input.c | |||
@@ -923,7 +923,7 @@ void input_unregister_handler(struct input_handler *handler) | |||
923 | static int input_open_file(struct inode *inode, struct file *file) | 923 | static int input_open_file(struct inode *inode, struct file *file) |
924 | { | 924 | { |
925 | struct input_handler *handler = input_table[iminor(inode) >> 5]; | 925 | struct input_handler *handler = input_table[iminor(inode) >> 5]; |
926 | struct file_operations *old_fops, *new_fops = NULL; | 926 | const struct file_operations *old_fops, *new_fops = NULL; |
927 | int err; | 927 | int err; |
928 | 928 | ||
929 | /* No load-on-demand here? */ | 929 | /* No load-on-demand here? */ |
diff --git a/drivers/isdn/capi/kcapi_proc.c b/drivers/isdn/capi/kcapi_proc.c index 2cc8b27e4c3b..ca9dc00a45c4 100644 --- a/drivers/isdn/capi/kcapi_proc.c +++ b/drivers/isdn/capi/kcapi_proc.c | |||
@@ -233,7 +233,7 @@ static struct file_operations proc_applstats_ops = { | |||
233 | }; | 233 | }; |
234 | 234 | ||
235 | static void | 235 | static void |
236 | create_seq_entry(char *name, mode_t mode, struct file_operations *f) | 236 | create_seq_entry(char *name, mode_t mode, const struct file_operations *f) |
237 | { | 237 | { |
238 | struct proc_dir_entry *entry; | 238 | struct proc_dir_entry *entry; |
239 | entry = create_proc_entry(name, mode, NULL); | 239 | entry = create_proc_entry(name, mode, NULL); |
diff --git a/drivers/media/dvb/dvb-core/dvbdev.c b/drivers/media/dvb/dvb-core/dvbdev.c index 54f8b95717b0..96fe0ecae250 100644 --- a/drivers/media/dvb/dvb-core/dvbdev.c +++ b/drivers/media/dvb/dvb-core/dvbdev.c | |||
@@ -86,7 +86,7 @@ static int dvb_device_open(struct inode *inode, struct file *file) | |||
86 | 86 | ||
87 | if (dvbdev && dvbdev->fops) { | 87 | if (dvbdev && dvbdev->fops) { |
88 | int err = 0; | 88 | int err = 0; |
89 | struct file_operations *old_fops; | 89 | const struct file_operations *old_fops; |
90 | 90 | ||
91 | file->private_data = dvbdev; | 91 | file->private_data = dvbdev; |
92 | old_fops = file->f_op; | 92 | old_fops = file->f_op; |
diff --git a/drivers/media/video/videodev.c b/drivers/media/video/videodev.c index 75e3d41382f2..5f87dd5f1d0b 100644 --- a/drivers/media/video/videodev.c +++ b/drivers/media/video/videodev.c | |||
@@ -97,7 +97,7 @@ static int video_open(struct inode *inode, struct file *file) | |||
97 | unsigned int minor = iminor(inode); | 97 | unsigned int minor = iminor(inode); |
98 | int err = 0; | 98 | int err = 0; |
99 | struct video_device *vfl; | 99 | struct video_device *vfl; |
100 | struct file_operations *old_fops; | 100 | const struct file_operations *old_fops; |
101 | 101 | ||
102 | if(minor>=VIDEO_NUM_DEVICES) | 102 | if(minor>=VIDEO_NUM_DEVICES) |
103 | return -ENODEV; | 103 | return -ENODEV; |
diff --git a/drivers/message/i2o/i2o_proc.c b/drivers/message/i2o/i2o_proc.c index 2a0c42b8cda5..3d2e76eea93e 100644 --- a/drivers/message/i2o/i2o_proc.c +++ b/drivers/message/i2o/i2o_proc.c | |||
@@ -56,7 +56,7 @@ | |||
56 | typedef struct _i2o_proc_entry_t { | 56 | typedef struct _i2o_proc_entry_t { |
57 | char *name; /* entry name */ | 57 | char *name; /* entry name */ |
58 | mode_t mode; /* mode */ | 58 | mode_t mode; /* mode */ |
59 | struct file_operations *fops; /* open function */ | 59 | const struct file_operations *fops; /* open function */ |
60 | } i2o_proc_entry; | 60 | } i2o_proc_entry; |
61 | 61 | ||
62 | /* global I2O /proc/i2o entry */ | 62 | /* global I2O /proc/i2o entry */ |
diff --git a/drivers/oprofile/oprofilefs.c b/drivers/oprofile/oprofilefs.c index d6bae699749a..b62da9b0cbf0 100644 --- a/drivers/oprofile/oprofilefs.c +++ b/drivers/oprofile/oprofilefs.c | |||
@@ -130,7 +130,7 @@ static struct file_operations ulong_ro_fops = { | |||
130 | 130 | ||
131 | 131 | ||
132 | static struct dentry * __oprofilefs_create_file(struct super_block * sb, | 132 | static struct dentry * __oprofilefs_create_file(struct super_block * sb, |
133 | struct dentry * root, char const * name, struct file_operations * fops, | 133 | struct dentry * root, char const * name, const struct file_operations * fops, |
134 | int perm) | 134 | int perm) |
135 | { | 135 | { |
136 | struct dentry * dentry; | 136 | struct dentry * dentry; |
@@ -203,7 +203,7 @@ int oprofilefs_create_ro_atomic(struct super_block * sb, struct dentry * root, | |||
203 | 203 | ||
204 | 204 | ||
205 | int oprofilefs_create_file(struct super_block * sb, struct dentry * root, | 205 | int oprofilefs_create_file(struct super_block * sb, struct dentry * root, |
206 | char const * name, struct file_operations * fops) | 206 | char const * name, const struct file_operations * fops) |
207 | { | 207 | { |
208 | if (!__oprofilefs_create_file(sb, root, name, fops, 0644)) | 208 | if (!__oprofilefs_create_file(sb, root, name, fops, 0644)) |
209 | return -EFAULT; | 209 | return -EFAULT; |
@@ -212,7 +212,7 @@ int oprofilefs_create_file(struct super_block * sb, struct dentry * root, | |||
212 | 212 | ||
213 | 213 | ||
214 | int oprofilefs_create_file_perm(struct super_block * sb, struct dentry * root, | 214 | int oprofilefs_create_file_perm(struct super_block * sb, struct dentry * root, |
215 | char const * name, struct file_operations * fops, int perm) | 215 | char const * name, const struct file_operations * fops, int perm) |
216 | { | 216 | { |
217 | if (!__oprofilefs_create_file(sb, root, name, fops, perm)) | 217 | if (!__oprofilefs_create_file(sb, root, name, fops, perm)) |
218 | return -EFAULT; | 218 | return -EFAULT; |
diff --git a/drivers/telephony/phonedev.c b/drivers/telephony/phonedev.c index 7a6db1c5c8c5..e166fffea86b 100644 --- a/drivers/telephony/phonedev.c +++ b/drivers/telephony/phonedev.c | |||
@@ -49,7 +49,7 @@ static int phone_open(struct inode *inode, struct file *file) | |||
49 | unsigned int minor = iminor(inode); | 49 | unsigned int minor = iminor(inode); |
50 | int err = 0; | 50 | int err = 0; |
51 | struct phone_device *p; | 51 | struct phone_device *p; |
52 | struct file_operations *old_fops, *new_fops = NULL; | 52 | const struct file_operations *old_fops, *new_fops = NULL; |
53 | 53 | ||
54 | if (minor >= PHONE_NUM_DEVICES) | 54 | if (minor >= PHONE_NUM_DEVICES) |
55 | return -ENODEV; | 55 | return -ENODEV; |
diff --git a/drivers/usb/core/file.c b/drivers/usb/core/file.c index 37b13368c814..b263a54a13c0 100644 --- a/drivers/usb/core/file.c +++ b/drivers/usb/core/file.c | |||
@@ -24,15 +24,15 @@ | |||
24 | #include "usb.h" | 24 | #include "usb.h" |
25 | 25 | ||
26 | #define MAX_USB_MINORS 256 | 26 | #define MAX_USB_MINORS 256 |
27 | static struct file_operations *usb_minors[MAX_USB_MINORS]; | 27 | static const struct file_operations *usb_minors[MAX_USB_MINORS]; |
28 | static DEFINE_SPINLOCK(minor_lock); | 28 | static DEFINE_SPINLOCK(minor_lock); |
29 | 29 | ||
30 | static int usb_open(struct inode * inode, struct file * file) | 30 | static int usb_open(struct inode * inode, struct file * file) |
31 | { | 31 | { |
32 | int minor = iminor(inode); | 32 | int minor = iminor(inode); |
33 | struct file_operations *c; | 33 | const struct file_operations *c; |
34 | int err = -ENODEV; | 34 | int err = -ENODEV; |
35 | struct file_operations *old_fops, *new_fops = NULL; | 35 | const struct file_operations *old_fops, *new_fops = NULL; |
36 | 36 | ||
37 | spin_lock (&minor_lock); | 37 | spin_lock (&minor_lock); |
38 | c = usb_minors[minor]; | 38 | c = usb_minors[minor]; |
diff --git a/drivers/usb/gadget/inode.c b/drivers/usb/gadget/inode.c index b44cfda76b61..3f618ce6998d 100644 --- a/drivers/usb/gadget/inode.c +++ b/drivers/usb/gadget/inode.c | |||
@@ -1581,7 +1581,7 @@ restart: | |||
1581 | 1581 | ||
1582 | static struct inode * | 1582 | static struct inode * |
1583 | gadgetfs_create_file (struct super_block *sb, char const *name, | 1583 | gadgetfs_create_file (struct super_block *sb, char const *name, |
1584 | void *data, struct file_operations *fops, | 1584 | void *data, const struct file_operations *fops, |
1585 | struct dentry **dentry_p); | 1585 | struct dentry **dentry_p); |
1586 | 1586 | ||
1587 | static int activate_ep_files (struct dev_data *dev) | 1587 | static int activate_ep_files (struct dev_data *dev) |
@@ -1955,7 +1955,7 @@ module_param (default_perm, uint, 0644); | |||
1955 | 1955 | ||
1956 | static struct inode * | 1956 | static struct inode * |
1957 | gadgetfs_make_inode (struct super_block *sb, | 1957 | gadgetfs_make_inode (struct super_block *sb, |
1958 | void *data, struct file_operations *fops, | 1958 | void *data, const struct file_operations *fops, |
1959 | int mode) | 1959 | int mode) |
1960 | { | 1960 | { |
1961 | struct inode *inode = new_inode (sb); | 1961 | struct inode *inode = new_inode (sb); |
@@ -1979,7 +1979,7 @@ gadgetfs_make_inode (struct super_block *sb, | |||
1979 | */ | 1979 | */ |
1980 | static struct inode * | 1980 | static struct inode * |
1981 | gadgetfs_create_file (struct super_block *sb, char const *name, | 1981 | gadgetfs_create_file (struct super_block *sb, char const *name, |
1982 | void *data, struct file_operations *fops, | 1982 | void *data, const struct file_operations *fops, |
1983 | struct dentry **dentry_p) | 1983 | struct dentry **dentry_p) |
1984 | { | 1984 | { |
1985 | struct dentry *dentry; | 1985 | struct dentry *dentry; |