[PATCH RFC v2 05/12] soc: samsung: pm_domain: Use unique compatible name for power domain

From: Amit Daniel Kachhap
Date: Mon Nov 24 2014 - 08:15:40 EST


This patch adds support for second optional compatible complate name. If
this compatible name is present then this name will be used to create
the Power Domain and not the DT node name. This will be useful to carry
out any power domain specific changes in the PD on/off handlers for new
SoCs.
This change will not affect the existing DT bindings.

Cc: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
Signed-off-by: Amit Daniel Kachhap <amit.daniel@xxxxxxxxxxx>
---
.../bindings/arm/exynos/power_domain.txt | 8 ++++++++
drivers/soc/samsung/pm_domains.c | 10 +++++++++-
2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/arm/exynos/power_domain.txt b/Documentation/devicetree/bindings/arm/exynos/power_domain.txt
index abde1ea..00ebda1 100644
--- a/Documentation/devicetree/bindings/arm/exynos/power_domain.txt
+++ b/Documentation/devicetree/bindings/arm/exynos/power_domain.txt
@@ -12,6 +12,8 @@ Required Properties:
must be 0.

Optional Properties:
+- compatible: This is a second compatible name and gives the complete Power
+ Domain name like "samsung,exynos7-pd-mfc".
- clocks: List of clock handles. The parent clocks of the input clocks to the
devices in this power domain are set to oscclk before power gating
and restored back after powering on a domain. This is required for
@@ -43,5 +45,11 @@ Example:
#power-domain-cells = <0>;
};

+ mfc_pd: power-domain@10044060 {
+ compatible = "samsung,exynos4210-pd", "samsung,exynos7-pd-mfc";
+ reg = <0x10044060 0x20>;
+ #power-domain-cells = <0>;
+ };
+
See Documentation/devicetree/bindings/power/power_domain.txt for description
of consumer-side bindings.
diff --git a/drivers/soc/samsung/pm_domains.c b/drivers/soc/samsung/pm_domains.c
index 69bc8b1..f4f2a6c 100644
--- a/drivers/soc/samsung/pm_domains.c
+++ b/drivers/soc/samsung/pm_domains.c
@@ -120,12 +120,20 @@ static int exynos_power_domain_probe(struct platform_device *pdev)
struct exynos_pm_domain *pd;
int i;
bool on;
+ const char *name;

pd = devm_kzalloc(&pdev->dev, sizeof(*pd), GFP_KERNEL);
if (!pd)
return -ENOMEM;

- pd->pd.name = kstrdup(np->name, GFP_KERNEL);
+ /*
+ * Check if second compatible name found then use as power
+ * domain name or else use the DT node name as before.
+ */
+ if (of_property_read_string_index(np, "compatible", 1, &name))
+ name = np->name;
+
+ pd->pd.name = kstrdup(name, GFP_KERNEL);
pd->name = pd->pd.name;
pd->base = of_iomap(np, 0);
pd->pd.power_off = exynos_pd_power_off;
--
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/