Similar to how it works on other SoCs, the top frequency of the SDHCI2 core clock is generated by a separate PLL (peculiar design choice) that is not guaranteed to be enabled (why does the clock framework not handle this by default?). Add the CLK_OPS_PARENT_ENABLE flag to make sure we're not muxing the RCG input to a dormant source. Fixes: db0c944ee92b ("clk: qcom: Add clock driver for SM8450") Reported-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> --- drivers/clk/qcom/gcc-sm8450.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/qcom/gcc-sm8450.c b/drivers/clk/qcom/gcc-sm8450.c index e86c58bc5e48..9a1d48ff22bc 100644 --- a/drivers/clk/qcom/gcc-sm8450.c +++ b/drivers/clk/qcom/gcc-sm8450.c @@ -935,7 +935,7 @@ static struct clk_rcg2 gcc_sdcc2_apps_clk_src = { .name = "gcc_sdcc2_apps_clk_src", .parent_data = gcc_parent_data_7, .num_parents = ARRAY_SIZE(gcc_parent_data_7), - .flags = CLK_SET_RATE_PARENT, + .flags = CLK_SET_RATE_PARENT | CLK_OPS_PARENT_ENABLE, .ops = &clk_rcg2_floor_ops, }, }; --- base-commit: c0b832517f627ead3388c6f0c74e8ac10ad5774b change-id: 20240427-topic-8450sdc2-3fcfebe1e8ad Best regards, -- Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>