On Tue, Jan 8, 2013 at 2:50 AM, Inderpal Singh <inderpal.singh@xxxxxxxxxx> wrote: > Add freq_attr attribute to show list of available frequencies. > > Signed-off-by: Donggeun Kim <dg77.kim@xxxxxxxxxxx> > Signed-off-by: MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx> > Signed-off-by: KyungMin Park <kyungmin.park@xxxxxxxxxxx> > Signed-off-by: Inderpal Singh <inderpal.singh@xxxxxxxxxx> > --- > drivers/cpufreq/exynos-cpufreq.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/drivers/cpufreq/exynos-cpufreq.c b/drivers/cpufreq/exynos-cpufreq.c > index 7012ea8..bc1e833 100644 > --- a/drivers/cpufreq/exynos-cpufreq.c > +++ b/drivers/cpufreq/exynos-cpufreq.c > @@ -244,13 +244,26 @@ static int exynos_cpufreq_cpu_init(struct cpufreq_policy *policy) > return cpufreq_frequency_table_cpuinfo(policy, exynos_info->freq_table); > } > > +static int exynos_cpufreq_cpu_exit(struct cpufreq_policy *policy) > +{ > + cpufreq_frequency_table_put_attr(policy->cpu); > + return 0; > +} > + > +static struct freq_attr *exynos_cpufreq_attr[] = { > + &cpufreq_freq_attr_scaling_available_freqs, > + NULL, > +}; > + This change looks fine. I guess this was posted before also but could not go mainline. Reviewed-by: Amit Daniel Kachhap<amit.daniel@xxxxxxxxxxx> Thanks, Amit Daniel > static struct cpufreq_driver exynos_driver = { > .flags = CPUFREQ_STICKY, > .verify = exynos_verify_speed, > .target = exynos_target, > .get = exynos_getspeed, > .init = exynos_cpufreq_cpu_init, > + .exit = exynos_cpufreq_cpu_exit, > .name = "exynos_cpufreq", > + .attr = exynos_cpufreq_attr, > #ifdef CONFIG_PM > .suspend = exynos_cpufreq_suspend, > .resume = exynos_cpufreq_resume, > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html