diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2013-05-05 21:52:57 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2013-05-07 02:16:21 -0400 |
| commit | db2a144bedd58b3dcf19950c2f476c58c9f39d18 (patch) | |
| tree | 931f1b5a8e6bafe388b317bce02a9fd9af309d38 /include/linux | |
| parent | a8ca889ed9585894d53fd8919d80cbe8baff09e7 (diff) | |
block_device_operations->release() should return void
The value passed is 0 in all but "it can never happen" cases (and those
only in a couple of drivers) *and* it would've been lost on the way
out anyway, even if something tried to pass something meaningful.
Just don't bother.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/blkdev.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h index 78feda9bbae2..e38cfe77f7f0 100644 --- a/include/linux/blkdev.h +++ b/include/linux/blkdev.h | |||
| @@ -1484,7 +1484,7 @@ static inline bool blk_integrity_is_initialized(struct gendisk *g) | |||
| 1484 | 1484 | ||
| 1485 | struct block_device_operations { | 1485 | struct block_device_operations { |
| 1486 | int (*open) (struct block_device *, fmode_t); | 1486 | int (*open) (struct block_device *, fmode_t); |
| 1487 | int (*release) (struct gendisk *, fmode_t); | 1487 | void (*release) (struct gendisk *, fmode_t); |
| 1488 | int (*ioctl) (struct block_device *, fmode_t, unsigned, unsigned long); | 1488 | int (*ioctl) (struct block_device *, fmode_t, unsigned, unsigned long); |
| 1489 | int (*compat_ioctl) (struct block_device *, fmode_t, unsigned, unsigned long); | 1489 | int (*compat_ioctl) (struct block_device *, fmode_t, unsigned, unsigned long); |
| 1490 | int (*direct_access) (struct block_device *, sector_t, | 1490 | int (*direct_access) (struct block_device *, sector_t, |