From: Maulik Shah <quic_mkshah@quicinc.com>
To: <bjorn.andersson@linaro.org>, <ulf.hansson@linaro.org>
Cc: <linux-arm-msm@vger.kernel.org>, <linux-pm@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <rafael@kernel.org>,
<daniel.lezcano@linaro.org>, <quic_lsrao@quicinc.com>,
<quic_rjendra@quicinc.com>, Maulik Shah <quic_mkshah@quicinc.com>
Subject: [PATCH v2 4/6] PM: domains: Store the closest hrtimer event of the domain CPUs
Date: Wed, 11 May 2022 18:46:54 +0530 [thread overview]
Message-ID: <1652275016-13423-5-git-send-email-quic_mkshah@quicinc.com> (raw)
In-Reply-To: <1652275016-13423-1-git-send-email-quic_mkshah@quicinc.com>
The arch timer can not wake up the Qualcomm Technologies, Inc. (QTI)
SoCs when the deepest CPUidle modes results in the SoC also to enter
the low power mode.
RSC is part of CPU subsystem and APSS rsc device is attached to cluster
power domain. RSC has to setup next hrtimer wakeup in CONTROL_TCS which
can wakeup the SoC from deepest low power states. The CONTROL_TCS does
this by writing next wakeup in always on domain timer when the SoC is
entering the low power state.
Add dev_pm_genpd_get_next_hrtimer() to get the genpd wakeup time.
Signed-off-by: Maulik Shah <quic_mkshah@quicinc.com>
---
drivers/base/power/domain.c | 24 ++++++++++++++++++++++++
drivers/base/power/domain_governor.c | 1 +
include/linux/pm_domain.h | 7 +++++++
3 files changed, 32 insertions(+)
diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
index 18cd796..f0d70d0 100644
--- a/drivers/base/power/domain.c
+++ b/drivers/base/power/domain.c
@@ -487,6 +487,29 @@ void dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next)
}
EXPORT_SYMBOL_GPL(dev_pm_genpd_set_next_wakeup);
+/**
+ * dev_pm_genpd_get_next_hrtimer - Return genpd domain next_hrtimer.
+ *
+ * @dev: Device to handle
+ *
+ * Returns the aggregated domain wakeup time for CPU PM domain
+ * when all the subdomains are off.
+ */
+ktime_t dev_pm_genpd_get_next_hrtimer(struct device *dev)
+{
+ struct generic_pm_domain *genpd;
+
+ genpd = dev_to_genpd_safe(dev);
+ if (!genpd)
+ return KTIME_MAX;
+
+ if (atomic_read(&genpd->sd_count) > 0)
+ return KTIME_MAX;
+
+ return genpd->next_hrtimer;
+}
+EXPORT_SYMBOL_GPL(dev_pm_genpd_get_next_hrtimer);
+
static int _genpd_power_on(struct generic_pm_domain *genpd, bool timed)
{
unsigned int state_idx = genpd->state_idx;
@@ -1998,6 +2021,7 @@ int pm_genpd_init(struct generic_pm_domain *genpd,
genpd->max_off_time_changed = true;
genpd->provider = NULL;
genpd->has_provider = false;
+ genpd->next_hrtimer = KTIME_MAX;
genpd->accounting_time = ktime_get_mono_fast_ns();
genpd->domain.ops.runtime_suspend = genpd_runtime_suspend;
genpd->domain.ops.runtime_resume = genpd_runtime_resume;
diff --git a/drivers/base/power/domain_governor.c b/drivers/base/power/domain_governor.c
index cd08c58..a4c7dd8 100644
--- a/drivers/base/power/domain_governor.c
+++ b/drivers/base/power/domain_governor.c
@@ -363,6 +363,7 @@ static bool cpu_power_down_ok(struct dev_pm_domain *pd)
domain_wakeup = next_hrtimer;
}
}
+ genpd->next_hrtimer = domain_wakeup;
/* The minimum idle duration is from now - until the next wakeup. */
idle_duration_ns = ktime_to_ns(ktime_sub(domain_wakeup, now));
diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
index 043d48e..6d9fb79 100644
--- a/include/linux/pm_domain.h
+++ b/include/linux/pm_domain.h
@@ -17,6 +17,7 @@
#include <linux/notifier.h>
#include <linux/spinlock.h>
#include <linux/cpumask.h>
+#include <linux/time64.h>
/*
* Flags to control the behaviour of a genpd.
@@ -136,6 +137,7 @@ struct generic_pm_domain {
struct gpd_dev_ops dev_ops;
s64 max_off_time_ns; /* Maximum allowed "suspended" time. */
ktime_t next_wakeup; /* Maintained by the domain governor */
+ ktime_t next_hrtimer; /* Next hrtimer for the CPU PM domain */
bool max_off_time_changed;
bool cached_power_down_ok;
bool cached_power_down_state_idx;
@@ -228,6 +230,7 @@ int dev_pm_genpd_set_performance_state(struct device *dev, unsigned int state);
int dev_pm_genpd_add_notifier(struct device *dev, struct notifier_block *nb);
int dev_pm_genpd_remove_notifier(struct device *dev);
void dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next);
+ktime_t dev_pm_genpd_get_next_hrtimer(struct device *dev);
extern struct dev_power_governor simple_qos_governor;
extern struct dev_power_governor pm_domain_always_on_gov;
@@ -289,6 +292,10 @@ static inline int dev_pm_genpd_remove_notifier(struct device *dev)
static inline void dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next)
{ }
+static inline ktime_t dev_pm_genpd_get_next_hrtimer(struct device *dev)
+{
+ return KTIME_MAX;
+}
#define simple_qos_governor (*(struct dev_power_governor *)(NULL))
#define pm_domain_always_on_gov (*(struct dev_power_governor *)(NULL))
#endif
--
2.7.4
next prev parent reply other threads:[~2022-05-11 13:17 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-11 13:16 [PATCH v2 0/6] Add APSS RSC to Cluster power domain Maulik Shah
2022-05-11 13:16 ` [PATCH v2 1/6] dt-bindings: soc: qcom: Update devicetree binding document for rpmh-rsc Maulik Shah
2022-05-17 14:21 ` Rob Herring
2022-05-19 13:01 ` Ulf Hansson
2022-05-11 13:16 ` [PATCH v2 2/6] soc: qcom: rpmh-rsc: Attach RSC to cluster PM domain Maulik Shah
2022-05-19 13:11 ` Ulf Hansson
2022-05-11 13:16 ` [PATCH v2 3/6] arm64: dts: qcom: Add power-domains property for apps_rsc Maulik Shah
2022-05-11 13:16 ` Maulik Shah [this message]
2022-05-19 12:56 ` [PATCH v2 4/6] PM: domains: Store the closest hrtimer event of the domain CPUs Ulf Hansson
2022-06-29 4:26 ` Bjorn Andersson
2022-05-11 13:16 ` [PATCH v2 5/6] soc: qcom: rpmh-rsc: Save base address of drv Maulik Shah
2022-05-11 13:16 ` [PATCH v2 6/6] soc: qcom: rpmh-rsc: Write CONTROL_TCS with next timer wakeup Maulik Shah
2022-05-20 9:39 ` Ulf Hansson
2022-06-08 9:36 ` Ulf Hansson
2022-07-04 16:09 ` [PATCH v2 0/6] Add APSS RSC to Cluster power domain Dmitry Baryshkov
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=1652275016-13423-5-git-send-email-quic_mkshah@quicinc.com \
--to=quic_mkshah@quicinc.com \
--cc=bjorn.andersson@linaro.org \
--cc=daniel.lezcano@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=quic_lsrao@quicinc.com \
--cc=quic_rjendra@quicinc.com \
--cc=rafael@kernel.org \
--cc=ulf.hansson@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
all inboxes | Powered by JetHome®