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

From: Gautham R Shenoy <[email-masked]>
Date: Tue, 3 Jul 2018 15:16:36 +0530

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 <[email-masked]>
> Reviewed-by: Nicholas Piggin <[email-masked]>

Looks good.
Reviewed-by: Gautham R. Shenoy <[email-masked]>