From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756628Ab3BESY1 (ORCPT ); Tue, 5 Feb 2013 13:24:27 -0500 Received: from mail-pa0-f42.google.com ([209.85.220.42]:36065 "EHLO mail-pa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756549Ab3BESYY (ORCPT ); Tue, 5 Feb 2013 13:24:24 -0500 From: dirk.brandewie@gmail.com To: linux-kernel@vger.kernel.org, cpufreq@vger.kernel.org Cc: Dirk Brandewie , Dirk Brandewie Subject: [PATCH 2/7] cpufreq: Retrieve current frequency from scaling drivers with internal governors Date: Tue, 5 Feb 2013 10:24:01 -0800 Message-Id: <1360088646-19104-3-git-send-email-dirk.brandewie@gmail.com> X-Mailer: git-send-email 1.7.7.6 In-Reply-To: <1360088646-19104-1-git-send-email-dirk.brandewie@gmail.com> References: <1360088646-19104-1-git-send-email-dirk.brandewie@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Dirk Brandewie Scaling drivers that implement the cpufreq_driver.setpolicy() versus the cpufreq_driver.target() interface do not set policy->cur. Normally policy->cur is set during the call to cpufreq_driver.target() when the frequnecy request is made by the governor. If the scaling driver implements cpufreq_driver.setpolicy() and cpufreq_driver.get() interfaces use cpufreq_driver.get() to retrieve the current frequency. Signed-off-by: Dirk Brandewie From: Dirk Brandewie Date: 2013-02-04 18:44:40 -0800 cpufreq: balance out cpufreq_cpu_{get,put} for scaling drivers using setpolicy There is an additional reference added to the driver in cpufreq_add_dev() that is removed in__cpufreq_governor() if the driver implements target(). Remove the last reference when the driver implements setpolicy() Signed-off-by: Dirk Brandewie --- drivers/cpufreq/cpufreq.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 2817c3c..96bc302 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1181,6 +1181,13 @@ unsigned int cpufreq_quick_get(unsigned int cpu) struct cpufreq_policy *policy = cpufreq_cpu_get(cpu); unsigned int ret_freq = 0; + if (cpufreq_driver && cpufreq_driver->setpolicy && + cpufreq_driver->get) { + ret_freq = cpufreq_driver->get(cpu); + cpufreq_cpu_put(policy); + return ret_freq; + } + if (policy) { ret_freq = policy->cur; cpufreq_cpu_put(policy); -- 1.7.7.6