Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-14 Thread Rafael J. Wysocki
On Thursday, February 14, 2013 07:34:56 AM Dirk Brandewie wrote: > On 02/14/2013 04:21 AM, Rafael J. Wysocki wrote: > > On Thursday, February 14, 2013 09:38:21 AM Viresh Kumar wrote: > >> On Wed, Feb 13, 2013 at 10:08 PM, Dirk Brandewie > >> wrote: > >>> For the case where both are built-in the lo

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-14 Thread Dirk Brandewie
On 02/14/2013 04:21 AM, Rafael J. Wysocki wrote: On Thursday, February 14, 2013 09:38:21 AM Viresh Kumar wrote: On Wed, Feb 13, 2013 at 10:08 PM, Dirk Brandewie wrote: For the case where both are built-in the load order works my driver uses device_initcall() and acpi_cpufreq uses late_initcall

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-14 Thread Dirk Brandewie
On 02/14/2013 04:21 AM, Rafael J. Wysocki wrote: On Thursday, February 14, 2013 09:38:21 AM Viresh Kumar wrote: On Wed, Feb 13, 2013 at 10:08 PM, Dirk Brandewie wrote: For the case where both are built-in the load order works my driver uses device_initcall() and acpi_cpufreq uses late_initcall

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-14 Thread Rafael J. Wysocki
On Thursday, February 14, 2013 09:38:21 AM Viresh Kumar wrote: > On Wed, Feb 13, 2013 at 10:08 PM, Dirk Brandewie > wrote: > > For the case where both are built-in the load order works my driver uses > > device_initcall() and acpi_cpufreq uses late_initcall(). > > > > For the case where both are a

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-13 Thread Viresh Kumar
On Wed, Feb 13, 2013 at 10:08 PM, Dirk Brandewie wrote: > For the case where both are built-in the load order works my driver uses > device_initcall() and acpi_cpufreq uses late_initcall(). > > For the case where both are a module (which I was sure I tested) you are > right > I will have to do som

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-13 Thread Rafael J. Wysocki
On Wednesday, February 13, 2013 08:38:04 AM Dirk Brandewie wrote: > Hi Dave, > > On 02/12/2013 01:49 PM, Dave Jones wrote: > > On Wed, Feb 06, 2013 at 09:02:07AM -0800, dirk.brande...@gmail.com wrote: > > > > Won't you also need to patch drivers/cpufreq/acpi-cpufreq.c to not load > > on the proces

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-13 Thread Dirk Brandewie
Hi Dave, On 02/12/2013 01:49 PM, Dave Jones wrote: On Wed, Feb 06, 2013 at 09:02:07AM -0800, dirk.brande...@gmail.com wrote: Won't you also need to patch drivers/cpufreq/acpi-cpufreq.c to not load on the processors that you want this driver to run on ? Dave For the case where both a

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-12 Thread Dave Jones
On Wed, Feb 06, 2013 at 09:02:07AM -0800, dirk.brande...@gmail.com wrote: > From: Dirk Brandewie > > This driver implements a scaling driver with an internal governor for > Intel Core processors. The driver follows the same model as the > Transmeta scaling driver (longrun.c) and implements

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-07 Thread Rafael J. Wysocki
On Thursday, February 07, 2013 07:32:26 PM Viresh Kumar wrote: > On Thu, Feb 7, 2013 at 4:12 PM, Viresh Kumar wrote: > > @Rafael: I wasn't sure if these would apply over my patches without any > > conflicts and so i applied them in my repo. Added my Acked-by's and fixed > > a minor thing in one of

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-07 Thread Viresh Kumar
On Thu, Feb 7, 2013 at 4:12 PM, Viresh Kumar wrote: > @Rafael: I wasn't sure if these would apply over my patches without any > conflicts and so i applied them in my repo. Added my Acked-by's and fixed > a minor thing in one of the patches (as mentioned in the other mail). > > You can pick these f

Re: [PATCH 0/5] Add P state driver for Intel Core Processors

2013-02-07 Thread Viresh Kumar
On Wed, Feb 6, 2013 at 10:32 PM, wrote: > From: Dirk Brandewie > > This driver implements a scaling driver with an internal governor for > Intel Core processors. The driver follows the same model as the > Transmeta scaling driver (longrun.c) and implements the setpolicy() > instead of target().