Re: [PATCH v3 1/2] powernv/cpuidle: Parse dt idle properties into global structure

From: Gautham R Shenoy
Date: Tue Jul 03 2018 - 05:46:49 EST


Hi Akshay,

On Tue, Jul 03, 2018 at 02:50:55PM +0530, Akshay Adiga wrote:
> Device-tree parsing happens twice, once while deciding idle state to be
> used for hotplug and once during cpuidle init. Hence, parsing the device
> tree and caching it will reduce code duplication. Parsing code has been
> moved to pnv_parse_cpuidle_dt() from pnv_probe_idle_states(). In addition
> to the properties in the device tree the number of available states is
> also required.
>
> Signed-off-by: Akshay Adiga <akshay.adiga@xxxxxxxxxxxxxxxxxx>
> Reviewed-by: Nicholas Piggin <npiggin@xxxxxxxxx>

Looks good.
Reviewed-by: Gautham R. Shenoy <ego@xxxxxxxxxxxxxxxxxx>