diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2016-09-12 14:23:21 -0400 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2016-09-19 15:22:10 -0400 |
commit | f04f7aef7f6aafdc0ba54c6b9670cd11f9d9200a (patch) | |
tree | e71ecade622eee6f384f5e52ce15a358e86bee01 /drivers/power | |
parent | 1dff6ce0262bc42beb801527bd4725c6b8af0683 (diff) |
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/power')
-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); | |||
245 | */ | 245 | */ |
246 | struct ab8500_fg *ab8500_fg_get(void) | 246 | struct ab8500_fg *ab8500_fg_get(void) |
247 | { | 247 | { |
248 | struct ab8500_fg *fg; | 248 | return list_first_entry_or_null(&ab8500_fg_list, struct ab8500_fg, |
249 | 249 | node); | |
250 | if (list_empty(&ab8500_fg_list)) | ||
251 | return NULL; | ||
252 | |||
253 | fg = list_first_entry(&ab8500_fg_list, struct ab8500_fg, node); | ||
254 | return fg; | ||
255 | } | 250 | } |
256 | 251 | ||
257 | /* Main battery properties */ | 252 | /* Main battery properties */ |