From patchwork Wed Nov 9 03:37:45 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: huang lin X-Patchwork-Id: 9418437 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id BEB836048E for ; Wed, 9 Nov 2016 03:38:09 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id B4E8728620 for ; Wed, 9 Nov 2016 03:38:09 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id A826928705; Wed, 9 Nov 2016 03:38:09 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D8F50286CD for ; Wed, 9 Nov 2016 03:38:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752043AbcKIDh6 (ORCPT ); Tue, 8 Nov 2016 22:37:58 -0500 Received: from mail-pf0-f193.google.com ([209.85.192.193]:36720 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751800AbcKIDh6 (ORCPT ); Tue, 8 Nov 2016 22:37:58 -0500 Received: by mail-pf0-f193.google.com with SMTP id n85so21840099pfi.3; Tue, 08 Nov 2016 19:37:57 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=KBl6FJU/x8AJRspNftFOIsyDkr6Wvp05q+jI2Owyr78=; b=IAMMvMffvnqGV+3UZgP5YdO7aUKAwn/OHsDJA78dcBti0qnCPBOQz1hNC+IwFWFBNl P/tMlv5KdSYpGwrx50ImUoxy8bumaLQFYmxbyz/o6Z55zOmrQSLS9e9Dlyr6Su6SZ1Ie bSQnK/YM8e3521jedryoqsDy13l9ZWIOQ4WkUc0sn5SQ+RvUdkKks70I4mHdaup/TfIN Y8vVxtL+CuYjKfEzdklH9gjMZ9mvlIRxhBVdEVWV6YwfEs44gWwKxULnEzJv2vg/y29r qMqREDNE8VMgyN4w8nSfxlEkgMFRvR0U7xhvoLHymXuTfJmC17huHSL1sP0wo1ssNHn0 GysA== X-Gm-Message-State: ABUngveagFb/r9M7YHzar5u02GLFwPu3UpV76Me0/+OUE7OCVAN2v2Wth65ecgK2tdw8wg== X-Received: by 10.99.60.11 with SMTP id j11mr11739266pga.26.1478662677280; Tue, 08 Nov 2016 19:37:57 -0800 (PST) Received: from localhost.localdomain ([103.29.142.67]) by smtp.gmail.com with ESMTPSA id g10sm10153338pac.14.2016.11.08.19.37.54 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 08 Nov 2016 19:37:56 -0800 (PST) From: Lin Huang To: myungjoo.ham@samsung.com Cc: cw00.choi@samsung.com, dianders@chromium.org, linux-rockchip@lists.infradead.org, linux-pm@vger.kernel.org, dbasehore@chromium.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Lin Huang Subject: [PATCH v6] PM/devfreq: add suspend frequency support Date: Wed, 9 Nov 2016 11:37:45 +0800 Message-Id: <1478662665-6357-1-git-send-email-hl@rock-chips.com> X-Mailer: git-send-email 1.9.1 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Add suspend frequency support and if needed set it to the frequency obtained from the suspend opp (can be defined using opp-v2 bindings and is optional). Signed-off-by: Lin Huang --- Changes in v2: - use update_devfreq() instead devfreq_update_status() Changes in v3: - fix build error Changes in v4: - move dev_pm_opp_get_suspend_opp() to devfreq_add_device() Changes in v5: - delete devfreq_opp_get_suspend_opp() in devfreq.h Changes in v6: - return to use stop_polling check suspend status drivers/devfreq/devfreq.c | 14 +++++++++++--- drivers/devfreq/governor_simpleondemand.c | 9 +++++++++ include/linux/devfreq.h | 1 + 3 files changed, 21 insertions(+), 3 deletions(-) diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c index da72d97..cfa64a0 100644 --- a/drivers/devfreq/devfreq.c +++ b/drivers/devfreq/devfreq.c @@ -359,9 +359,11 @@ void devfreq_monitor_suspend(struct devfreq *devfreq) mutex_unlock(&devfreq->lock); return; } - - devfreq_update_status(devfreq, devfreq->previous_freq); devfreq->stop_polling = true; + if (devfreq->suspend_freq) + update_devfreq(devfreq); + else + devfreq_update_status(devfreq, devfreq->previous_freq); mutex_unlock(&devfreq->lock); cancel_delayed_work_sync(&devfreq->work); } @@ -390,7 +392,6 @@ void devfreq_monitor_resume(struct devfreq *devfreq) devfreq->last_stat_updated = jiffies; devfreq->stop_polling = false; - if (devfreq->profile->get_cur_freq && !devfreq->profile->get_cur_freq(devfreq->dev.parent, &freq)) devfreq->previous_freq = freq; @@ -524,6 +525,7 @@ struct devfreq *devfreq_add_device(struct device *dev, struct devfreq *devfreq; struct devfreq_governor *governor; int err = 0; + struct dev_pm_opp *suspend_opp; if (!dev || !profile || !governor_name) { dev_err(dev, "%s: Invalid parameters.\n", __func__); @@ -558,6 +560,12 @@ struct devfreq *devfreq_add_device(struct device *dev, devfreq->data = data; devfreq->nb.notifier_call = devfreq_notifier_call; + rcu_read_lock(); + suspend_opp = dev_pm_opp_get_suspend_opp(dev); + if (suspend_opp) + devfreq->suspend_freq = dev_pm_opp_get_freq(suspend_opp); + rcu_read_unlock(); + if (!devfreq->profile->max_state && !devfreq->profile->freq_table) { mutex_unlock(&devfreq->lock); devfreq_set_freq_table(devfreq); diff --git a/drivers/devfreq/governor_simpleondemand.c b/drivers/devfreq/governor_simpleondemand.c index ae72ba5..b82f089 100644 --- a/drivers/devfreq/governor_simpleondemand.c +++ b/drivers/devfreq/governor_simpleondemand.c @@ -29,6 +29,15 @@ static int devfreq_simple_ondemand_func(struct devfreq *df, struct devfreq_simple_ondemand_data *data = df->data; unsigned long max = (df->max_freq) ? df->max_freq : UINT_MAX; + /* + * if devfreq in suspend status and have suspend_freq, + * the frequency need to set to suspend_freq + */ + if (df->stop_polling) { + *freq = df->suspend_freq; + return 0; + } + err = devfreq_update_stats(df); if (err) return err; diff --git a/include/linux/devfreq.h b/include/linux/devfreq.h index 98c6993..517e394 100644 --- a/include/linux/devfreq.h +++ b/include/linux/devfreq.h @@ -172,6 +172,7 @@ struct devfreq { struct delayed_work work; unsigned long previous_freq; + unsigned long suspend_freq; struct devfreq_dev_status last_status; void *data; /* private data for governors */