Skip to content

Commit 667f39b

Browse files
fltobebarino
authored andcommitted
clk: qcom: gcc: fix sm8150 GPU and NPU clocks
Fix the parents and set BRANCH_HALT_SKIP. From the downstream driver it should be a 500us delay and not skip, however this matches what was done for other clocks that had 500us delay in downstream. Fixes: f73a423 ("clk: qcom: gcc: Add GPU and NPU clocks for SM8150") Signed-off-by: Jonathan Marek <[email protected]> Tested-by: Dmitry Baryshkov <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Stephen Boyd <[email protected]>
1 parent ba937f5 commit 667f39b

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

drivers/clk/qcom/gcc-sm8150.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1617,6 +1617,7 @@ static struct clk_branch gcc_gpu_cfg_ahb_clk = {
16171617
};
16181618

16191619
static struct clk_branch gcc_gpu_gpll0_clk_src = {
1620+
.halt_check = BRANCH_HALT_SKIP,
16201621
.clkr = {
16211622
.enable_reg = 0x52004,
16221623
.enable_mask = BIT(15),
@@ -1632,13 +1633,14 @@ static struct clk_branch gcc_gpu_gpll0_clk_src = {
16321633
};
16331634

16341635
static struct clk_branch gcc_gpu_gpll0_div_clk_src = {
1636+
.halt_check = BRANCH_HALT_SKIP,
16351637
.clkr = {
16361638
.enable_reg = 0x52004,
16371639
.enable_mask = BIT(16),
16381640
.hw.init = &(struct clk_init_data){
16391641
.name = "gcc_gpu_gpll0_div_clk_src",
16401642
.parent_hws = (const struct clk_hw *[]){
1641-
&gcc_gpu_gpll0_clk_src.clkr.hw },
1643+
&gpll0_out_even.clkr.hw },
16421644
.num_parents = 1,
16431645
.flags = CLK_SET_RATE_PARENT,
16441646
.ops = &clk_branch2_ops,
@@ -1729,6 +1731,7 @@ static struct clk_branch gcc_npu_cfg_ahb_clk = {
17291731
};
17301732

17311733
static struct clk_branch gcc_npu_gpll0_clk_src = {
1734+
.halt_check = BRANCH_HALT_SKIP,
17321735
.clkr = {
17331736
.enable_reg = 0x52004,
17341737
.enable_mask = BIT(18),
@@ -1744,13 +1747,14 @@ static struct clk_branch gcc_npu_gpll0_clk_src = {
17441747
};
17451748

17461749
static struct clk_branch gcc_npu_gpll0_div_clk_src = {
1750+
.halt_check = BRANCH_HALT_SKIP,
17471751
.clkr = {
17481752
.enable_reg = 0x52004,
17491753
.enable_mask = BIT(19),
17501754
.hw.init = &(struct clk_init_data){
17511755
.name = "gcc_npu_gpll0_div_clk_src",
17521756
.parent_hws = (const struct clk_hw *[]){
1753-
&gcc_npu_gpll0_clk_src.clkr.hw },
1757+
&gpll0_out_even.clkr.hw },
17541758
.num_parents = 1,
17551759
.flags = CLK_SET_RATE_PARENT,
17561760
.ops = &clk_branch2_ops,

0 commit comments

Comments
 (0)