Re: [PATCH 1/5] powernv:idle: Move device-tree parsing to one place.

2017-07-08 Thread Nicholas Piggin
On Fri, 7 Jul 2017 16:55:39 +0530 Gautham R Shenoy wrote: > Hello Nicholas, > > On Fri, Jul 07, 2017 at 12:53:40AM +1000, Nicholas Piggin wrote: > > I wouldn't have the wrapper function... but it's your code so it's > > up to you. One thing though is that this function you have called get_ > >

Re: [PATCH 1/5] powernv:idle: Move device-tree parsing to one place.

2017-07-07 Thread Gautham R Shenoy
Hello Nicholas, On Fri, Jul 07, 2017 at 12:53:40AM +1000, Nicholas Piggin wrote: > On Wed, 5 Jul 2017 22:08:12 +0530 > "Gautham R. Shenoy" wrote: > > > From: "Gautham R. Shenoy" > > > > The details of the platform idle state are exposed by the firmware to > > the kernel via device tree. > >

Re: [PATCH 1/5] powernv:idle: Move device-tree parsing to one place.

2017-07-06 Thread Nicholas Piggin
On Wed, 5 Jul 2017 22:08:12 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > The details of the platform idle state are exposed by the firmware to > the kernel via device tree. > > In the current code, we parse the device tree twice : > > 1) During the boot up in arch/powerpc