power: supply: ab8500: cleanup with list_first_entry_or_null()
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Mon, 12 Sep 2016 18:23:21 +0000 (03:23 +0900)
committerSebastian Reichel <sre@kernel.org>
Mon, 19 Sep 2016 19:22:10 +0000 (21:22 +0200)
The combo of list_empty() check and return list_first_entry()
can be replaced with list_first_entry_or_null().

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
drivers/power/supply/ab8500_fg.c

index 199f2db..2199f67 100644 (file)
@@ -245,13 +245,8 @@ static LIST_HEAD(ab8500_fg_list);
  */
 struct ab8500_fg *ab8500_fg_get(void)
 {
-       struct ab8500_fg *fg;
-
-       if (list_empty(&ab8500_fg_list))
-               return NULL;
-
-       fg = list_first_entry(&ab8500_fg_list, struct ab8500_fg, node);
-       return fg;
+       return list_first_entry_or_null(&ab8500_fg_list, struct ab8500_fg,
+                                       node);
 }
 
 /* Main battery properties */