diff mbox

clk: qcom: add parent for venus core0 and core1 gdsc's

Message ID 1486653630-15552-1-git-send-email-stanimir.varbanov@linaro.org
State Accepted
Commit 4a43e35d19c1379c255a8abab3d6303ed382dcf1
Headers show

Commit Message

Stanimir Varbanov Feb. 9, 2017, 3:20 p.m. UTC
Make venus_gdsc parent of venus gdsc core0 and core1.

Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>

---
 drivers/clk/qcom/mmcc-msm8996.c | 2 ++
 1 file changed, 2 insertions(+)

-- 
2.7.4

Comments

Stephen Boyd Feb. 10, 2017, 5:20 p.m. UTC | #1
On 02/09, Stanimir Varbanov wrote:
> Make venus_gdsc parent of venus gdsc core0 and core1.

> 


Ok, but why? Do we need the venus_gdsc to be on for the
venus_core0_gdsc to work? More details in commit text please.


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
Stephen Boyd April 19, 2017, 4:55 p.m. UTC | #2
On 02/09, Stanimir Varbanov wrote:
> Make venus_gdsc parent of venus gdsc core0 and core1.

> 

> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>

> ---


Applied to clk-next

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
diff mbox

Patch

diff --git a/drivers/clk/qcom/mmcc-msm8996.c b/drivers/clk/qcom/mmcc-msm8996.c
index 9b97246287a7..d9aa886d260f 100644
--- a/drivers/clk/qcom/mmcc-msm8996.c
+++ b/drivers/clk/qcom/mmcc-msm8996.c
@@ -2944,6 +2944,7 @@  static struct gdsc venus_core0_gdsc = {
 	.pd = {
 		.name = "venus_core0",
 	},
+	.parent = &venus_gdsc.pd,
 	.pwrsts = PWRSTS_OFF_ON,
 	.flags = HW_CTRL,
 };
@@ -2955,6 +2956,7 @@  static struct gdsc venus_core1_gdsc = {
 	.pd = {
 		.name = "venus_core1",
 	},
+	.parent = &venus_gdsc.pd,
 	.pwrsts = PWRSTS_OFF_ON,
 	.flags = HW_CTRL,
 };