cpufreq: stats: cleanups
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v3 Latest16 patches 1 replyIn review
- v214 patches 24 repliesSuperseded
- v113 patches 7 repliesSuperseded
V1->V2 from the cover letter
- New patches:
cpufreq: stats: remove unused cpufreq_stats_attribute
cpufreq: stats: initialize 'cur_time' on its definition
cpufreq: stats: rename 'struct cpufreq_stats' objects as 'stats'
cpufreq: stats: drop unnecessary locking
- Patches dropped:
cpufreq: stats: replace spinlock with mutex
cpufreq: stats: Fix locking
cpufreq: stats: call cpufreq_stats_update() with locks held
- Logs improved
- don't return error values notifiers on missing policy/stats
- rename stats_data as stats and make it cpufreq_stats pointer instead of void *
Patches in v3 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/16 | cpufreq: stats: Improve module description stringdrivers/ | +1 | 0 replies | |
| 2/16 | cpufreq: stats: return -EEXIST when stats are already allocateddrivers/ | +3 | 0 replies | |
| 3/16 | cpufreq: stats: remove unused cpufreq_stats_attributedrivers/ | +0 | 0 replies | |
| 4/16 | cpufreq: stats: initialize 'cur_time' on its definitiondrivers/ | +1 | 0 replies | |
| 5/16 | cpufreq: stats: don't check for freq table while freeing statsdrivers/ | +1 | 0 replies | |
| 6/16 | cpufreq: stats: pass 'stat' to cpufreq_stats_update()drivers/ | +4 | 0 replies | |
| 7/16 | cpufreq: stats: get rid of per-cpu cpufreq_stats_tabledrivers/ include/ | +28 | 0 replies | |
| 7/16 | cpufreq: stats: get rid of per-cpu cpufreq_stats_tabledrivers/ include/ | +32 | 0 replies | |
| 8/16 | cpufreq: stats: rename 'struct cpufreq_stats' objects as 'stats'drivers/ | +51 | 0 replies | |
| 9/16 | cpufreq: Remove (now) unused 'last_cpu' from struct cpufreq_policydrivers/ include/ | +0 | 0 replies | |
| 10/16 | cpufreq: stats: drop 'cpu' field of struct cpufreq_statsdrivers/ | +0 | 0 replies | |
| 11/16 | cpufreq: remove CPUFREQ_UPDATE_POLICY_CPU notificationsdrivers/ include/ | +2 | 0 replies | |
| 12/16 | cpufreq: stats: create sysfs group once we are readydrivers/ | +25 | 0 replies | |
| 13/16 | cpufreq: stats: time_in_state can't be NULL in cpufreq_stats_update()drivers/ | +1 | 0 replies | |
| 14/16 | cpufreq: stats: don't update stats from show_trans_table()drivers/ | +0 | 0 replies | |
| 15/16 | cpufreq: stats: don't update stats on false notifiersdrivers/ | +2 | 0 replies | |
| 16/16 | cpufreq: stats: drop unnecessary lockingdrivers/ | +1 | 0 replies |
Reviews trailers per patch
12 of 17 patches have a Reviewed-by or Acked-by.
| Patch | Prarit Bhargava |
|---|---|
| 1/16 | R-b |
| 2/16 | R-b |
| 3/16 | |
| 4/16 | |
| 5/16 | R-b |
| 6/16 | R-b |
| 7/16 | R-b R-b |
| 7/16 | R-b R-b |
| 8/16 | |
| 9/16 | R-b |
| 10/16 | R-b |
| 11/16 | R-b |
| 12/16 | R-b |
| 13/16 | |
| 14/16 | R-b |
| 15/16 | R-b |
| 16/16 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH V3 00/16] cpufreq: stats: cleanups
Hi Rafael,
This is V3 of the stats cleanup I sent earlier. Few things are improved based on the feedback received (mostly from you). Please see if it looks any better.
Pushed here:git://git.linaro.org/people/viresh.kumar/linux.git cpufreq/stats/cleanups
V1->V2:
- New patches:
cpufreq: stats: remove unused cpufreq_stats_attribute
cpufreq: stats: initialize 'cur_time' on its definition
cpufreq: stats: rename 'struct cpufreq_stats' objects as 'stats'
cpufreq: stats: drop unnecessary locking
- Patches dropped:
cpufreq: stats: replace spinlock with mutex
cpufreq: stats: Fix locking
cpufreq: stats: call cpufreq_stats_update() with locks held
- Logs improved
- don't return error values notifiers on missing policy/stats
- rename stats_data as stats and make it cpufreq_stats pointer instead of void *
Viresh Kumar (16):
cpufreq: stats: Improve module description string
cpufreq: stats: return -EEXIST when stats are already allocated
cpufreq: stats: remove unused cpufreq_stats_attribute
cpufreq: stats: initialize 'cur_time' on its definition
cpufreq: stats: don't check for freq table while freeing stats
cpufreq: stats: pass 'stat' to cpufreq_stats_update()
cpufreq: stats: get rid of per-cpu cpufreq_stats_table
cpufreq: stats: rename 'struct cpufreq_stats' objects as 'stats'
cpufreq: Remove (now) unused 'last_cpu' from struct cpufreq_policy
cpufreq: stats: drop 'cpu' field of struct cpufreq_stats
cpufreq: remove CPUFREQ_UPDATE_POLICY_CPU notifications
cpufreq: stats: create sysfs group once we are ready
cpufreq: stats: time_in_state can't be NULL in cpufreq_stats_update()
cpufreq: stats: don't update stats from show_trans_table()
cpufreq: stats: don't update stats on false notifiers
cpufreq: stats: drop unnecessary locking
drivers/cpufreq/cpufreq.c | 6 -- drivers/cpufreq/cpufreq_stats.c | 211 ++++++++++++++++++---------------------- include/linux/cpufreq.h | 10 +- 3 files changed, 98 insertions(+), 129 deletions(-)
Signature · 1 line
2.2.0