scripts/kallsyms: make check_symbol_range() void function
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Sat, 23 Nov 2019 16:04:42 +0000 (01:04 +0900)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Mon, 25 Nov 2019 12:06:37 +0000 (21:06 +0900)
There is no more reason to check the return value of
check_symbol_range().

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
scripts/kallsyms.c

index d90a613..f4d5f13 100644 (file)
@@ -155,8 +155,8 @@ static bool is_ignored_symbol(const char *name, char type)
        return false;
 }
 
-static int check_symbol_range(const char *sym, unsigned long long addr,
-                             struct addr_range *ranges, int entries)
+static void check_symbol_range(const char *sym, unsigned long long addr,
+                              struct addr_range *ranges, int entries)
 {
        size_t i;
        struct addr_range *ar;
@@ -166,14 +166,12 @@ static int check_symbol_range(const char *sym, unsigned long long addr,
 
                if (strcmp(sym, ar->start_sym) == 0) {
                        ar->start = addr;
-                       return 0;
+                       return;
                } else if (strcmp(sym, ar->end_sym) == 0) {
                        ar->end = addr;
-                       return 0;
+                       return;
                }
        }
-
-       return 1;
 }
 
 static int read_symbol(FILE *in, struct sym_entry *s)
@@ -200,9 +198,8 @@ static int read_symbol(FILE *in, struct sym_entry *s)
        /* Ignore most absolute/undefined (?) symbols. */
        if (strcmp(sym, "_text") == 0)
                _text = s->addr;
-       else if (check_symbol_range(sym, s->addr, text_ranges,
-                                   ARRAY_SIZE(text_ranges)) == 0)
-               /* nothing to do */;
+
+       check_symbol_range(sym, s->addr, text_ranges, ARRAY_SIZE(text_ranges));
 
        /* include the type field in the symbol name, so that it gets
         * compressed together */