diff options
| author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2016-09-12 21:23:21 +0300 |
|---|---|---|
| committer | Sebastian Reichel <sre@kernel.org> | 2016-09-19 22:22:10 +0300 |
| commit | f04f7aef7f6aafdc0ba54c6b9670cd11f9d9200a (patch) | |
| tree | e71ecade622eee6f384f5e52ce15a358e86bee01 /drivers | |
| parent | 1dff6ce0262bc42beb801527bd4725c6b8af0683 (diff) | |
| download | linux-f04f7aef7f6aafdc0ba54c6b9670cd11f9d9200a.tar.xz | |
power: supply: ab8500: cleanup with list_first_entry_or_null()
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>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/power/supply/ab8500_fg.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/drivers/power/supply/ab8500_fg.c b/drivers/power/supply/ab8500_fg.c index 199f2dbb0044..2199f673118c 100644 --- a/drivers/power/supply/ab8500_fg.c +++ b/drivers/power/supply/ab8500_fg.c @@ -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 */ |