On 8 August 2013 21:14, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote: > On 08/08/2013 07:48 AM, Viresh Kumar wrote: >> Lets use cpufreq_table_validate_and_show() instead of calling >> cpufreq_frequency_table_cpuinfo() and cpufreq_frequency_table_get_attr(). > > Assuming the new function simply does what the two removed lines do, > this looks fine to me. http://lkml.org/lkml/2013/8/8/266 >> static int tegra_cpu_exit(struct cpufreq_policy *policy) >> { >> - cpufreq_frequency_table_cpuinfo(policy, freq_table); > > This doesn't seem to be mentioned in the commit description. I assume > this is simply dead/useless code removal? Its useless and the correct routine isn't called at all :) .. I will add that additional patch and send it to you and will get this change out of this commit. -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html