RE: [linux-pm] [PATCH] cpufreq: exynos: Show list of available frequencies

2013-01-17 Thread Kukjin Kim 
amit kachhap wrote:
 
 On Tue, Jan 8, 2013 at 2:50 AM, Inderpal Singh
 inderpal.si...@linaro.org wrote:
  Add freq_attr attribute to show list of available frequencies.
 
  Signed-off-by: Donggeun Kim dg77@samsung.com
  Signed-off-by: MyungJoo Ham myungjoo@samsung.com
  Signed-off-by: KyungMin Park kyungmin.p...@samsung.com
  Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
  ---
   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 Kachhapamit.dan...@samsung.com
 
Looks OK to me, applied.

Thanks.

- Kukjin

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [linux-pm] [PATCH] cpufreq: exynos: Show list of available frequencies

2013-01-08 Thread amit kachhap 
On Tue, Jan 8, 2013 at 2:50 AM, Inderpal Singh
inderpal.si...@linaro.org wrote:
 Add freq_attr attribute to show list of available frequencies.

 Signed-off-by: Donggeun Kim dg77@samsung.com
 Signed-off-by: MyungJoo Ham myungjoo@samsung.com
 Signed-off-by: KyungMin Park kyungmin.p...@samsung.com
 Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
 ---
  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 Kachhapamit.dan...@samsung.com

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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html