diff options
| author | Daniel Lezcano <daniel.lezcano@linaro.org> | 2022-01-30 22:02:07 +0100 |
|---|---|---|
| committer | Daniel Lezcano <daniel.lezcano@linaro.org> | 2022-02-23 19:45:45 +0100 |
| commit | 4712a236db409d5ee5dccb8c7e57fe54d7d3ec66 (patch) | |
| tree | ceaed9850937f48a2aaeefcdf68945c9f274e478 /drivers/powercap | |
| parent | powercap/dtpm: Destroy hierarchy function (diff) | |
| download | linux-4712a236db409d5ee5dccb8c7e57fe54d7d3ec66.tar.xz linux-4712a236db409d5ee5dccb8c7e57fe54d7d3ec66.zip | |
powercap/dtpm: Move the 'root' reset place
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
Diffstat (limited to 'drivers/powercap')
| -rw-r--r-- | drivers/powercap/dtpm.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/powercap/dtpm.c b/drivers/powercap/dtpm.c index d9d74f981118..ec931a06d90a 100644 --- a/drivers/powercap/dtpm.c +++ b/drivers/powercap/dtpm.c @@ -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); } |