proftool: Remove unused variables in make_flame_tree
authorTom Rini <trini@konsulko.com>
Mon, 27 Feb 2023 22:08:38 +0000 (17:08 -0500)
committerTom Rini <trini@konsulko.com>
Wed, 22 Mar 2023 19:22:48 +0000 (15:22 -0400)
With clang-15 we now get reported that in the make_flame_tree function,
neither the missing_count nor depth variables are used, only
incremenete/decremented. Remove these.

Signed-off-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
tools/proftool.c

index 0893604..101bcb6 100644 (file)
@@ -1713,18 +1713,11 @@ static int make_flame_tree(enum out_format_t out_format,
        struct flame_state state;
        struct flame_node *tree;
        struct trace_call *call;
-       int missing_count = 0;
-       int i, depth;
+       int i;
 
        /* maintain a stack of start times, etc. for 'calling' functions */
        state.stack_ptr = 0;
 
-       /*
-        * The first thing in the trace may not be the top-level function, so
-        * set the initial depth so that no function goes below depth 0
-        */
-       depth = -calc_min_depth();
-
        tree = create_node("tree");
        if (!tree)
                return -1;
@@ -1736,16 +1729,10 @@ static int make_flame_tree(enum out_format_t out_format,
                ulong timestamp = call->flags & FUNCF_TIMESTAMP_MASK;
                struct func_info *func;
 
-               if (entry)
-                       depth++;
-               else
-                       depth--;
-
                func = find_func_by_offset(call->func);
                if (!func) {
                        warn("Cannot find function at %lx\n",
                             text_offset + call->func);
-                       missing_count++;
                        continue;
                }