Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Ulf Hansson
On Fri, 8 Mar 2019 at 14:31, Lorenzo Pieralisi wrote: > > On Fri, Mar 08, 2019 at 02:23:26PM +0100, Ulf Hansson wrote: > > On Fri, 8 Mar 2019 at 14:17, Lorenzo Pieralisi > > wrote: > > > > > > On Fri, Mar 08, 2019 at 02:07:51PM +0100, Ulf Hansson wrote: > > > > On Fri, 8 Mar 2019 at 12:49,

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Lorenzo Pieralisi
On Fri, Mar 08, 2019 at 02:23:26PM +0100, Ulf Hansson wrote: > On Fri, 8 Mar 2019 at 14:17, Lorenzo Pieralisi > wrote: > > > > On Fri, Mar 08, 2019 at 02:07:51PM +0100, Ulf Hansson wrote: > > > On Fri, 8 Mar 2019 at 12:49, Lorenzo Pieralisi > > > wrote: > > > > > > > > On Fri, Mar 08, 2019 at

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Ulf Hansson
On Fri, 8 Mar 2019 at 14:17, Lorenzo Pieralisi wrote: > > On Fri, Mar 08, 2019 at 02:07:51PM +0100, Ulf Hansson wrote: > > On Fri, 8 Mar 2019 at 12:49, Lorenzo Pieralisi > > wrote: > > > > > > On Fri, Mar 08, 2019 at 11:36:49AM +0100, Ulf Hansson wrote: > > > > > > [...] > > > > > > > Instead,

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Lorenzo Pieralisi
On Fri, Mar 08, 2019 at 02:07:51PM +0100, Ulf Hansson wrote: > On Fri, 8 Mar 2019 at 12:49, Lorenzo Pieralisi > wrote: > > > > On Fri, Mar 08, 2019 at 11:36:49AM +0100, Ulf Hansson wrote: > > > > [...] > > > > > Instead, my suggestion is according to what I propose in patch 4 and > > > $subject

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Ulf Hansson
On Fri, 8 Mar 2019 at 12:49, Lorenzo Pieralisi wrote: > > On Fri, Mar 08, 2019 at 11:36:49AM +0100, Ulf Hansson wrote: > > [...] > > > Instead, my suggestion is according to what I propose in patch 4 and > > $subject patch, which means minor adjustments to be able to pass the > > struct

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Lorenzo Pieralisi
On Fri, Mar 08, 2019 at 11:36:49AM +0100, Ulf Hansson wrote: [...] > Instead, my suggestion is according to what I propose in patch 4 and > $subject patch, which means minor adjustments to be able to pass the > struct cpuidle_driver * to the init functions. This, I need it for > next steps, but

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-08 Thread Ulf Hansson
Lorenzo, Mark, On Wed, 6 Mar 2019 at 19:15, Lorenzo Pieralisi wrote: > > On Mon, Mar 04, 2019 at 11:14:18AM +0100, Ulf Hansson wrote: > > On Fri, 1 Mar 2019 at 18:28, Mark Rutland wrote: > > > > > > On Thu, Feb 28, 2019 at 02:59:17PM +0100, Ulf Hansson wrote: > > > > Instead of iterating

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-06 Thread Lorenzo Pieralisi
On Mon, Mar 04, 2019 at 11:14:18AM +0100, Ulf Hansson wrote: > On Fri, 1 Mar 2019 at 18:28, Mark Rutland wrote: > > > > On Thu, Feb 28, 2019 at 02:59:17PM +0100, Ulf Hansson wrote: > > > Instead of iterating through all the state nodes in DT, to find out how > > > many states that needs to be

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-04 Thread Ulf Hansson
On Fri, 1 Mar 2019 at 18:28, Mark Rutland wrote: > > On Thu, Feb 28, 2019 at 02:59:17PM +0100, Ulf Hansson wrote: > > Instead of iterating through all the state nodes in DT, to find out how > > many states that needs to be allocated, let's use the number already known > > by the cpuidle driver.

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-03-01 Thread Mark Rutland
On Thu, Feb 28, 2019 at 02:59:17PM +0100, Ulf Hansson wrote: > Instead of iterating through all the state nodes in DT, to find out how > many states that needs to be allocated, let's use the number already known > by the cpuidle driver. In this way we can drop the iteration altogether. > >

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-02-28 Thread Daniel Lezcano
On 28/02/2019 23:26, Ulf Hansson wrote: > On Thu, 28 Feb 2019 at 16:40, Daniel Lezcano > wrote: >> >> On 28/02/2019 14:59, Ulf Hansson wrote: >>> Instead of iterating through all the state nodes in DT, to find out how >>> many states that needs to be allocated, let's use the number already known

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-02-28 Thread Ulf Hansson
On Thu, 28 Feb 2019 at 16:40, Daniel Lezcano wrote: > > On 28/02/2019 14:59, Ulf Hansson wrote: > > Instead of iterating through all the state nodes in DT, to find out how > > many states that needs to be allocated, let's use the number already known > > by the cpuidle driver. In this way we can

Re: [PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-02-28 Thread Daniel Lezcano
On 28/02/2019 14:59, Ulf Hansson wrote: > Instead of iterating through all the state nodes in DT, to find out how > many states that needs to be allocated, let's use the number already known > by the cpuidle driver. In this way we can drop the iteration altogether. > > Signed-off-by: Ulf Hansson

[PATCH 5/7] drivers: firmware: psci: Simplify state node parsing

2019-02-28 Thread Ulf Hansson
Instead of iterating through all the state nodes in DT, to find out how many states that needs to be allocated, let's use the number already known by the cpuidle driver. In this way we can drop the iteration altogether. Signed-off-by: Ulf Hansson --- drivers/firmware/psci/psci.c | 25