On Fri, Sep 09, 2016 at 05:04:59PM +0530, Viresh Kumar wrote:
On 09-09-16, 12:28, Russell King - ARM Linux wrote:
On Fri, Sep 09, 2016 at 04:52:04PM +0530, Viresh Kumar wrote:
Thanks for testing it.. You need another patch from Rafael, which should be in linux-next by now..
commit 3689ad7ed6a8 ("cpufreq: Drop unnecessary check from cpufreq_policy_alloc()")
Both patches combined will fix the problem you were getting.
Please send me this other patch. Seems absurd to fix a reported problem without copying appropriate patches to the reporter...
It got merged separately, and yes you should have been cc'd for that as well.
Please find it attached now..
Thanks, the warning is now gone. From what I can tell, the sysfs contents looks correct:
/sys/devices/system/cpu |-- cpu0 | |-- cpufreq -> ../cpufreq/policy0 | |-- power | | |-- autosuspend_delay_ms | | |-- control | | |-- runtime_active_time | | |-- runtime_status | | `-- runtime_suspended_time | |-- subsystem -> ../../../../bus/cpu | |-- topology | | |-- core_id | | |-- core_siblings | | |-- core_siblings_list | | |-- physical_package_id | | |-- thread_siblings | | `-- thread_siblings_list | `-- uevent |-- cpufreq | `-- policy0 | |-- affected_cpus | |-- cpuinfo_cur_freq | |-- cpuinfo_max_freq | |-- cpuinfo_min_freq | |-- cpuinfo_transition_latency | |-- related_cpus | |-- scaling_available_governors | |-- scaling_cur_freq | |-- scaling_driver | |-- scaling_governor | |-- scaling_max_freq | |-- scaling_min_freq | |-- scaling_setspeed | `-- stats | |-- time_in_state | `-- total_trans |-- isolated |-- kernel_max |-- offline |-- online |-- possible |-- power | |-- autosuspend_delay_ms | |-- control | |-- runtime_active_time | |-- runtime_status | `-- runtime_suspended_time |-- present `-- uevent
9 directories, 39 files
So, for both patches:
Tested-by: Russell King rmk+kernel@arm.linux.org.uk