clk: mediatek: mt8183: Correct parent of CLK_INFRA_SSPM_32K_SELF
authorChen-Yu Tsai <wenst@chromium.org>
Mon, 19 Feb 2024 10:51:24 +0000 (18:51 +0800)
committerStephen Boyd <sboyd@kernel.org>
Thu, 22 Feb 2024 04:55:30 +0000 (20:55 -0800)
CLK_INFRA_SSPM_32K_SELF has the "f_f26m_ck" clock assigned as its parent.
This is inconsistent as the clock is part of a group that are all gates
without dividers, and this makes the kernel think it runs at 26 MHz.

After clarification from MediaTek engineers, the correct parent is
actually the system 32 KHz clock.

Fixes: 1eb8d61ac5c9 ("clk: mediatek: mt8183: Add back SSPM related clocks")
Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
Link: https://lore.kernel.org/r/20240219105125.956278-1-wenst@chromium.org
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/mediatek/clk-mt8183.c

index 6e23461a04559c725d41a2c10bf9dc911cb11782..934d5a15acfc5898e2ba8c79510d7350465b7de9 100644 (file)
@@ -790,7 +790,7 @@ static const struct mtk_gate infra_clks[] = {
        /* infra_sspm_26m_self is main clock in co-processor, should not be closed in Linux. */
        GATE_INFRA3_FLAGS(CLK_INFRA_SSPM_26M_SELF, "infra_sspm_26m_self", "f_f26m_ck", 3, CLK_IS_CRITICAL),
        /* infra_sspm_32k_self is main clock in co-processor, should not be closed in Linux. */
-       GATE_INFRA3_FLAGS(CLK_INFRA_SSPM_32K_SELF, "infra_sspm_32k_self", "f_f26m_ck", 4, CLK_IS_CRITICAL),
+       GATE_INFRA3_FLAGS(CLK_INFRA_SSPM_32K_SELF, "infra_sspm_32k_self", "clk32k", 4, CLK_IS_CRITICAL),
        GATE_INFRA3(CLK_INFRA_UFS_AXI, "infra_ufs_axi", "axi_sel", 5),
        GATE_INFRA3(CLK_INFRA_I2C6, "infra_i2c6", "i2c_sel", 6),
        GATE_INFRA3(CLK_INFRA_AP_MSDC0, "infra_ap_msdc0", "msdc50_hclk_sel", 7),