powercap/dtpm: Move the 'root' reset place
authorDaniel Lezcano <daniel.lezcano@linaro.org>
Sun, 30 Jan 2022 21:02:07 +0000 (22:02 +0100)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 23 Feb 2022 18:45:45 +0000 (19:45 +0100)
The 'root' node is checked everytime a dtpm node is destroyed.

When we reach the end of the hierarchy destruction function, we can
unconditionnaly set the 'root' node to NULL again.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Link: https://lore.kernel.org/r/20220130210210.549877-5-daniel.lezcano@linaro.org
drivers/powercap/dtpm.c

index d9d74f9..ec931a0 100644 (file)
@@ -184,9 +184,6 @@ int dtpm_release_zone(struct powercap_zone *pcz)
        else
                kfree(dtpm);
 
-       if (root == dtpm)
-               root = NULL;
-
        return 0;
 }
 
@@ -656,6 +653,8 @@ void dtpm_destroy_hierarchy(void)
 
        pct = NULL;
 
+       root = NULL;
+
 out_unlock:
        mutex_unlock(&dtpm_lock);
 }