Jean Pihet <jean.pihet@xxxxxxxxxxxxxx> writes: > On Thu, Mar 10, 2011 at 1:08 AM, Kevin Hilman <khilman@xxxxxx> wrote: >> Jean Pihet <jean.pihet@xxxxxxxxxxxxxx> writes: >> >>> The patch adds the new power management trace points for >>> the OMAP architecture. >>> >>> The trace points are for: >>> - default idle handler. Since the cpuidle framework is >>> Â instrumented in the generic way there is no need to >>> Â add trace points in the OMAP specific cpuidle handler; >>> - cpufreq (DVFS), >> >> Minor: AFAICT, this patch doesn't instrument CPUfreq. > The generic framework provides a cpu_frequency event. Furthermore the > clock_set_rate events are reporting the various SoC clock rate > changes. > Indeed the changelog is not accurate enough. Is '- cpufreq (DVFS) > events are covered by the generic cpu_frequency events and the SoC > clock changes events' better? I'd prefer to just leave this out as either way, I find it confusing based on the content of this patch. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
