Re: [PATCH v6 3/5] cpuidle:powernv: Add helper function to populate powernv idle states.

2017-01-30 Thread Gautham R Shenoy
On Mon, Jan 30, 2017 at 10:17:50PM +1100, Michael Ellerman wrote: > "Rafael J. Wysocki" writes: > > > On Mon, Jan 30, 2017 at 4:47 AM, Michael Ellerman > > wrote: > >> "Gautham R. Shenoy" writes: > >> > >>> From: "Gautham R. Shenoy" > >>> > >>> In the current code for powernv_add_idle_states,

Re: [PATCH v6 3/5] cpuidle:powernv: Add helper function to populate powernv idle states.

2017-01-30 Thread Michael Ellerman
"Rafael J. Wysocki" writes: > On Mon, Jan 30, 2017 at 4:47 AM, Michael Ellerman wrote: >> "Gautham R. Shenoy" writes: >> >>> From: "Gautham R. Shenoy" >>> >>> In the current code for powernv_add_idle_states, there is a lot of code >>> duplication while initializing an idle state in powernv_sta

Re: [PATCH v6 3/5] cpuidle:powernv: Add helper function to populate powernv idle states.

2017-01-29 Thread Rafael J. Wysocki
On Mon, Jan 30, 2017 at 4:47 AM, Michael Ellerman wrote: > "Gautham R. Shenoy" writes: > >> From: "Gautham R. Shenoy" >> >> In the current code for powernv_add_idle_states, there is a lot of code >> duplication while initializing an idle state in powernv_states table. >> >> Add an inline helper

Re: [PATCH v6 3/5] cpuidle:powernv: Add helper function to populate powernv idle states.

2017-01-29 Thread Michael Ellerman
"Gautham R. Shenoy" writes: > From: "Gautham R. Shenoy" > > In the current code for powernv_add_idle_states, there is a lot of code > duplication while initializing an idle state in powernv_states table. > > Add an inline helper function to populate the powernv_states[] table > for a given idle

[PATCH v6 3/5] cpuidle:powernv: Add helper function to populate powernv idle states.

2017-01-25 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" In the current code for powernv_add_idle_states, there is a lot of code duplication while initializing an idle state in powernv_states table. Add an inline helper function to populate the powernv_states[] table for a given idle state. Invoke this for populating the "Nap