Commit e53252d97e670a38b1d2e9723b48077bba11ddda
Committed by
Linus Torvalds
1 parent
68fc4fabca
Exists in
master
and in
7 other branches
unregister_chrdev() return void
unregister_chrdev() does not return meaningful value. This patch makes it return void like most unregister_* functions. Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Showing 2 changed files with 2 additions and 3 deletions Side-by-side Diff
fs/char_dev.c
... | ... | @@ -321,14 +321,13 @@ |
321 | 321 | } |
322 | 322 | } |
323 | 323 | |
324 | -int unregister_chrdev(unsigned int major, const char *name) | |
324 | +void unregister_chrdev(unsigned int major, const char *name) | |
325 | 325 | { |
326 | 326 | struct char_device_struct *cd; |
327 | 327 | cd = __unregister_chrdev_region(major, 0, 256); |
328 | 328 | if (cd && cd->cdev) |
329 | 329 | cdev_del(cd->cdev); |
330 | 330 | kfree(cd); |
331 | - return 0; | |
332 | 331 | } |
333 | 332 | |
334 | 333 | static DEFINE_SPINLOCK(cdev_lock); |
include/linux/fs.h
... | ... | @@ -1463,7 +1463,7 @@ |
1463 | 1463 | extern int register_chrdev_region(dev_t, unsigned, const char *); |
1464 | 1464 | extern int register_chrdev(unsigned int, const char *, |
1465 | 1465 | const struct file_operations *); |
1466 | -extern int unregister_chrdev(unsigned int, const char *); | |
1466 | +extern void unregister_chrdev(unsigned int, const char *); | |
1467 | 1467 | extern void unregister_chrdev_region(dev_t, unsigned); |
1468 | 1468 | extern int chrdev_open(struct inode *, struct file *); |
1469 | 1469 | extern void chrdev_show(struct seq_file *,off_t); |