Re: [PATCH v2 2/2] ARM: cpuidle: Refactor rollback operations if init fails

From: Daniel Lezcano
Date: Wed Oct 11 2017 - 05:49:24 EST


On 10/10/2017 07:47, Leo Yan wrote:
> If init fails, we need execute two levels rollback operations: the first
> level is for the failed CPU rollback operations, the second level is to
> iterate all succeeded CPUs to cancel their registration; currently the
> code uses one function to finish these two levels rollback operations.
>
> This commit is to refactor rollback operations, so it adds a new
> function arm_idle_init_cpu() to encapsulate one specified CPU driver
> registration and rollback the first level operations; and use function
> arm_idle_init() to iterate all CPUs and finish the second level's
> rollback operations.
>
> Suggested-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> Signed-off-by: Leo Yan <leo.yan@xxxxxxxxxx>
> ---

Acked-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>


--
<http://www.linaro.org/> Linaro.org â Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog