From: Viresh Kumar <viresh.kumar@linaro.org>
To: Rafael Wysocki <rjw@rjwysocki.net>,
ulf.hansson@linaro.org, Kevin Hilman <khilman@kernel.org>
Cc: Viresh Kumar <viresh.kumar@linaro.org>,
linux-pm@vger.kernel.org,
Vincent Guittot <vincent.guittot@linaro.org>,
Stephen Boyd <sboyd@codeaurora.org>, Nishanth Menon <nm@ti.com>,
robh+dt@kernel.org, lina.iyer@linaro.org, rnayak@codeaurora.org,
sudeep.holla@arm.com, linux-kernel@vger.kernel.org,
Len Brown <len.brown@intel.com>, Pavel Machek <pavel@ucw.cz>,
Andy Gross <andy.gross@linaro.org>,
David Brown <david.brown@linaro.org>
Subject: [NOT-FOR-MERGE V8 4/6] soc: qcom: rpmpd: Add support for get/set performance state
Date: Wed, 21 Jun 2017 12:40:51 +0530 [thread overview]
Message-ID: <f880d3e38cdbf07d7f4cbb26453be160b33c8635.1498026827.git.viresh.kumar@linaro.org> (raw)
In-Reply-To: <cover.1498026827.git.viresh.kumar@linaro.org>
In-Reply-To: <cover.1498026827.git.viresh.kumar@linaro.org>
From: Rajendra Nayak <rnayak@codeaurora.org>
THIS IS TEST CODE, SHOULDN'T BE MERGED.
With genpd now expecting powerdomain drivers supporting performance
state to support get/set performance state callbacks, add support for it
in the rpmpd driver.
NOT-signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>
NOT-signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
drivers/soc/qcom/rpmpd.c | 100 +++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 100 insertions(+)
diff --git a/drivers/soc/qcom/rpmpd.c b/drivers/soc/qcom/rpmpd.c
index d34d9c363815..7ef81429c5c5 100644
--- a/drivers/soc/qcom/rpmpd.c
+++ b/drivers/soc/qcom/rpmpd.c
@@ -101,6 +101,20 @@ struct rpmpd_desc {
size_t num_pds;
};
+enum rpmpd_levels {
+ NONE,
+ LOWER, /* SVS2 */
+ LOW, /* SVS */
+ NOMINAL, /* NOMINAL */
+ HIGH, /* Turbo */
+ MAX_LEVEL,
+};
+
+struct rpmpd_freq_map {
+ struct rpmpd *pd;
+ unsigned long freq[MAX_LEVEL];
+};
+
static DEFINE_MUTEX(rpmpd_lock);
/* msm8996 RPM powerdomains */
@@ -126,6 +140,47 @@ static const struct rpmpd_desc msm8996_desc = {
.num_pds = ARRAY_SIZE(msm8996_rpmpds),
};
+enum msm8996_devices {
+ SDHCI,
+ UFS,
+ PCIE,
+ USB3,
+};
+
+static struct rpmpd_freq_map msm8996_rpmpd_freq_map[] = {
+ [SDHCI] = {
+ .pd = &msm8996_vddcx,
+ .freq[LOWER] = 19200000,
+ .freq[LOW] = 200000000,
+ .freq[NOMINAL] = 400000000,
+ },
+ [UFS] = {
+ .pd = &msm8996_vddcx,
+ .freq[LOWER] = 19200000,
+ .freq[LOW] = 100000000,
+ .freq[NOMINAL] = 200000000,
+ .freq[HIGH] = 240000000,
+ },
+ [PCIE] = {
+ .pd = &msm8996_vddcx,
+ .freq[LOWER] = 1011000,
+ },
+ [USB3] = {
+ .pd = &msm8996_vddcx,
+ .freq[LOWER] = 60000000,
+ .freq[LOW] = 120000000,
+ .freq[NOMINAL] = 150000000,
+ },
+};
+
+static const struct of_device_id rpmpd_performance_table[] = {
+ { .compatible = "qcom,sdhci-msm-v4", .data = (void *)SDHCI },
+ { .compatible = "qcom,ufshc", .data = (void *)UFS },
+ { .compatible = "qcom,pcie-msm8996", .data = (void *)PCIE },
+ { .compatible = "qcom,dwc3", .data = (void *)USB3 },
+ { }
+};
+
static const struct of_device_id rpmpd_match_table[] = {
{ .compatible = "qcom,rpmpd-msm8996", .data = &msm8996_desc },
{ }
@@ -230,6 +285,49 @@ static int rpmpd_power_off(struct generic_pm_domain *domain)
return ret;
}
+static int rpmpd_set_performance(struct generic_pm_domain *domain,
+ unsigned int state)
+{
+ int ret = 0;
+ struct rpmpd *pd = domain_to_rpmpd(domain);
+
+ mutex_lock(&rpmpd_lock);
+
+ pd->corner = state;
+
+ if (!pd->enabled && (pd->key != KEY_FLOOR_CORNER))
+ goto out;
+
+ ret = rpmpd_aggregate_corner(pd);
+
+out:
+ mutex_unlock(&rpmpd_lock);
+
+ return ret;
+}
+
+
+static int rpmpd_get_performance(struct device *dev, unsigned long rate)
+{
+ int i;
+ unsigned long index;
+ const struct of_device_id *id;
+
+ if (!rate)
+ return 0;
+
+ id = of_match_device(rpmpd_performance_table, dev);
+ if (!id)
+ return -EINVAL;
+
+ index = (unsigned long)id->data;
+ for (i = 0; i < MAX_LEVEL; i++)
+ if (msm8996_rpmpd_freq_map[index].freq[i] >= rate)
+ return i;
+
+ return MAX_LEVEL;
+}
+
static int rpmpd_probe(struct platform_device *pdev)
{
int i;
@@ -267,6 +365,8 @@ static int rpmpd_probe(struct platform_device *pdev)
rpmpds[i]->rpm = rpm;
rpmpds[i]->pd.power_off = rpmpd_power_off;
rpmpds[i]->pd.power_on = rpmpd_power_on;
+ rpmpds[i]->pd.set_performance_state = rpmpd_set_performance;
+ rpmpds[i]->pd.get_performance_state = rpmpd_get_performance;
pm_genpd_init(&rpmpds[i]->pd, NULL, true);
data->domains[i] = &rpmpds[i]->pd;
--
2.13.0.71.gd7076ec9c9cb
next prev parent reply other threads:[~2017-06-21 7:11 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-21 7:10 [PATCH V8 0/6] PM / Domains: Power domain performance states Viresh Kumar
2017-06-21 7:10 ` [PATCH V8 1/6] PM / Domains: Add support to select performance-state of domains Viresh Kumar
2017-07-17 12:38 ` Ulf Hansson
2017-07-19 12:37 ` Viresh Kumar
2017-07-21 8:35 ` Ulf Hansson
2017-07-21 9:05 ` Viresh Kumar
2017-07-23 7:20 ` Ulf Hansson
2017-07-24 10:32 ` Viresh Kumar
2017-07-28 11:00 ` Viresh Kumar
2017-07-29 8:24 ` Ulf Hansson
2017-07-31 4:14 ` Viresh Kumar
2017-08-02 8:21 ` Viresh Kumar
2017-06-21 7:10 ` [PATCH V8 2/6] PM / OPP: Support updating performance state of device's power domains Viresh Kumar
2017-06-21 7:10 ` [NOT-FOR-MERGE V8 3/6] soc: qcom: rpmpd: Add a powerdomain driver to model cx/mx powerdomains Viresh Kumar
2017-06-21 7:10 ` Viresh Kumar [this message]
2017-06-21 7:10 ` [NOT-FOR-MERGE V8 5/6] mmc: sdhci-msm: Adapt the driver to use OPPs to set clocks/performance state Viresh Kumar
2017-06-21 7:10 ` [NOT-FOR-MERGE V8 6/6] remoteproc: qcom: q6v5: Vote for proxy powerdomain performance state Viresh Kumar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f880d3e38cdbf07d7f4cbb26453be160b33c8635.1498026827.git.viresh.kumar@linaro.org \
--to=viresh.kumar@linaro.org \
--cc=andy.gross@linaro.org \
--cc=david.brown@linaro.org \
--cc=khilman@kernel.org \
--cc=len.brown@intel.com \
--cc=lina.iyer@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=nm@ti.com \
--cc=pavel@ucw.cz \
--cc=rjw@rjwysocki.net \
--cc=rnayak@codeaurora.org \
--cc=robh+dt@kernel.org \
--cc=sboyd@codeaurora.org \
--cc=sudeep.holla@arm.com \
--cc=ulf.hansson@linaro.org \
--cc=vincent.guittot@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).