Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-02-01 Thread Santosh Shilimkar
Viresh, On Friday 01 February 2013 02:22 PM, Santosh Shilimkar wrote: On Friday 01 February 2013 01:32 PM, Viresh Kumar wrote: On 1 February 2013 13:03, Santosh Shilimkar wrote: I am not talking about just notifiers. This is for external users who has subscribed for notifiers. The point is

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-02-01 Thread Santosh Shilimkar
On Friday 01 February 2013 01:32 PM, Viresh Kumar wrote: On 1 February 2013 13:03, Santosh Shilimkar wrote: I am not talking about just notifiers. This is for external users who has subscribed for notifiers. The point is whether the core CPUFReq gets updated without that flag for all affected

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-02-01 Thread Viresh Kumar
On 1 February 2013 13:03, Santosh Shilimkar wrote: > I am not talking about just notifiers. This is for external users who > has subscribed for notifiers. The point is whether the core CPUFReq > gets updated without that flag for all affected CPU. Yes, its safe. Follow this thread, yesterday i

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-02-01 Thread Viresh Kumar
On 1 February 2013 13:03, Santosh Shilimkar santosh.shilim...@ti.com wrote: I am not talking about just notifiers. This is for external users who has subscribed for notifiers. The point is whether the core CPUFReq gets updated without that flag for all affected CPU. Yes, its safe. Follow this

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-02-01 Thread Santosh Shilimkar
On Friday 01 February 2013 01:32 PM, Viresh Kumar wrote: On 1 February 2013 13:03, Santosh Shilimkar santosh.shilim...@ti.com wrote: I am not talking about just notifiers. This is for external users who has subscribed for notifiers. The point is whether the core CPUFReq gets updated without

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-02-01 Thread Santosh Shilimkar
Viresh, On Friday 01 February 2013 02:22 PM, Santosh Shilimkar wrote: On Friday 01 February 2013 01:32 PM, Viresh Kumar wrote: On 1 February 2013 13:03, Santosh Shilimkar santosh.shilim...@ti.com wrote: I am not talking about just notifiers. This is for external users who has subscribed for

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-01-31 Thread Santosh Shilimkar
On Friday 01 February 2013 12:43 PM, Viresh Kumar wrote: On 1 February 2013 12:17, Santosh Shilimkar wrote: I haven't looked at the cpufreq code recently but remember that it was needed to ensure that all the CPU which share clock/voltage gets updated (affected cpus) on freq change. The CPUs

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-01-31 Thread Viresh Kumar
On 1 February 2013 12:17, Santosh Shilimkar wrote: > I haven't looked at the cpufreq code recently but remember > that it was needed to ensure that all the CPU which > share clock/voltage gets updated (affected cpus) on > freq change. The CPUs which needs SW co-ordination, should > have this flag

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-01-31 Thread Santosh Shilimkar
On Friday 01 February 2013 12:10 PM, Viresh Kumar wrote: policy->shared_type field was added only for SoCs with ACPI support: commit 3b2d99429e3386b6e2ac949fc72486509c8bbe36 Author: Venkatesh Pallipadi Date: Wed Dec 14 15:05:00 2005 -0500 P-state software coordination for ACPI core

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-01-31 Thread Viresh Kumar
On 1 February 2013 12:10, Viresh Kumar wrote: > policy->shared_type field was added only for SoCs with ACPI support: > > commit 3b2d99429e3386b6e2ac949fc72486509c8bbe36 > Author: Venkatesh Pallipadi > Date: Wed Dec 14 15:05:00 2005 -0500 > > P-state software coordination for ACPI core > >

[PATCH 3/3] cpufreq: Remove unnecessary use of policy->shared_type

2013-01-31 Thread Viresh Kumar
policy->shared_type field was added only for SoCs with ACPI support: commit 3b2d99429e3386b6e2ac949fc72486509c8bbe36 Author: Venkatesh Pallipadi Date: Wed Dec 14 15:05:00 2005 -0500 P-state software coordination for ACPI core http://bugzilla.kernel.org/show_bug.cgi?id=5737 Many

[PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-01-31 Thread Viresh Kumar
policy-shared_type field was added only for SoCs with ACPI support: commit 3b2d99429e3386b6e2ac949fc72486509c8bbe36 Author: Venkatesh Pallipadi venkatesh.pallip...@intel.com Date: Wed Dec 14 15:05:00 2005 -0500 P-state software coordination for ACPI core

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-01-31 Thread Viresh Kumar
On 1 February 2013 12:10, Viresh Kumar viresh.ku...@linaro.org wrote: policy-shared_type field was added only for SoCs with ACPI support: commit 3b2d99429e3386b6e2ac949fc72486509c8bbe36 Author: Venkatesh Pallipadi venkatesh.pallip...@intel.com Date: Wed Dec 14 15:05:00 2005 -0500

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-01-31 Thread Santosh Shilimkar
On Friday 01 February 2013 12:10 PM, Viresh Kumar wrote: policy-shared_type field was added only for SoCs with ACPI support: commit 3b2d99429e3386b6e2ac949fc72486509c8bbe36 Author: Venkatesh Pallipadi venkatesh.pallip...@intel.com Date: Wed Dec 14 15:05:00 2005 -0500 P-state software

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-01-31 Thread Viresh Kumar
On 1 February 2013 12:17, Santosh Shilimkar santosh.shilim...@ti.com wrote: I haven't looked at the cpufreq code recently but remember that it was needed to ensure that all the CPU which share clock/voltage gets updated (affected cpus) on freq change. The CPUs which needs SW co-ordination,

Re: [PATCH 3/3] cpufreq: Remove unnecessary use of policy-shared_type

2013-01-31 Thread Santosh Shilimkar
On Friday 01 February 2013 12:43 PM, Viresh Kumar wrote: On 1 February 2013 12:17, Santosh Shilimkar santosh.shilim...@ti.com wrote: I haven't looked at the cpufreq code recently but remember that it was needed to ensure that all the CPU which share clock/voltage gets updated (affected cpus) on