On 06/15/2016 11:11 AM, Michael Ellerman wrote:

On Wed, 2016-08-06 at 16:54:27 UTC, "Shreyas B. Prabhu" wrote:

quoted

pnv_init_idle_states discovers supported idle states from the
device tree and does the required initialization. Set power_save
function pointer only after this initialization is done

This looks like a bug fix? Or is this not a concern in practice for some reason
(and if so what is that reason)?

This isn't a concern currently because, all powernv machines so far
supported nap and nap does not need any initialization from kernel side.

Thanks,
Shreyas