Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-11 Thread Daniel Lezcano
On 10/11/2012 07:21 PM, Rafael J. Wysocki wrote: > On Thursday 11 of October 2012 12:04:37 Daniel Lezcano wrote: >> On 10/07/2012 11:26 PM, Rafael J. Wysocki wrote: >>> On Tuesday 25 of September 2012 00:43:54 Daniel Lezcano wrote: With the tegra3 and the big.LITTLE [1] new architectures, seve

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-11 Thread Rafael J. Wysocki
On Thursday 11 of October 2012 12:04:37 Daniel Lezcano wrote: > On 10/07/2012 11:26 PM, Rafael J. Wysocki wrote: > > On Tuesday 25 of September 2012 00:43:54 Daniel Lezcano wrote: > >> With the tegra3 and the big.LITTLE [1] new architectures, several cpus > >> with different characteristics (latenc

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-11 Thread Daniel Lezcano
On 10/07/2012 11:26 PM, Rafael J. Wysocki wrote: > On Tuesday 25 of September 2012 00:43:54 Daniel Lezcano wrote: >> With the tegra3 and the big.LITTLE [1] new architectures, several cpus >> with different characteristics (latencies and states) can co-exists on the >> system. >> >> The cpuidle fram

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-07 Thread Rafael J. Wysocki
On Tuesday 25 of September 2012 00:43:54 Daniel Lezcano wrote: > With the tegra3 and the big.LITTLE [1] new architectures, several cpus > with different characteristics (latencies and states) can co-exists on the > system. > > The cpuidle framework has the limitation of handling only identical cpu

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-07 Thread Rafael J. Wysocki
On Thursday 04 of October 2012 17:34:42 Peter De Schrijver wrote: > > >> > > >> eg. > > >> cpu0: acpi_idle > > >> cpu1: acpi_idle > > >> > > > > > > Is this allowed? I thought sysfs files can output only 1 line of data? > > > > Ah yeah :) > > > > Does it make sense to put the following files: >

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-04 Thread Peter De Schrijver
> >> > >> eg. > >> cpu0: acpi_idle > >> cpu1: acpi_idle > >> > > > > Is this allowed? I thought sysfs files can output only 1 line of data? > > Ah yeah :) > > Does it make sense to put the following files: > > /sys/devices/system/cpu/cpu0/cpuidle/driver > => acpi_idle > /sys/devices/system/cpu

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-04 Thread Daniel Lezcano
On 10/04/2012 04:14 PM, Peter De Schrijver wrote: > On Tue, Sep 25, 2012 at 12:43:54AM +0200, Daniel Lezcano wrote: >> With the tegra3 and the big.LITTLE [1] new architectures, several cpus >> with different characteristics (latencies and states) can co-exists on the >> system. >> >> The cpuidle fr

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-04 Thread Peter De Schrijver
On Tue, Sep 25, 2012 at 12:43:54AM +0200, Daniel Lezcano wrote: > With the tegra3 and the big.LITTLE [1] new architectures, several cpus > with different characteristics (latencies and states) can co-exists on the > system. > > The cpuidle framework has the limitation of handling only identical cp

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-02 Thread Rafael J. Wysocki
On Tuesday 02 of October 2012 14:27:17 Daniel Lezcano wrote: > On 10/02/2012 04:08 AM, Rafael J. Wysocki wrote: > > On Sunday 30 of September 2012 18:34:31 Daniel Lezcano wrote: > >> On 09/30/2012 12:07 AM, Rafael J. Wysocki wrote: > >>> On Saturday, September 29, 2012, Daniel Lezcano wrote: >

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-02 Thread Daniel Lezcano
On 10/02/2012 04:08 AM, Rafael J. Wysocki wrote: > On Sunday 30 of September 2012 18:34:31 Daniel Lezcano wrote: >> On 09/30/2012 12:07 AM, Rafael J. Wysocki wrote: >>> On Saturday, September 29, 2012, Daniel Lezcano wrote: On 09/29/2012 11:41 AM, Francesco Lavra wrote: > Hi, Hi Franc

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-10-01 Thread Rafael J. Wysocki
On Sunday 30 of September 2012 18:34:31 Daniel Lezcano wrote: > On 09/30/2012 12:07 AM, Rafael J. Wysocki wrote: > > On Saturday, September 29, 2012, Daniel Lezcano wrote: > >> On 09/29/2012 11:41 AM, Francesco Lavra wrote: > >>> Hi, > >> Hi Francesco, > >> > >> thanks for reviewing the patch. > >>

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-09-30 Thread Daniel Lezcano
On 09/30/2012 12:07 AM, Rafael J. Wysocki wrote: > On Saturday, September 29, 2012, Daniel Lezcano wrote: >> On 09/29/2012 11:41 AM, Francesco Lavra wrote: >>> Hi, >> Hi Francesco, >> >> thanks for reviewing the patch. >> static ssize_t show_current_driver(struct device *dev,

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-09-29 Thread Rafael J. Wysocki
On Saturday, September 29, 2012, Daniel Lezcano wrote: > On 09/29/2012 11:41 AM, Francesco Lavra wrote: > > Hi, > > Hi Francesco, > > thanks for reviewing the patch. > > >> static ssize_t show_current_driver(struct device *dev, > >> struct device_attribute *attr, >

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-09-29 Thread Daniel Lezcano
On 09/29/2012 11:41 AM, Francesco Lavra wrote: > Hi, Hi Francesco, thanks for reviewing the patch. >> static ssize_t show_current_driver(struct device *dev, >> struct device_attribute *attr, >> char *buf) >> { >> -ssize_t ret;

Re: [PATCH 4/4] cpuidle - support multiple drivers

2012-09-29 Thread Francesco Lavra
Hi, On 09/25/2012 12:43 AM, Daniel Lezcano wrote: ... > diff --git a/drivers/cpuidle/sysfs.c b/drivers/cpuidle/sysfs.c > index 5f809e3..2596422 100644 > --- a/drivers/cpuidle/sysfs.c > +++ b/drivers/cpuidle/sysfs.c > @@ -43,21 +43,46 @@ out: > return i; > } > > +struct cbarg { > + cha

[PATCH 4/4] cpuidle - support multiple drivers

2012-09-24 Thread Daniel Lezcano
With the tegra3 and the big.LITTLE [1] new architectures, several cpus with different characteristics (latencies and states) can co-exists on the system. The cpuidle framework has the limitation of handling only identical cpus. This patch removes this limitation by introducing the multiple driver