Skip to content

Commit a76ce61

Browse files
Vladimir Zapolskiyandersson
authored andcommitted
clk: qcom: camcc-sm6350: Specify Titan GDSC power domain as a parent to other
When a consumer turns on/off a power domain dependent on another power domain in hardware, the parent power domain shall be turned on/off by the power domain provider as well, and to get it the power domain hardware hierarchy shall be described in the CAMCC driver. Establish the power domain hierarchy with a Titan GDSC set as a parent of all other GDSC power domains provided by the SM6350 camera clock controller to enforce a correct sequence of enabling and disabling power domains by the consumers, this fixes the CAMCC as a supplier of power domains to CAMSS IP and its driver. Fixes: 80f5451 ("clk: qcom: Add camera clock controller driver for SM6350") Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Reviewed-by: Imran Shaik <imran.shaik@oss.qualcomm.com> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org> Link: https://lore.kernel.org/r/20251021234450.2271279-3-vladimir.zapolskiy@linaro.org Signed-off-by: Bjorn Andersson <andersson@kernel.org>
1 parent d8f1121 commit a76ce61

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

drivers/clk/qcom/camcc-sm6350.c

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1693,6 +1693,8 @@ static struct clk_branch camcc_sys_tmr_clk = {
16931693
},
16941694
};
16951695

1696+
static struct gdsc titan_top_gdsc;
1697+
16961698
static struct gdsc bps_gdsc = {
16971699
.gdscr = 0x6004,
16981700
.en_rest_wait_val = 0x2,
@@ -1702,6 +1704,7 @@ static struct gdsc bps_gdsc = {
17021704
.name = "bps_gdsc",
17031705
},
17041706
.pwrsts = PWRSTS_OFF_ON,
1707+
.parent = &titan_top_gdsc.pd,
17051708
.flags = VOTABLE,
17061709
};
17071710

@@ -1714,6 +1717,7 @@ static struct gdsc ipe_0_gdsc = {
17141717
.name = "ipe_0_gdsc",
17151718
},
17161719
.pwrsts = PWRSTS_OFF_ON,
1720+
.parent = &titan_top_gdsc.pd,
17171721
.flags = VOTABLE,
17181722
};
17191723

@@ -1726,6 +1730,7 @@ static struct gdsc ife_0_gdsc = {
17261730
.name = "ife_0_gdsc",
17271731
},
17281732
.pwrsts = PWRSTS_OFF_ON,
1733+
.parent = &titan_top_gdsc.pd,
17291734
};
17301735

17311736
static struct gdsc ife_1_gdsc = {
@@ -1737,6 +1742,7 @@ static struct gdsc ife_1_gdsc = {
17371742
.name = "ife_1_gdsc",
17381743
},
17391744
.pwrsts = PWRSTS_OFF_ON,
1745+
.parent = &titan_top_gdsc.pd,
17401746
};
17411747

17421748
static struct gdsc ife_2_gdsc = {
@@ -1748,6 +1754,7 @@ static struct gdsc ife_2_gdsc = {
17481754
.name = "ife_2_gdsc",
17491755
},
17501756
.pwrsts = PWRSTS_OFF_ON,
1757+
.parent = &titan_top_gdsc.pd,
17511758
};
17521759

17531760
static struct gdsc titan_top_gdsc = {

0 commit comments

Comments
 (0)