Skip to content

Commit 9586e27

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 9676ca7 commit 9586e27

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
@@ -1161,7 +1161,6 @@ static struct clk_regmap_div disp0_cc_mdss_byte0_div_clk_src = {
11611161
&disp0_cc_mdss_byte0_clk_src.clkr.hw,
11621162
},
11631163
.num_parents = 1,
1164-
.flags = CLK_SET_RATE_PARENT,
11651164
.ops = &clk_regmap_div_ops,
11661165
},
11671166
};
@@ -1176,7 +1175,6 @@ static struct clk_regmap_div disp1_cc_mdss_byte0_div_clk_src = {
11761175
&disp1_cc_mdss_byte0_clk_src.clkr.hw,
11771176
},
11781177
.num_parents = 1,
1179-
.flags = CLK_SET_RATE_PARENT,
11801178
.ops = &clk_regmap_div_ops,
11811179
},
11821180
};
@@ -1191,7 +1189,6 @@ static struct clk_regmap_div disp0_cc_mdss_byte1_div_clk_src = {
11911189
&disp0_cc_mdss_byte1_clk_src.clkr.hw,
11921190
},
11931191
.num_parents = 1,
1194-
.flags = CLK_SET_RATE_PARENT,
11951192
.ops = &clk_regmap_div_ops,
11961193
},
11971194
};
@@ -1206,7 +1203,6 @@ static struct clk_regmap_div disp1_cc_mdss_byte1_div_clk_src = {
12061203
&disp1_cc_mdss_byte1_clk_src.clkr.hw,
12071204
},
12081205
.num_parents = 1,
1209-
.flags = CLK_SET_RATE_PARENT,
12101206
.ops = &clk_regmap_div_ops,
12111207
},
12121208
};

0 commit comments

Comments
 (0)