From: Alexey Dobriyan It always returns 0, so relying on it is useless. The only caller isn't checking return value. In general, un-, de-, -free functions should return void. Signed-off-by: Alexey Dobriyan Signed-off-by: Andrew Morton --- fs/cramfs/uncompress.c | 3 +-- include/linux/cramfs_fs.h | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff -puN fs/cramfs/uncompress.c~cramfs-make-cramfs_uncompress_exit-return-void fs/cramfs/uncompress.c --- a/fs/cramfs/uncompress.c~cramfs-make-cramfs_uncompress_exit-return-void +++ a/fs/cramfs/uncompress.c @@ -68,11 +68,10 @@ int cramfs_uncompress_init(void) return 0; } -int cramfs_uncompress_exit(void) +void cramfs_uncompress_exit(void) { if (!--initialized) { zlib_inflateEnd(&stream); vfree(stream.workspace); } - return 0; } diff -puN include/linux/cramfs_fs.h~cramfs-make-cramfs_uncompress_exit-return-void include/linux/cramfs_fs.h --- a/include/linux/cramfs_fs.h~cramfs-make-cramfs_uncompress_exit-return-void +++ a/include/linux/cramfs_fs.h @@ -87,6 +87,6 @@ struct cramfs_super { /* Uncompression interfaces to the underlying zlib */ int cramfs_uncompress_block(void *dst, int dstlen, void *src, int srclen); int cramfs_uncompress_init(void); -int cramfs_uncompress_exit(void); +void cramfs_uncompress_exit(void); #endif _