drivers/android: remove redundant ret variable
authorMinghao Chi <chi.minghao@zte.com.cn>
Tue, 4 Jan 2022 11:35:00 +0000 (11:35 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 4 Feb 2022 14:40:44 +0000 (15:40 +0100)
Return value from list_lru_count() directly instead
of taking this in another redundant variable.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Signed-off-by: CGEL ZTE <cgel.zte@gmail.com>
Link: https://lore.kernel.org/r/20220104113500.602158-1-chi.minghao@zte.com.cn
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/android/binder_alloc.c

index 47bc74a..2ac1008 100644 (file)
@@ -1049,18 +1049,14 @@ err_get_alloc_mutex_failed:
 static unsigned long
 binder_shrink_count(struct shrinker *shrink, struct shrink_control *sc)
 {
-       unsigned long ret = list_lru_count(&binder_alloc_lru);
-       return ret;
+       return list_lru_count(&binder_alloc_lru);
 }
 
 static unsigned long
 binder_shrink_scan(struct shrinker *shrink, struct shrink_control *sc)
 {
-       unsigned long ret;
-
-       ret = list_lru_walk(&binder_alloc_lru, binder_alloc_free_page,
+       return list_lru_walk(&binder_alloc_lru, binder_alloc_free_page,
                            NULL, sc->nr_to_scan);
-       return ret;
 }
 
 static struct shrinker binder_shrinker = {