The assignments and return checks on ret are redundant. Clean up the code by just returning the return value from the call to v9fs_init_inode_cache. Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> --- fs/9p/v9fs.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c index 281a1ed03a04..ee0a374e0d9d 100644 --- a/fs/9p/v9fs.c +++ b/fs/9p/v9fs.c @@ -661,12 +661,7 @@ static void v9fs_destroy_inode_cache(void) static int v9fs_cache_register(void) { - int ret; - - ret = v9fs_init_inode_cache(); - if (ret < 0) - return ret; - return ret; + return v9fs_init_inode_cache(); } static void v9fs_cache_unregister(void) -- 2.39.5