tree 0502467742659777a64b04385cb6bc54e32a2e6d
parent cb8bd2ff1528a3f0234ec04b667bf91a3be7ac30
author Viresh Kumar <viresh.kumar@linaro.org> 1519279183 +0530
committer Rafael J. Wysocki <rafael.j.wysocki@intel.com> 1519752132 +0100

cpufreq: Reorder cpufreq_online() error code path

Ideally the de-allocation of resources should happen in the exact
opposite order in which they were allocated. It helps maintain the code
in long term, even if nothing really breaks with incorrect ordering.

That wasn't followed in cpufreq_online() and it has some
inconsistencies.  For example, the symlinks were created from within
the locked region while they are removed only after putting the locks.
Also ->exit() should have been called only after the symlinks are
removed and the lock is dropped, as that was the case when ->init()
was first called.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
[ rjw: Subject ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>