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 (mailing list archive)
State Accepted
Delegated to: Stephen Boyd
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(+)

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.
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
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,
 };