summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorMasahiro Yamada <yamada.masahiro@socionext.com>2016-09-12 21:23:21 +0300
committerSebastian Reichel <sre@kernel.org>2016-09-19 22:22:10 +0300
commitf04f7aef7f6aafdc0ba54c6b9670cd11f9d9200a (patch)
treee71ecade622eee6f384f5e52ce15a358e86bee01 /drivers
parent1dff6ce0262bc42beb801527bd4725c6b8af0683 (diff)
downloadlinux-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.c9
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 */