在 2016年03月03日 16:03, Elaine Zhang 写道:
This patch adds support for making one power domain a sub-domain of
other domain. This is useful for modeling power dependences,
which needs to have more than one power domain enabled to be operational.
Signed-off-by: Elaine Zhang <zhangq...@rock-chips.com>
---
drivers/soc/rockchip/pm_domains.c | 64 +++++++++++++++++++++++++++++++++++++++
1 file changed, 64 insertions(+)
diff --git a/drivers/soc/rockchip/pm_domains.c
b/drivers/soc/rockchip/pm_domains.c
index 0465a06..1777e8e 100644
--- a/drivers/soc/rockchip/pm_domains.c
+++ b/drivers/soc/rockchip/pm_domains.c
@@ -370,6 +370,63 @@ static void rockchip_configure_pd_cnt(struct rockchip_pmu
*pmu,
regmap_write(pmu->regmap, domain_reg_offset + 4, count);
}
+static int rockchip_pm_add_subdomain(struct rockchip_pmu *pmu,
+ struct device_node *parent)
+{
[...]
+ child_domain = pmu->genpd_data.domains[idx];
+
+ if (pm_genpd_add_subdomain(parent_domain, child_domain)) {
+ dev_err(pmu->dev, "%s failed to add subdomain: %s\n",
+ parent_domain->name, child_domain->name);
+ goto err_out;
+ } else {
+ dev_info(pmu->dev, "%s add subdomain: %s\n",
+ parent_domain->name, child_domain->name);
+ }
s/dev_info/dev_dbg
CHECK: Alignment should match open parenthesis
#416: FILE: drivers/soc/rockchip/pm_domains.c:416:
+ dev_info(pmu->dev, "%s add subdomain: %s\n",
+ parent_domain->name, child_domain->name);
+
+ error = rockchip_pm_add_subdomain(pmu, np);
+ if (error < 0)
+ goto rm_sub_domain;
+ }
+ return 0;
+
+err_out:
+ of_node_put(parent);
+ of_node_put(np);
+ return -EINVAL;
+rm_sub_domain:
+ pm_genpd_remove_subdomain(parent_domain, child_domain);
+ return error;
+}
+
static int rockchip_pm_domain_probe(struct platform_device *pdev)
{
struct device *dev = &pdev->dev;
@@ -440,6 +497,13 @@ static int rockchip_pm_domain_probe(struct platform_device
*pdev)
of_node_put(node);
goto err_out;
}
+
+ error = rockchip_pm_add_subdomain(pmu, node);
+ if (error < 0) {
+ dev_err(dev, "failed to handle subdomain node %s: %d\n",
+ node->name, error);
+ goto err_out;
+ }
}
if (error) {
--
Thanks,
Caesar
