Skip to content

Commit 7a0716e

Browse files
valpackettgregkh
authored andcommitted
clk: qcom: dispcc-sm8250: Use shared ops on the mdss vsync clk
[ Upstream commit 8c522da ] mdss_gdsc can get stuck on boot due to RCGs being left on from last boot. As a fix, commit 01a0a6c ("clk: qcom: Park shared RCGs upon registration") introduced a callback to ensure the RCG is off upon init. However, the fix depends on all shared RCGs being marked as such in code. For SM8150/SC8180X/SM8250 the MDSS vsync clock was using regular ops, unlike the same clock in the SC7180 code. This was causing display to frequently fail to initialize after rebooting on the Surface Pro X. Fix by using shared ops for this clock. Fixes: 80a18f4 ("clk: qcom: Add display clock controller driver for SM8150 and SM8250") Signed-off-by: Val Packett <val@packett.cool> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Link: https://lore.kernel.org/r/20260312112321.370983-8-val@packett.cool Signed-off-by: Bjorn Andersson <andersson@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
1 parent 35d53b5 commit 7a0716e

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/clk/qcom/dispcc-sm8250.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -632,7 +632,7 @@ static struct clk_rcg2 disp_cc_mdss_vsync_clk_src = {
632632
.parent_data = disp_cc_parent_data_1,
633633
.num_parents = ARRAY_SIZE(disp_cc_parent_data_1),
634634
.flags = CLK_SET_RATE_PARENT,
635-
.ops = &clk_rcg2_ops,
635+
.ops = &clk_rcg2_shared_ops,
636636
},
637637
};
638638

0 commit comments

Comments
 (0)