lkml.org 
[lkml]   [2013]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 28/37] cpufreq: pxa: don't initialize part of policy that is set by core too
    Date
    Many common initializations of struct policy are moved to core now and hence
    this driver doesn't need to do it. This patch removes such code.

    Most recent of those changes is to call ->get() in the core after calling
    ->init().

    Cc: Eric Miao <eric.y.miao@gmail.com>
    Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
    ---
    drivers/cpufreq/pxa2xx-cpufreq.c | 2 --
    drivers/cpufreq/pxa3xx-cpufreq.c | 7 +++----
    2 files changed, 3 insertions(+), 6 deletions(-)

    diff --git a/drivers/cpufreq/pxa2xx-cpufreq.c b/drivers/cpufreq/pxa2xx-cpufreq.c
    index a91274a..183bc13 100644
    --- a/drivers/cpufreq/pxa2xx-cpufreq.c
    +++ b/drivers/cpufreq/pxa2xx-cpufreq.c
    @@ -388,8 +388,6 @@ static int pxa_cpufreq_init(struct cpufreq_policy *policy)

    /* set default policy and cpuinfo */
    policy->cpuinfo.transition_latency = 1000; /* FIXME: 1 ms, assumed */
    - policy->cur = get_clk_frequency_khz(0); /* current freq */
    - policy->min = policy->max = policy->cur;

    /* Generate pxa25x the run cpufreq_frequency_table struct */
    for (i = 0; i < NUM_PXA25x_RUN_FREQS; i++) {
    diff --git a/drivers/cpufreq/pxa3xx-cpufreq.c b/drivers/cpufreq/pxa3xx-cpufreq.c
    index 61e3e6a..132e37d 100644
    --- a/drivers/cpufreq/pxa3xx-cpufreq.c
    +++ b/drivers/cpufreq/pxa3xx-cpufreq.c
    @@ -190,11 +190,10 @@ static int pxa3xx_cpufreq_init(struct cpufreq_policy *policy)
    int ret = -EINVAL;

    /* set default policy and cpuinfo */
    - policy->cpuinfo.min_freq = 104000;
    - policy->cpuinfo.max_freq = (cpu_is_pxa320()) ? 806000 : 624000;
    + policy->min = policy->cpuinfo.min_freq = 104000;
    + policy->max = policy->cpuinfo.max_freq =
    + (cpu_is_pxa320()) ? 806000 : 624000;
    policy->cpuinfo.transition_latency = 1000; /* FIXME: 1 ms, assumed */
    - policy->max = pxa3xx_get_clk_frequency_khz(0);
    - policy->cur = policy->min = policy->max;

    if (cpu_is_pxa300() || cpu_is_pxa310())
    ret = setup_freqs_table(policy, pxa300_freqs,
    --
    1.7.12.rc2.18.g61b472e


    \
     
     \ /
      Last update: 2013-08-14 16:01    [W:4.015 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site