Skip to content

Commit 6c87b3a

Browse files
Lewis-Petergregkh
authored andcommitted
clk: qcom: dispcc-sc8280xp: remove CLK_SET_RATE_PARENT from byte_div_clk_src dividers
[ Upstream commit 0b151a6 ] The four byte_div_clk_src dividers (disp{0,1}_cc_mdss_byte{0,1}_div_clk_src) had CLK_SET_RATE_PARENT set. When the DSI driver calls clk_set_rate() on byte_intf_clk, the rate-change propagates through the divider up to the parent PLL (byte_clk_src), halving the byte clock rate. A simiar issue had been also encountered on SM8750. b8501fe ("clk: qcom: dispcc-sm8750: Drop incorrect CLK_SET_RATE_PARENT on byte intf parent"). Likewise, remove CLK_SET_RATE_PARENT from all four byte divider clocks so that clk_set_rate() on the divider adjusts only the divider ratio, leaving the parent PLL untouched. Fixes: 4a66e76 ("clk: qcom: Add SC8280XP display clock controller") Signed-off-by: White Lewis <liu224806@gmail.com> [pengyu: reword] Signed-off-by: Pengyu Luo <mitltlatltl@gmail.com> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Link: https://lore.kernel.org/r/20260303115550.9279-1-mitltlatltl@gmail.com Signed-off-by: Bjorn Andersson <andersson@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
1 parent 1afd963 commit 6c87b3a

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

drivers/clk/qcom/dispcc-sc8280xp.c

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1160,7 +1160,6 @@ static struct clk_regmap_div disp0_cc_mdss_byte0_div_clk_src = {
11601160
&disp0_cc_mdss_byte0_clk_src.clkr.hw,
11611161
},
11621162
.num_parents = 1,
1163-
.flags = CLK_SET_RATE_PARENT,
11641163
.ops = &clk_regmap_div_ops,
11651164
},
11661165
};
@@ -1175,7 +1174,6 @@ static struct clk_regmap_div disp1_cc_mdss_byte0_div_clk_src = {
11751174
&disp1_cc_mdss_byte0_clk_src.clkr.hw,
11761175
},
11771176
.num_parents = 1,
1178-
.flags = CLK_SET_RATE_PARENT,
11791177
.ops = &clk_regmap_div_ops,
11801178
},
11811179
};
@@ -1190,7 +1188,6 @@ static struct clk_regmap_div disp0_cc_mdss_byte1_div_clk_src = {
11901188
&disp0_cc_mdss_byte1_clk_src.clkr.hw,
11911189
},
11921190
.num_parents = 1,
1193-
.flags = CLK_SET_RATE_PARENT,
11941191
.ops = &clk_regmap_div_ops,
11951192
},
11961193
};
@@ -1205,7 +1202,6 @@ static struct clk_regmap_div disp1_cc_mdss_byte1_div_clk_src = {
12051202
&disp1_cc_mdss_byte1_clk_src.clkr.hw,
12061203
},
12071204
.num_parents = 1,
1208-
.flags = CLK_SET_RATE_PARENT,
12091205
.ops = &clk_regmap_div_ops,
12101206
},
12111207
};

0 commit comments

Comments
 (0)