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

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

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

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

2017-07-05 Thread Gautham R. Shenoy
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/platforms/powernv/idle.c Here, the device tree is