* [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM
@ 2021-10-21 20:31 Sam Protsenko
2021-10-21 20:31 ` [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain Sam Protsenko
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Sam Protsenko @ 2021-10-21 20:31 UTC (permalink / raw)
To: Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel,
Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd,
Michael Turquette
Cc: Sumit Semwal, devicetree, linux-arm-kernel, linux-clk,
linux-kernel, linux-samsung-soc
CMU_APM generates clocks for APM IP-core (Active Power Management). In
particular it generates RTC clocks, which are needed to enable rtc-s3c
driver on Exynos850 SoC.
Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
---
.../clock/samsung,exynos850-clock.yaml | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml b/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml
index 7f8c91a29b91..5618cfa62f80 100644
--- a/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml
@@ -32,6 +32,7 @@ properties:
compatible:
enum:
- samsung,exynos850-cmu-top
+ - samsung,exynos850-cmu-apm
- samsung,exynos850-cmu-core
- samsung,exynos850-cmu-dpu
- samsung,exynos850-cmu-hsi
@@ -68,6 +69,24 @@ allOf:
items:
- const: oscclk
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: samsung,exynos850-cmu-apm
+
+ then:
+ properties:
+ clocks:
+ items:
+ - description: External reference clock (26 MHz)
+ - description: CMU_APM bus clock (from CMU_TOP)
+
+ clock-names:
+ items:
+ - const: oscclk
+ - const: dout_clkcmu_apm_bus
+
- if:
properties:
compatible:
--
2.30.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-21 20:31 [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Sam Protsenko @ 2021-10-21 20:31 ` Sam Protsenko 2021-10-22 8:54 ` Krzysztof Kozlowski 2021-10-22 8:58 ` Chanwoo Choi 2021-10-22 8:50 ` [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Krzysztof Kozlowski 2021-10-22 22:02 ` Chanwoo Choi 2 siblings, 2 replies; 10+ messages in thread From: Sam Protsenko @ 2021-10-21 20:31 UTC (permalink / raw) To: Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette Cc: Sumit Semwal, devicetree, linux-arm-kernel, linux-clk, linux-kernel, linux-samsung-soc CMU_APM clock domain provides clocks for APM IP-core (Active Power Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. This patch adds next clocks: - bus clocks in CMU_TOP needed for CMU_APM - all internal CMU_APM clocks - leaf clocks for I3C, Speedy and RTC IP-cores - bus clocks for CMU_CMGP and CMU_CHUB CMU_APM doesn't belong to Power Domains, but platform driver is used for its registration to keep its bus clock always running. Otherwise rtc-s3c driver disables that clock and system freezes. Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> --- drivers/clk/samsung/clk-exynos850.c | 142 +++++++++++++++++++++++++- include/dt-bindings/clock/exynos850.h | 107 +++++++++++-------- 2 files changed, 208 insertions(+), 41 deletions(-) diff --git a/drivers/clk/samsung/clk-exynos850.c b/drivers/clk/samsung/clk-exynos850.c index 2294989e244c..95e373d17b42 100644 --- a/drivers/clk/samsung/clk-exynos850.c +++ b/drivers/clk/samsung/clk-exynos850.c @@ -72,6 +72,7 @@ static void __init exynos850_init_clocks(struct device_node *np, #define PLL_CON3_PLL_SHARED0 0x014c #define PLL_CON0_PLL_SHARED1 0x0180 #define PLL_CON3_PLL_SHARED1 0x018c +#define CLK_CON_MUX_MUX_CLKCMU_APM_BUS 0x1000 #define CLK_CON_MUX_MUX_CLKCMU_CORE_BUS 0x1014 #define CLK_CON_MUX_MUX_CLKCMU_CORE_CCI 0x1018 #define CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD 0x101c @@ -83,6 +84,7 @@ static void __init exynos850_init_clocks(struct device_node *np, #define CLK_CON_MUX_MUX_CLKCMU_PERI_BUS 0x1070 #define CLK_CON_MUX_MUX_CLKCMU_PERI_IP 0x1074 #define CLK_CON_MUX_MUX_CLKCMU_PERI_UART 0x1078 +#define CLK_CON_DIV_CLKCMU_APM_BUS 0x180c #define CLK_CON_DIV_CLKCMU_CORE_BUS 0x1820 #define CLK_CON_DIV_CLKCMU_CORE_CCI 0x1824 #define CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD 0x1828 @@ -100,6 +102,7 @@ static void __init exynos850_init_clocks(struct device_node *np, #define CLK_CON_DIV_PLL_SHARED1_DIV2 0x1898 #define CLK_CON_DIV_PLL_SHARED1_DIV3 0x189c #define CLK_CON_DIV_PLL_SHARED1_DIV4 0x18a0 +#define CLK_CON_GAT_GATE_CLKCMU_APM_BUS 0x2008 #define CLK_CON_GAT_GATE_CLKCMU_CORE_BUS 0x201c #define CLK_CON_GAT_GATE_CLKCMU_CORE_CCI 0x2020 #define CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD 0x2024 @@ -122,6 +125,7 @@ static const unsigned long top_clk_regs[] __initconst = { PLL_CON3_PLL_SHARED0, PLL_CON0_PLL_SHARED1, PLL_CON3_PLL_SHARED1, + CLK_CON_MUX_MUX_CLKCMU_APM_BUS, CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, CLK_CON_MUX_MUX_CLKCMU_CORE_CCI, CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD, @@ -133,6 +137,7 @@ static const unsigned long top_clk_regs[] __initconst = { CLK_CON_MUX_MUX_CLKCMU_PERI_BUS, CLK_CON_MUX_MUX_CLKCMU_PERI_IP, CLK_CON_MUX_MUX_CLKCMU_PERI_UART, + CLK_CON_DIV_CLKCMU_APM_BUS, CLK_CON_DIV_CLKCMU_CORE_BUS, CLK_CON_DIV_CLKCMU_CORE_CCI, CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD, @@ -150,6 +155,7 @@ static const unsigned long top_clk_regs[] __initconst = { CLK_CON_DIV_PLL_SHARED1_DIV2, CLK_CON_DIV_PLL_SHARED1_DIV3, CLK_CON_DIV_PLL_SHARED1_DIV4, + CLK_CON_GAT_GATE_CLKCMU_APM_BUS, CLK_CON_GAT_GATE_CLKCMU_CORE_BUS, CLK_CON_GAT_GATE_CLKCMU_CORE_CCI, CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD, @@ -183,6 +189,8 @@ static const struct samsung_pll_clock top_pll_clks[] __initconst = { PNAME(mout_shared0_pll_p) = { "oscclk", "fout_shared0_pll" }; PNAME(mout_shared1_pll_p) = { "oscclk", "fout_shared1_pll" }; PNAME(mout_mmc_pll_p) = { "oscclk", "fout_mmc_pll" }; +/* List of parent clocks for Muxes in CMU_TOP: for CMU_APM */ +PNAME(mout_clkcmu_apm_bus_p) = { "dout_shared0_div4", "pll_shared1_div4" }; /* List of parent clocks for Muxes in CMU_TOP: for CMU_CORE */ PNAME(mout_core_bus_p) = { "dout_shared1_div2", "dout_shared0_div3", "dout_shared1_div3", "dout_shared0_div4" }; @@ -222,6 +230,10 @@ static const struct samsung_mux_clock top_mux_clks[] __initconst = { MUX(CLK_MOUT_MMC_PLL, "mout_mmc_pll", mout_mmc_pll_p, PLL_CON0_PLL_MMC, 4, 1), + /* APM */ + MUX(CLK_MOUT_CLKCMU_APM_BUS, "mout_clkcmu_apm_bus", + mout_clkcmu_apm_bus_p, CLK_CON_MUX_MUX_CLKCMU_APM_BUS, 0, 1), + /* CORE */ MUX(CLK_MOUT_CORE_BUS, "mout_core_bus", mout_core_bus_p, CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, 0, 2), @@ -268,6 +280,10 @@ static const struct samsung_div_clock top_div_clks[] __initconst = { DIV(CLK_DOUT_SHARED1_DIV4, "dout_shared1_div4", "dout_shared1_div2", CLK_CON_DIV_PLL_SHARED1_DIV4, 0, 1), + /* APM */ + DIV(CLK_DOUT_CLKCMU_APM_BUS, "dout_clkcmu_apm_bus", + "gout_clkcmu_apm_bus", CLK_CON_DIV_CLKCMU_APM_BUS, 0, 3), + /* CORE */ DIV(CLK_DOUT_CORE_BUS, "dout_core_bus", "gout_core_bus", CLK_CON_DIV_CLKCMU_CORE_BUS, 0, 4), @@ -310,6 +326,10 @@ static const struct samsung_gate_clock top_gate_clks[] __initconst = { GATE(CLK_GOUT_CORE_SSS, "gout_core_sss", "mout_core_sss", CLK_CON_GAT_GATE_CLKCMU_CORE_SSS, 21, 0, 0), + /* APM */ + GATE(CLK_GOUT_CLKCMU_APM_BUS, "gout_clkcmu_apm_bus", + "mout_clkcmu_apm_bus", CLK_CON_GAT_GATE_CLKCMU_APM_BUS, 21, 0, 0), + /* DPU */ GATE(CLK_GOUT_DPU, "gout_dpu", "mout_dpu", CLK_CON_GAT_GATE_CLKCMU_DPU, 21, 0, 0), @@ -354,6 +374,124 @@ static void __init exynos850_cmu_top_init(struct device_node *np) CLK_OF_DECLARE(exynos850_cmu_top, "samsung,exynos850-cmu-top", exynos850_cmu_top_init); +/* ---- CMU_APM ------------------------------------------------------------- */ + +/* Register Offset definitions for CMU_APM (0x11800000) */ +#define PLL_CON0_MUX_CLKCMU_APM_BUS_USER 0x0600 +#define PLL_CON0_MUX_CLK_RCO_APM_I3C_USER 0x0610 +#define PLL_CON0_MUX_CLK_RCO_APM_USER 0x0620 +#define PLL_CON0_MUX_DLL_USER 0x0630 +#define CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS 0x1000 +#define CLK_CON_MUX_MUX_CLK_APM_BUS 0x1004 +#define CLK_CON_MUX_MUX_CLK_APM_I3C 0x1008 +#define CLK_CON_DIV_CLKCMU_CHUB_BUS 0x1800 +#define CLK_CON_DIV_DIV_CLK_APM_BUS 0x1804 +#define CLK_CON_DIV_DIV_CLK_APM_I3C 0x1808 +#define CLK_CON_GAT_CLKCMU_CMGP_BUS 0x2000 +#define CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS 0x2014 +#define CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK 0x2024 +#define CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK 0x2028 +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK 0x2034 +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK 0x2038 +#define CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK 0x20bc + +static const unsigned long apm_clk_regs[] __initconst = { + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, + PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, + PLL_CON0_MUX_CLK_RCO_APM_USER, + PLL_CON0_MUX_DLL_USER, + CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, + CLK_CON_MUX_MUX_CLK_APM_BUS, + CLK_CON_MUX_MUX_CLK_APM_I3C, + CLK_CON_DIV_CLKCMU_CHUB_BUS, + CLK_CON_DIV_DIV_CLK_APM_BUS, + CLK_CON_DIV_DIV_CLK_APM_I3C, + CLK_CON_GAT_CLKCMU_CMGP_BUS, + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, +}; + +/* List of parent clocks for Muxes in CMU_APM */ +PNAME(mout_apm_bus_user_p) = { "oscclk_rco_apm", "dout_clkcmu_apm_bus" }; +PNAME(mout_rco_apm_i3c_user_p) = { "oscclk_rco_apm", "clk_rco_i3c_pmic" }; +PNAME(mout_rco_apm_user_p) = { "oscclk_rco_apm", "clk_rco_apm__alv" }; +PNAME(mout_dll_user_p) = { "oscclk_rco_apm", "clk_dll_dco" }; +PNAME(mout_clkcmu_chub_bus_p) = { "mout_apm_bus_user", "mout_dll_user" }; +PNAME(mout_apm_bus_p) = { "mout_rco_apm_user", "mout_apm_bus_user", + "mout_dll_user", "oscclk_rco_apm" }; +PNAME(mout_apm_i3c_p) = { "dout_apm_i3c", "mout_rco_apm_i3c_user" }; + +static const struct samsung_fixed_rate_clock apm_fixed_clks[] __initconst = { + FRATE(CLK_RCO_I3C_PMIC, "clk_rco_i3c_pmic", NULL, 0, 491520000), + FRATE(OSCCLK_RCO_APM, "oscclk_rco_apm", NULL, 0, 24576000), + FRATE(CLK_RCO_APM__ALV, "clk_rco_apm__alv", NULL, 0, 49152000), + FRATE(CLK_DLL_DCO, "clk_dll_dco", NULL, 0, 360000000), +}; + +static const struct samsung_mux_clock apm_mux_clks[] __initconst = { + MUX(CLK_MOUT_APM_BUS_USER, "mout_apm_bus_user", mout_apm_bus_user_p, + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, 4, 1), + MUX(CLK_MOUT_RCO_APM_I3C_USER, "mout_rco_apm_i3c_user", + mout_rco_apm_i3c_user_p, PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, 4, 1), + MUX(CLK_MOUT_RCO_APM_USER, "mout_rco_apm_user", mout_rco_apm_user_p, + PLL_CON0_MUX_CLK_RCO_APM_USER, 4, 1), + MUX(CLK_MOUT_DLL_USER, "mout_dll_user", mout_dll_user_p, + PLL_CON0_MUX_DLL_USER, 4, 1), + MUX(CLK_MOUT_CLKCMU_CHUB_BUS, "mout_clkcmu_chub_bus", + mout_clkcmu_chub_bus_p, CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, 0, 1), + MUX(CLK_MOUT_APM_BUS, "mout_apm_bus", mout_apm_bus_p, + CLK_CON_MUX_MUX_CLK_APM_BUS, 0, 2), + MUX(CLK_MOUT_APM_I3C, "mout_apm_i3c", mout_apm_i3c_p, + CLK_CON_MUX_MUX_CLK_APM_I3C, 0, 1), +}; + +static const struct samsung_div_clock apm_div_clks[] __initconst = { + DIV(CLK_DOUT_CLKCMU_CHUB_BUS, "dout_clkcmu_chub_bus", + "gout_clkcmu_chub_bus", + CLK_CON_DIV_CLKCMU_CHUB_BUS, 0, 3), + DIV(CLK_DOUT_APM_BUS, "dout_apm_bus", "mout_apm_bus", + CLK_CON_DIV_DIV_CLK_APM_BUS, 0, 3), + DIV(CLK_DOUT_APM_I3C, "dout_apm_i3c", "mout_apm_bus", + CLK_CON_DIV_DIV_CLK_APM_I3C, 0, 3), +}; + +static const struct samsung_gate_clock apm_gate_clks[] __initconst = { + GATE(CLK_GOUT_CLKCMU_CMGP_BUS, "gout_clkcmu_cmgp_bus", "dout_apm_bus", + CLK_CON_GAT_CLKCMU_CMGP_BUS, 21, 0, 0), + GATE(CLK_GOUT_CLKCMU_CHUB_BUS, "gout_clkcmu_chub_bus", + "mout_clkcmu_chub_bus", + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, 21, 0, 0), + GATE(CLK_GOUT_RTC_PCLK, "gout_rtc_pclk", "dout_apm_bus", + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, 21, 0, 0), + GATE(CLK_GOUT_TOP_RTC_PCLK, "gout_top_rtc_pclk", "dout_apm_bus", + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, 21, 0, 0), + GATE(CLK_GOUT_I3C_PCLK, "gout_i3c_pclk", "dout_apm_bus", + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, 21, 0, 0), + GATE(CLK_GOUT_I3C_SCLK, "gout_i3c_sclk", "mout_apm_i3c", + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, 21, 0, 0), + GATE(CLK_GOUT_SPEEDY_PCLK, "gout_speedy_pclk", "dout_apm_bus", + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, 21, 0, 0), +}; + +static const struct samsung_cmu_info apm_cmu_info __initconst = { + .mux_clks = apm_mux_clks, + .nr_mux_clks = ARRAY_SIZE(apm_mux_clks), + .div_clks = apm_div_clks, + .nr_div_clks = ARRAY_SIZE(apm_div_clks), + .gate_clks = apm_gate_clks, + .nr_gate_clks = ARRAY_SIZE(apm_gate_clks), + .fixed_clks = apm_fixed_clks, + .nr_fixed_clks = ARRAY_SIZE(apm_fixed_clks), + .nr_clk_ids = APM_NR_CLK, + .clk_regs = apm_clk_regs, + .nr_clk_regs = ARRAY_SIZE(apm_clk_regs), + .clk_name = "dout_clkcmu_apm_bus", +}; + /* ---- CMU_HSI ------------------------------------------------------------- */ /* Register Offset definitions for CMU_HSI (0x13400000) */ @@ -801,9 +939,11 @@ static int __init exynos850_cmu_probe(struct platform_device *pdev) return 0; } -/* CMUs which belong to Power Domains and need runtime PM to be implemented */ static const struct of_device_id exynos850_cmu_of_match[] = { { + .compatible = "samsung,exynos850-cmu-apm", + .data = &apm_cmu_info, + }, { .compatible = "samsung,exynos850-cmu-hsi", .data = &hsi_cmu_info, }, { diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h index 8999184f94a2..c65e0300a8d3 100644 --- a/include/dt-bindings/clock/exynos850.h +++ b/include/dt-bindings/clock/exynos850.h @@ -16,46 +16,73 @@ #define CLK_MOUT_SHARED0_PLL 4 #define CLK_MOUT_SHARED1_PLL 5 #define CLK_MOUT_MMC_PLL 6 -#define CLK_MOUT_CORE_BUS 7 -#define CLK_MOUT_CORE_CCI 8 -#define CLK_MOUT_CORE_MMC_EMBD 9 -#define CLK_MOUT_CORE_SSS 10 -#define CLK_MOUT_DPU 11 -#define CLK_MOUT_HSI_BUS 12 -#define CLK_MOUT_HSI_MMC_CARD 13 -#define CLK_MOUT_HSI_USB20DRD 14 -#define CLK_MOUT_PERI_BUS 15 -#define CLK_MOUT_PERI_UART 16 -#define CLK_MOUT_PERI_IP 17 -#define CLK_DOUT_SHARED0_DIV3 18 -#define CLK_DOUT_SHARED0_DIV2 19 -#define CLK_DOUT_SHARED1_DIV3 20 -#define CLK_DOUT_SHARED1_DIV2 21 -#define CLK_DOUT_SHARED0_DIV4 22 -#define CLK_DOUT_SHARED1_DIV4 23 -#define CLK_DOUT_CORE_BUS 24 -#define CLK_DOUT_CORE_CCI 25 -#define CLK_DOUT_CORE_MMC_EMBD 26 -#define CLK_DOUT_CORE_SSS 27 -#define CLK_DOUT_DPU 28 -#define CLK_DOUT_HSI_BUS 29 -#define CLK_DOUT_HSI_MMC_CARD 30 -#define CLK_DOUT_HSI_USB20DRD 31 -#define CLK_DOUT_PERI_BUS 32 -#define CLK_DOUT_PERI_UART 33 -#define CLK_DOUT_PERI_IP 34 -#define CLK_GOUT_CORE_BUS 35 -#define CLK_GOUT_CORE_CCI 36 -#define CLK_GOUT_CORE_MMC_EMBD 37 -#define CLK_GOUT_CORE_SSS 38 -#define CLK_GOUT_DPU 39 -#define CLK_GOUT_HSI_BUS 40 -#define CLK_GOUT_HSI_MMC_CARD 41 -#define CLK_GOUT_HSI_USB20DRD 42 -#define CLK_GOUT_PERI_BUS 43 -#define CLK_GOUT_PERI_UART 44 -#define CLK_GOUT_PERI_IP 45 -#define TOP_NR_CLK 46 +#define CLK_MOUT_CLKCMU_APM_BUS 7 +#define CLK_MOUT_CORE_BUS 8 +#define CLK_MOUT_CORE_CCI 9 +#define CLK_MOUT_CORE_MMC_EMBD 10 +#define CLK_MOUT_CORE_SSS 11 +#define CLK_MOUT_DPU 12 +#define CLK_MOUT_HSI_BUS 13 +#define CLK_MOUT_HSI_MMC_CARD 14 +#define CLK_MOUT_HSI_USB20DRD 15 +#define CLK_MOUT_PERI_BUS 16 +#define CLK_MOUT_PERI_UART 17 +#define CLK_MOUT_PERI_IP 18 +#define CLK_DOUT_SHARED0_DIV3 19 +#define CLK_DOUT_SHARED0_DIV2 20 +#define CLK_DOUT_SHARED1_DIV3 21 +#define CLK_DOUT_SHARED1_DIV2 22 +#define CLK_DOUT_SHARED0_DIV4 23 +#define CLK_DOUT_SHARED1_DIV4 24 +#define CLK_DOUT_CLKCMU_APM_BUS 25 +#define CLK_DOUT_CORE_BUS 26 +#define CLK_DOUT_CORE_CCI 27 +#define CLK_DOUT_CORE_MMC_EMBD 28 +#define CLK_DOUT_CORE_SSS 29 +#define CLK_DOUT_DPU 30 +#define CLK_DOUT_HSI_BUS 31 +#define CLK_DOUT_HSI_MMC_CARD 32 +#define CLK_DOUT_HSI_USB20DRD 33 +#define CLK_DOUT_PERI_BUS 34 +#define CLK_DOUT_PERI_UART 35 +#define CLK_DOUT_PERI_IP 36 +#define CLK_GOUT_CLKCMU_APM_BUS 37 +#define CLK_GOUT_CORE_BUS 38 +#define CLK_GOUT_CORE_CCI 39 +#define CLK_GOUT_CORE_MMC_EMBD 40 +#define CLK_GOUT_CORE_SSS 41 +#define CLK_GOUT_DPU 42 +#define CLK_GOUT_HSI_BUS 43 +#define CLK_GOUT_HSI_MMC_CARD 44 +#define CLK_GOUT_HSI_USB20DRD 45 +#define CLK_GOUT_PERI_BUS 46 +#define CLK_GOUT_PERI_UART 47 +#define CLK_GOUT_PERI_IP 48 +#define TOP_NR_CLK 49 + +/* CMU_APM */ +#define CLK_RCO_I3C_PMIC 1 +#define OSCCLK_RCO_APM 2 +#define CLK_RCO_APM__ALV 3 +#define CLK_DLL_DCO 4 +#define CLK_MOUT_APM_BUS_USER 5 +#define CLK_MOUT_RCO_APM_I3C_USER 6 +#define CLK_MOUT_RCO_APM_USER 7 +#define CLK_MOUT_DLL_USER 8 +#define CLK_MOUT_CLKCMU_CHUB_BUS 9 +#define CLK_MOUT_APM_BUS 10 +#define CLK_MOUT_APM_I3C 11 +#define CLK_DOUT_CLKCMU_CHUB_BUS 12 +#define CLK_DOUT_APM_BUS 13 +#define CLK_DOUT_APM_I3C 14 +#define CLK_GOUT_CLKCMU_CMGP_BUS 15 +#define CLK_GOUT_CLKCMU_CHUB_BUS 16 +#define CLK_GOUT_RTC_PCLK 17 +#define CLK_GOUT_TOP_RTC_PCLK 18 +#define CLK_GOUT_I3C_PCLK 19 +#define CLK_GOUT_I3C_SCLK 20 +#define CLK_GOUT_SPEEDY_PCLK 21 +#define APM_NR_CLK 22 /* CMU_HSI */ #define CLK_MOUT_HSI_BUS_USER 1 -- 2.30.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-21 20:31 ` [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain Sam Protsenko @ 2021-10-22 8:54 ` Krzysztof Kozlowski 2021-10-22 8:58 ` Chanwoo Choi 1 sibling, 0 replies; 10+ messages in thread From: Krzysztof Kozlowski @ 2021-10-22 8:54 UTC (permalink / raw) To: Sam Protsenko, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette Cc: Sumit Semwal, devicetree, linux-arm-kernel, linux-clk, linux-kernel, linux-samsung-soc On 21/10/2021 22:31, Sam Protsenko wrote: > CMU_APM clock domain provides clocks for APM IP-core (Active Power > Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, > Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. > > This patch adds next clocks: > - bus clocks in CMU_TOP needed for CMU_APM > - all internal CMU_APM clocks > - leaf clocks for I3C, Speedy and RTC IP-cores > - bus clocks for CMU_CMGP and CMU_CHUB > > CMU_APM doesn't belong to Power Domains, but platform driver is used for > its registration to keep its bus clock always running. Otherwise rtc-s3c > driver disables that clock and system freezes. > > Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > --- > drivers/clk/samsung/clk-exynos850.c | 142 +++++++++++++++++++++++++- > include/dt-bindings/clock/exynos850.h | 107 +++++++++++-------- > 2 files changed, 208 insertions(+), 41 deletions(-) > > diff --git a/drivers/clk/samsung/clk-exynos850.c b/drivers/clk/samsung/clk-exynos850.c > index 2294989e244c..95e373d17b42 100644 > --- a/drivers/clk/samsung/clk-exynos850.c > +++ b/drivers/clk/samsung/clk-exynos850.c > @@ -72,6 +72,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > #define PLL_CON3_PLL_SHARED0 0x014c > #define PLL_CON0_PLL_SHARED1 0x0180 > #define PLL_CON3_PLL_SHARED1 0x018c > +#define CLK_CON_MUX_MUX_CLKCMU_APM_BUS 0x1000 > #define CLK_CON_MUX_MUX_CLKCMU_CORE_BUS 0x1014 > #define CLK_CON_MUX_MUX_CLKCMU_CORE_CCI 0x1018 > #define CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD 0x101c > @@ -83,6 +84,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > #define CLK_CON_MUX_MUX_CLKCMU_PERI_BUS 0x1070 > #define CLK_CON_MUX_MUX_CLKCMU_PERI_IP 0x1074 > #define CLK_CON_MUX_MUX_CLKCMU_PERI_UART 0x1078 > +#define CLK_CON_DIV_CLKCMU_APM_BUS 0x180c > #define CLK_CON_DIV_CLKCMU_CORE_BUS 0x1820 > #define CLK_CON_DIV_CLKCMU_CORE_CCI 0x1824 > #define CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD 0x1828 > @@ -100,6 +102,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > #define CLK_CON_DIV_PLL_SHARED1_DIV2 0x1898 > #define CLK_CON_DIV_PLL_SHARED1_DIV3 0x189c > #define CLK_CON_DIV_PLL_SHARED1_DIV4 0x18a0 > +#define CLK_CON_GAT_GATE_CLKCMU_APM_BUS 0x2008 > #define CLK_CON_GAT_GATE_CLKCMU_CORE_BUS 0x201c > #define CLK_CON_GAT_GATE_CLKCMU_CORE_CCI 0x2020 > #define CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD 0x2024 > @@ -122,6 +125,7 @@ static const unsigned long top_clk_regs[] __initconst = { > PLL_CON3_PLL_SHARED0, > PLL_CON0_PLL_SHARED1, > PLL_CON3_PLL_SHARED1, > + CLK_CON_MUX_MUX_CLKCMU_APM_BUS, > CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, > CLK_CON_MUX_MUX_CLKCMU_CORE_CCI, > CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD, > @@ -133,6 +137,7 @@ static const unsigned long top_clk_regs[] __initconst = { > CLK_CON_MUX_MUX_CLKCMU_PERI_BUS, > CLK_CON_MUX_MUX_CLKCMU_PERI_IP, > CLK_CON_MUX_MUX_CLKCMU_PERI_UART, > + CLK_CON_DIV_CLKCMU_APM_BUS, > CLK_CON_DIV_CLKCMU_CORE_BUS, > CLK_CON_DIV_CLKCMU_CORE_CCI, > CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD, > @@ -150,6 +155,7 @@ static const unsigned long top_clk_regs[] __initconst = { > CLK_CON_DIV_PLL_SHARED1_DIV2, > CLK_CON_DIV_PLL_SHARED1_DIV3, > CLK_CON_DIV_PLL_SHARED1_DIV4, > + CLK_CON_GAT_GATE_CLKCMU_APM_BUS, > CLK_CON_GAT_GATE_CLKCMU_CORE_BUS, > CLK_CON_GAT_GATE_CLKCMU_CORE_CCI, > CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD, > @@ -183,6 +189,8 @@ static const struct samsung_pll_clock top_pll_clks[] __initconst = { > PNAME(mout_shared0_pll_p) = { "oscclk", "fout_shared0_pll" }; > PNAME(mout_shared1_pll_p) = { "oscclk", "fout_shared1_pll" }; > PNAME(mout_mmc_pll_p) = { "oscclk", "fout_mmc_pll" }; > +/* List of parent clocks for Muxes in CMU_TOP: for CMU_APM */ > +PNAME(mout_clkcmu_apm_bus_p) = { "dout_shared0_div4", "pll_shared1_div4" }; > /* List of parent clocks for Muxes in CMU_TOP: for CMU_CORE */ > PNAME(mout_core_bus_p) = { "dout_shared1_div2", "dout_shared0_div3", > "dout_shared1_div3", "dout_shared0_div4" }; > @@ -222,6 +230,10 @@ static const struct samsung_mux_clock top_mux_clks[] __initconst = { > MUX(CLK_MOUT_MMC_PLL, "mout_mmc_pll", mout_mmc_pll_p, > PLL_CON0_PLL_MMC, 4, 1), > > + /* APM */ > + MUX(CLK_MOUT_CLKCMU_APM_BUS, "mout_clkcmu_apm_bus", > + mout_clkcmu_apm_bus_p, CLK_CON_MUX_MUX_CLKCMU_APM_BUS, 0, 1), > + > /* CORE */ > MUX(CLK_MOUT_CORE_BUS, "mout_core_bus", mout_core_bus_p, > CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, 0, 2), > @@ -268,6 +280,10 @@ static const struct samsung_div_clock top_div_clks[] __initconst = { > DIV(CLK_DOUT_SHARED1_DIV4, "dout_shared1_div4", "dout_shared1_div2", > CLK_CON_DIV_PLL_SHARED1_DIV4, 0, 1), > > + /* APM */ > + DIV(CLK_DOUT_CLKCMU_APM_BUS, "dout_clkcmu_apm_bus", > + "gout_clkcmu_apm_bus", CLK_CON_DIV_CLKCMU_APM_BUS, 0, 3), > + > /* CORE */ > DIV(CLK_DOUT_CORE_BUS, "dout_core_bus", "gout_core_bus", > CLK_CON_DIV_CLKCMU_CORE_BUS, 0, 4), > @@ -310,6 +326,10 @@ static const struct samsung_gate_clock top_gate_clks[] __initconst = { > GATE(CLK_GOUT_CORE_SSS, "gout_core_sss", "mout_core_sss", > CLK_CON_GAT_GATE_CLKCMU_CORE_SSS, 21, 0, 0), > > + /* APM */ > + GATE(CLK_GOUT_CLKCMU_APM_BUS, "gout_clkcmu_apm_bus", > + "mout_clkcmu_apm_bus", CLK_CON_GAT_GATE_CLKCMU_APM_BUS, 21, 0, 0), > + > /* DPU */ > GATE(CLK_GOUT_DPU, "gout_dpu", "mout_dpu", > CLK_CON_GAT_GATE_CLKCMU_DPU, 21, 0, 0), > @@ -354,6 +374,124 @@ static void __init exynos850_cmu_top_init(struct device_node *np) > CLK_OF_DECLARE(exynos850_cmu_top, "samsung,exynos850-cmu-top", > exynos850_cmu_top_init); > > +/* ---- CMU_APM ------------------------------------------------------------- */ > + > +/* Register Offset definitions for CMU_APM (0x11800000) */ > +#define PLL_CON0_MUX_CLKCMU_APM_BUS_USER 0x0600 > +#define PLL_CON0_MUX_CLK_RCO_APM_I3C_USER 0x0610 > +#define PLL_CON0_MUX_CLK_RCO_APM_USER 0x0620 > +#define PLL_CON0_MUX_DLL_USER 0x0630 > +#define CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS 0x1000 > +#define CLK_CON_MUX_MUX_CLK_APM_BUS 0x1004 > +#define CLK_CON_MUX_MUX_CLK_APM_I3C 0x1008 > +#define CLK_CON_DIV_CLKCMU_CHUB_BUS 0x1800 > +#define CLK_CON_DIV_DIV_CLK_APM_BUS 0x1804 > +#define CLK_CON_DIV_DIV_CLK_APM_I3C 0x1808 > +#define CLK_CON_GAT_CLKCMU_CMGP_BUS 0x2000 > +#define CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS 0x2014 > +#define CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK 0x2024 > +#define CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK 0x2028 > +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK 0x2034 > +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK 0x2038 > +#define CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK 0x20bc > + > +static const unsigned long apm_clk_regs[] __initconst = { > + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, > + PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, > + PLL_CON0_MUX_CLK_RCO_APM_USER, > + PLL_CON0_MUX_DLL_USER, > + CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, > + CLK_CON_MUX_MUX_CLK_APM_BUS, > + CLK_CON_MUX_MUX_CLK_APM_I3C, > + CLK_CON_DIV_CLKCMU_CHUB_BUS, > + CLK_CON_DIV_DIV_CLK_APM_BUS, > + CLK_CON_DIV_DIV_CLK_APM_I3C, > + CLK_CON_GAT_CLKCMU_CMGP_BUS, > + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, > + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, > + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, > + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, > +}; > + > +/* List of parent clocks for Muxes in CMU_APM */ > +PNAME(mout_apm_bus_user_p) = { "oscclk_rco_apm", "dout_clkcmu_apm_bus" }; > +PNAME(mout_rco_apm_i3c_user_p) = { "oscclk_rco_apm", "clk_rco_i3c_pmic" }; > +PNAME(mout_rco_apm_user_p) = { "oscclk_rco_apm", "clk_rco_apm__alv" }; > +PNAME(mout_dll_user_p) = { "oscclk_rco_apm", "clk_dll_dco" }; > +PNAME(mout_clkcmu_chub_bus_p) = { "mout_apm_bus_user", "mout_dll_user" }; > +PNAME(mout_apm_bus_p) = { "mout_rco_apm_user", "mout_apm_bus_user", > + "mout_dll_user", "oscclk_rco_apm" }; > +PNAME(mout_apm_i3c_p) = { "dout_apm_i3c", "mout_rco_apm_i3c_user" }; > + > +static const struct samsung_fixed_rate_clock apm_fixed_clks[] __initconst = { > + FRATE(CLK_RCO_I3C_PMIC, "clk_rco_i3c_pmic", NULL, 0, 491520000), > + FRATE(OSCCLK_RCO_APM, "oscclk_rco_apm", NULL, 0, 24576000), > + FRATE(CLK_RCO_APM__ALV, "clk_rco_apm__alv", NULL, 0, 49152000), > + FRATE(CLK_DLL_DCO, "clk_dll_dco", NULL, 0, 360000000), > +}; > + > +static const struct samsung_mux_clock apm_mux_clks[] __initconst = { > + MUX(CLK_MOUT_APM_BUS_USER, "mout_apm_bus_user", mout_apm_bus_user_p, > + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, 4, 1), > + MUX(CLK_MOUT_RCO_APM_I3C_USER, "mout_rco_apm_i3c_user", > + mout_rco_apm_i3c_user_p, PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, 4, 1), > + MUX(CLK_MOUT_RCO_APM_USER, "mout_rco_apm_user", mout_rco_apm_user_p, > + PLL_CON0_MUX_CLK_RCO_APM_USER, 4, 1), > + MUX(CLK_MOUT_DLL_USER, "mout_dll_user", mout_dll_user_p, > + PLL_CON0_MUX_DLL_USER, 4, 1), > + MUX(CLK_MOUT_CLKCMU_CHUB_BUS, "mout_clkcmu_chub_bus", > + mout_clkcmu_chub_bus_p, CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, 0, 1), > + MUX(CLK_MOUT_APM_BUS, "mout_apm_bus", mout_apm_bus_p, > + CLK_CON_MUX_MUX_CLK_APM_BUS, 0, 2), > + MUX(CLK_MOUT_APM_I3C, "mout_apm_i3c", mout_apm_i3c_p, > + CLK_CON_MUX_MUX_CLK_APM_I3C, 0, 1), > +}; > + > +static const struct samsung_div_clock apm_div_clks[] __initconst = { > + DIV(CLK_DOUT_CLKCMU_CHUB_BUS, "dout_clkcmu_chub_bus", > + "gout_clkcmu_chub_bus", > + CLK_CON_DIV_CLKCMU_CHUB_BUS, 0, 3), > + DIV(CLK_DOUT_APM_BUS, "dout_apm_bus", "mout_apm_bus", > + CLK_CON_DIV_DIV_CLK_APM_BUS, 0, 3), > + DIV(CLK_DOUT_APM_I3C, "dout_apm_i3c", "mout_apm_bus", > + CLK_CON_DIV_DIV_CLK_APM_I3C, 0, 3), > +}; > + > +static const struct samsung_gate_clock apm_gate_clks[] __initconst = { > + GATE(CLK_GOUT_CLKCMU_CMGP_BUS, "gout_clkcmu_cmgp_bus", "dout_apm_bus", > + CLK_CON_GAT_CLKCMU_CMGP_BUS, 21, 0, 0), > + GATE(CLK_GOUT_CLKCMU_CHUB_BUS, "gout_clkcmu_chub_bus", > + "mout_clkcmu_chub_bus", > + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, 21, 0, 0), > + GATE(CLK_GOUT_RTC_PCLK, "gout_rtc_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, 21, 0, 0), > + GATE(CLK_GOUT_TOP_RTC_PCLK, "gout_top_rtc_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, 21, 0, 0), > + GATE(CLK_GOUT_I3C_PCLK, "gout_i3c_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, 21, 0, 0), > + GATE(CLK_GOUT_I3C_SCLK, "gout_i3c_sclk", "mout_apm_i3c", > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, 21, 0, 0), > + GATE(CLK_GOUT_SPEEDY_PCLK, "gout_speedy_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, 21, 0, 0), > +}; > + > +static const struct samsung_cmu_info apm_cmu_info __initconst = { > + .mux_clks = apm_mux_clks, > + .nr_mux_clks = ARRAY_SIZE(apm_mux_clks), > + .div_clks = apm_div_clks, > + .nr_div_clks = ARRAY_SIZE(apm_div_clks), > + .gate_clks = apm_gate_clks, > + .nr_gate_clks = ARRAY_SIZE(apm_gate_clks), > + .fixed_clks = apm_fixed_clks, > + .nr_fixed_clks = ARRAY_SIZE(apm_fixed_clks), > + .nr_clk_ids = APM_NR_CLK, > + .clk_regs = apm_clk_regs, > + .nr_clk_regs = ARRAY_SIZE(apm_clk_regs), > + .clk_name = "dout_clkcmu_apm_bus", > +}; > + > /* ---- CMU_HSI ------------------------------------------------------------- */ > > /* Register Offset definitions for CMU_HSI (0x13400000) */ > @@ -801,9 +939,11 @@ static int __init exynos850_cmu_probe(struct platform_device *pdev) > return 0; > } > > -/* CMUs which belong to Power Domains and need runtime PM to be implemented */ > static const struct of_device_id exynos850_cmu_of_match[] = { > { > + .compatible = "samsung,exynos850-cmu-apm", > + .data = &apm_cmu_info, > + }, { > .compatible = "samsung,exynos850-cmu-hsi", > .data = &hsi_cmu_info, > }, { > diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h > index 8999184f94a2..c65e0300a8d3 100644 > --- a/include/dt-bindings/clock/exynos850.h > +++ b/include/dt-bindings/clock/exynos850.h > @@ -16,46 +16,73 @@ > #define CLK_MOUT_SHARED0_PLL 4 > #define CLK_MOUT_SHARED1_PLL 5 > #define CLK_MOUT_MMC_PLL 6 > -#define CLK_MOUT_CORE_BUS 7 > -#define CLK_MOUT_CORE_CCI 8 > -#define CLK_MOUT_CORE_MMC_EMBD 9 > -#define CLK_MOUT_CORE_SSS 10 > -#define CLK_MOUT_DPU 11 > -#define CLK_MOUT_HSI_BUS 12 > -#define CLK_MOUT_HSI_MMC_CARD 13 > -#define CLK_MOUT_HSI_USB20DRD 14 > -#define CLK_MOUT_PERI_BUS 15 > -#define CLK_MOUT_PERI_UART 16 > -#define CLK_MOUT_PERI_IP 17 > -#define CLK_DOUT_SHARED0_DIV3 18 > -#define CLK_DOUT_SHARED0_DIV2 19 > -#define CLK_DOUT_SHARED1_DIV3 20 > -#define CLK_DOUT_SHARED1_DIV2 21 > -#define CLK_DOUT_SHARED0_DIV4 22 > -#define CLK_DOUT_SHARED1_DIV4 23 > -#define CLK_DOUT_CORE_BUS 24 > -#define CLK_DOUT_CORE_CCI 25 > -#define CLK_DOUT_CORE_MMC_EMBD 26 > -#define CLK_DOUT_CORE_SSS 27 > -#define CLK_DOUT_DPU 28 > -#define CLK_DOUT_HSI_BUS 29 > -#define CLK_DOUT_HSI_MMC_CARD 30 > -#define CLK_DOUT_HSI_USB20DRD 31 > -#define CLK_DOUT_PERI_BUS 32 > -#define CLK_DOUT_PERI_UART 33 > -#define CLK_DOUT_PERI_IP 34 > -#define CLK_GOUT_CORE_BUS 35 > -#define CLK_GOUT_CORE_CCI 36 > -#define CLK_GOUT_CORE_MMC_EMBD 37 > -#define CLK_GOUT_CORE_SSS 38 > -#define CLK_GOUT_DPU 39 > -#define CLK_GOUT_HSI_BUS 40 > -#define CLK_GOUT_HSI_MMC_CARD 41 > -#define CLK_GOUT_HSI_USB20DRD 42 > -#define CLK_GOUT_PERI_BUS 43 > -#define CLK_GOUT_PERI_UART 44 > -#define CLK_GOUT_PERI_IP 45 > -#define TOP_NR_CLK 46 > +#define CLK_MOUT_CLKCMU_APM_BUS 7 > +#define CLK_MOUT_CORE_BUS 8 > +#define CLK_MOUT_CORE_CCI 9 > +#define CLK_MOUT_CORE_MMC_EMBD 10 > +#define CLK_MOUT_CORE_SSS 11 > +#define CLK_MOUT_DPU 12 > +#define CLK_MOUT_HSI_BUS 13 > +#define CLK_MOUT_HSI_MMC_CARD 14 > +#define CLK_MOUT_HSI_USB20DRD 15 > +#define CLK_MOUT_PERI_BUS 16 > +#define CLK_MOUT_PERI_UART 17 > +#define CLK_MOUT_PERI_IP 18 > +#define CLK_DOUT_SHARED0_DIV3 19 > +#define CLK_DOUT_SHARED0_DIV2 20 > +#define CLK_DOUT_SHARED1_DIV3 21 > +#define CLK_DOUT_SHARED1_DIV2 22 > +#define CLK_DOUT_SHARED0_DIV4 23 > +#define CLK_DOUT_SHARED1_DIV4 24 > +#define CLK_DOUT_CLKCMU_APM_BUS 25 > +#define CLK_DOUT_CORE_BUS 26 > +#define CLK_DOUT_CORE_CCI 27 > +#define CLK_DOUT_CORE_MMC_EMBD 28 > +#define CLK_DOUT_CORE_SSS 29 > +#define CLK_DOUT_DPU 30 > +#define CLK_DOUT_HSI_BUS 31 > +#define CLK_DOUT_HSI_MMC_CARD 32 > +#define CLK_DOUT_HSI_USB20DRD 33 > +#define CLK_DOUT_PERI_BUS 34 > +#define CLK_DOUT_PERI_UART 35 > +#define CLK_DOUT_PERI_IP 36 > +#define CLK_GOUT_CLKCMU_APM_BUS 37 > +#define CLK_GOUT_CORE_BUS 38 > +#define CLK_GOUT_CORE_CCI 39 > +#define CLK_GOUT_CORE_MMC_EMBD 40 > +#define CLK_GOUT_CORE_SSS 41 > +#define CLK_GOUT_DPU 42 > +#define CLK_GOUT_HSI_BUS 43 > +#define CLK_GOUT_HSI_MMC_CARD 44 > +#define CLK_GOUT_HSI_USB20DRD 45 > +#define CLK_GOUT_PERI_BUS 46 > +#define CLK_GOUT_PERI_UART 47 > +#define CLK_GOUT_PERI_IP 48 > +#define TOP_NR_CLK 49 Everything is fine here, because there are no users of this, but just for the record: once your board stabilizes in mainline please avoid shuffling IDs because it would be an ABI break. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-21 20:31 ` [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain Sam Protsenko 2021-10-22 8:54 ` Krzysztof Kozlowski @ 2021-10-22 8:58 ` Chanwoo Choi 2021-10-22 13:39 ` Sam Protsenko 1 sibling, 1 reply; 10+ messages in thread From: Chanwoo Choi @ 2021-10-22 8:58 UTC (permalink / raw) To: Sam Protsenko, Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette Cc: Sumit Semwal, devicetree, linux-arm-kernel, linux-clk, linux-kernel, linux-samsung-soc On 21. 10. 22. 오전 5:31, Sam Protsenko wrote: > CMU_APM clock domain provides clocks for APM IP-core (Active Power > Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, > Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. > > This patch adds next clocks: > - bus clocks in CMU_TOP needed for CMU_APM > - all internal CMU_APM clocks > - leaf clocks for I3C, Speedy and RTC IP-cores > - bus clocks for CMU_CMGP and CMU_CHUB > > CMU_APM doesn't belong to Power Domains, but platform driver is used for > its registration to keep its bus clock always running. Otherwise rtc-s3c > driver disables that clock and system freezes. > > Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > --- > drivers/clk/samsung/clk-exynos850.c | 142 +++++++++++++++++++++++++- > include/dt-bindings/clock/exynos850.h | 107 +++++++++++-------- > 2 files changed, 208 insertions(+), 41 deletions(-) > > diff --git a/drivers/clk/samsung/clk-exynos850.c b/drivers/clk/samsung/clk-exynos850.c > index 2294989e244c..95e373d17b42 100644 > --- a/drivers/clk/samsung/clk-exynos850.c > +++ b/drivers/clk/samsung/clk-exynos850.c > @@ -72,6 +72,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > #define PLL_CON3_PLL_SHARED0 0x014c > #define PLL_CON0_PLL_SHARED1 0x0180 > #define PLL_CON3_PLL_SHARED1 0x018c > +#define CLK_CON_MUX_MUX_CLKCMU_APM_BUS 0x1000 > #define CLK_CON_MUX_MUX_CLKCMU_CORE_BUS 0x1014 > #define CLK_CON_MUX_MUX_CLKCMU_CORE_CCI 0x1018 > #define CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD 0x101c > @@ -83,6 +84,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > #define CLK_CON_MUX_MUX_CLKCMU_PERI_BUS 0x1070 > #define CLK_CON_MUX_MUX_CLKCMU_PERI_IP 0x1074 > #define CLK_CON_MUX_MUX_CLKCMU_PERI_UART 0x1078 > +#define CLK_CON_DIV_CLKCMU_APM_BUS 0x180c > #define CLK_CON_DIV_CLKCMU_CORE_BUS 0x1820 > #define CLK_CON_DIV_CLKCMU_CORE_CCI 0x1824 > #define CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD 0x1828 > @@ -100,6 +102,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > #define CLK_CON_DIV_PLL_SHARED1_DIV2 0x1898 > #define CLK_CON_DIV_PLL_SHARED1_DIV3 0x189c > #define CLK_CON_DIV_PLL_SHARED1_DIV4 0x18a0 > +#define CLK_CON_GAT_GATE_CLKCMU_APM_BUS 0x2008 > #define CLK_CON_GAT_GATE_CLKCMU_CORE_BUS 0x201c > #define CLK_CON_GAT_GATE_CLKCMU_CORE_CCI 0x2020 > #define CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD 0x2024 > @@ -122,6 +125,7 @@ static const unsigned long top_clk_regs[] __initconst = { > PLL_CON3_PLL_SHARED0, > PLL_CON0_PLL_SHARED1, > PLL_CON3_PLL_SHARED1, > + CLK_CON_MUX_MUX_CLKCMU_APM_BUS, > CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, > CLK_CON_MUX_MUX_CLKCMU_CORE_CCI, > CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD, > @@ -133,6 +137,7 @@ static const unsigned long top_clk_regs[] __initconst = { > CLK_CON_MUX_MUX_CLKCMU_PERI_BUS, > CLK_CON_MUX_MUX_CLKCMU_PERI_IP, > CLK_CON_MUX_MUX_CLKCMU_PERI_UART, > + CLK_CON_DIV_CLKCMU_APM_BUS, > CLK_CON_DIV_CLKCMU_CORE_BUS, > CLK_CON_DIV_CLKCMU_CORE_CCI, > CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD, > @@ -150,6 +155,7 @@ static const unsigned long top_clk_regs[] __initconst = { > CLK_CON_DIV_PLL_SHARED1_DIV2, > CLK_CON_DIV_PLL_SHARED1_DIV3, > CLK_CON_DIV_PLL_SHARED1_DIV4, > + CLK_CON_GAT_GATE_CLKCMU_APM_BUS, > CLK_CON_GAT_GATE_CLKCMU_CORE_BUS, > CLK_CON_GAT_GATE_CLKCMU_CORE_CCI, > CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD, > @@ -183,6 +189,8 @@ static const struct samsung_pll_clock top_pll_clks[] __initconst = { > PNAME(mout_shared0_pll_p) = { "oscclk", "fout_shared0_pll" }; > PNAME(mout_shared1_pll_p) = { "oscclk", "fout_shared1_pll" }; > PNAME(mout_mmc_pll_p) = { "oscclk", "fout_mmc_pll" }; > +/* List of parent clocks for Muxes in CMU_TOP: for CMU_APM */ > +PNAME(mout_clkcmu_apm_bus_p) = { "dout_shared0_div4", "pll_shared1_div4" }; > /* List of parent clocks for Muxes in CMU_TOP: for CMU_CORE */ > PNAME(mout_core_bus_p) = { "dout_shared1_div2", "dout_shared0_div3", > "dout_shared1_div3", "dout_shared0_div4" }; > @@ -222,6 +230,10 @@ static const struct samsung_mux_clock top_mux_clks[] __initconst = { > MUX(CLK_MOUT_MMC_PLL, "mout_mmc_pll", mout_mmc_pll_p, > PLL_CON0_PLL_MMC, 4, 1), > > + /* APM */ > + MUX(CLK_MOUT_CLKCMU_APM_BUS, "mout_clkcmu_apm_bus", > + mout_clkcmu_apm_bus_p, CLK_CON_MUX_MUX_CLKCMU_APM_BUS, 0, 1), > + > /* CORE */ > MUX(CLK_MOUT_CORE_BUS, "mout_core_bus", mout_core_bus_p, > CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, 0, 2), > @@ -268,6 +280,10 @@ static const struct samsung_div_clock top_div_clks[] __initconst = { > DIV(CLK_DOUT_SHARED1_DIV4, "dout_shared1_div4", "dout_shared1_div2", > CLK_CON_DIV_PLL_SHARED1_DIV4, 0, 1), > > + /* APM */ > + DIV(CLK_DOUT_CLKCMU_APM_BUS, "dout_clkcmu_apm_bus", > + "gout_clkcmu_apm_bus", CLK_CON_DIV_CLKCMU_APM_BUS, 0, 3), > + > /* CORE */ > DIV(CLK_DOUT_CORE_BUS, "dout_core_bus", "gout_core_bus", > CLK_CON_DIV_CLKCMU_CORE_BUS, 0, 4), > @@ -310,6 +326,10 @@ static const struct samsung_gate_clock top_gate_clks[] __initconst = { > GATE(CLK_GOUT_CORE_SSS, "gout_core_sss", "mout_core_sss", > CLK_CON_GAT_GATE_CLKCMU_CORE_SSS, 21, 0, 0), > > + /* APM */ > + GATE(CLK_GOUT_CLKCMU_APM_BUS, "gout_clkcmu_apm_bus", > + "mout_clkcmu_apm_bus", CLK_CON_GAT_GATE_CLKCMU_APM_BUS, 21, 0, 0), > + > /* DPU */ > GATE(CLK_GOUT_DPU, "gout_dpu", "mout_dpu", > CLK_CON_GAT_GATE_CLKCMU_DPU, 21, 0, 0), > @@ -354,6 +374,124 @@ static void __init exynos850_cmu_top_init(struct device_node *np) > CLK_OF_DECLARE(exynos850_cmu_top, "samsung,exynos850-cmu-top", > exynos850_cmu_top_init); > > +/* ---- CMU_APM ------------------------------------------------------------- */ > + > +/* Register Offset definitions for CMU_APM (0x11800000) */ > +#define PLL_CON0_MUX_CLKCMU_APM_BUS_USER 0x0600 > +#define PLL_CON0_MUX_CLK_RCO_APM_I3C_USER 0x0610 > +#define PLL_CON0_MUX_CLK_RCO_APM_USER 0x0620 > +#define PLL_CON0_MUX_DLL_USER 0x0630 > +#define CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS 0x1000 > +#define CLK_CON_MUX_MUX_CLK_APM_BUS 0x1004 > +#define CLK_CON_MUX_MUX_CLK_APM_I3C 0x1008 > +#define CLK_CON_DIV_CLKCMU_CHUB_BUS 0x1800 > +#define CLK_CON_DIV_DIV_CLK_APM_BUS 0x1804 > +#define CLK_CON_DIV_DIV_CLK_APM_I3C 0x1808 > +#define CLK_CON_GAT_CLKCMU_CMGP_BUS 0x2000 > +#define CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS 0x2014 > +#define CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK 0x2024 > +#define CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK 0x2028 > +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK 0x2034 > +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK 0x2038 > +#define CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK 0x20bc > + > +static const unsigned long apm_clk_regs[] __initconst = { > + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, > + PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, > + PLL_CON0_MUX_CLK_RCO_APM_USER, > + PLL_CON0_MUX_DLL_USER, > + CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, > + CLK_CON_MUX_MUX_CLK_APM_BUS, > + CLK_CON_MUX_MUX_CLK_APM_I3C, > + CLK_CON_DIV_CLKCMU_CHUB_BUS, > + CLK_CON_DIV_DIV_CLK_APM_BUS, > + CLK_CON_DIV_DIV_CLK_APM_I3C, > + CLK_CON_GAT_CLKCMU_CMGP_BUS, > + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, > + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, > + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, > + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, > +}; > + > +/* List of parent clocks for Muxes in CMU_APM */ > +PNAME(mout_apm_bus_user_p) = { "oscclk_rco_apm", "dout_clkcmu_apm_bus" }; > +PNAME(mout_rco_apm_i3c_user_p) = { "oscclk_rco_apm", "clk_rco_i3c_pmic" }; > +PNAME(mout_rco_apm_user_p) = { "oscclk_rco_apm", "clk_rco_apm__alv" }; > +PNAME(mout_dll_user_p) = { "oscclk_rco_apm", "clk_dll_dco" }; > +PNAME(mout_clkcmu_chub_bus_p) = { "mout_apm_bus_user", "mout_dll_user" }; > +PNAME(mout_apm_bus_p) = { "mout_rco_apm_user", "mout_apm_bus_user", > + "mout_dll_user", "oscclk_rco_apm" }; > +PNAME(mout_apm_i3c_p) = { "dout_apm_i3c", "mout_rco_apm_i3c_user" }; > + > +static const struct samsung_fixed_rate_clock apm_fixed_clks[] __initconst = { > + FRATE(CLK_RCO_I3C_PMIC, "clk_rco_i3c_pmic", NULL, 0, 491520000), > + FRATE(OSCCLK_RCO_APM, "oscclk_rco_apm", NULL, 0, 24576000), > + FRATE(CLK_RCO_APM__ALV, "clk_rco_apm__alv", NULL, 0, 49152000), > + FRATE(CLK_DLL_DCO, "clk_dll_dco", NULL, 0, 360000000), > +}; > + > +static const struct samsung_mux_clock apm_mux_clks[] __initconst = { > + MUX(CLK_MOUT_APM_BUS_USER, "mout_apm_bus_user", mout_apm_bus_user_p, > + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, 4, 1), > + MUX(CLK_MOUT_RCO_APM_I3C_USER, "mout_rco_apm_i3c_user", > + mout_rco_apm_i3c_user_p, PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, 4, 1), > + MUX(CLK_MOUT_RCO_APM_USER, "mout_rco_apm_user", mout_rco_apm_user_p, > + PLL_CON0_MUX_CLK_RCO_APM_USER, 4, 1), > + MUX(CLK_MOUT_DLL_USER, "mout_dll_user", mout_dll_user_p, > + PLL_CON0_MUX_DLL_USER, 4, 1), > + MUX(CLK_MOUT_CLKCMU_CHUB_BUS, "mout_clkcmu_chub_bus", > + mout_clkcmu_chub_bus_p, CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, 0, 1), > + MUX(CLK_MOUT_APM_BUS, "mout_apm_bus", mout_apm_bus_p, > + CLK_CON_MUX_MUX_CLK_APM_BUS, 0, 2), > + MUX(CLK_MOUT_APM_I3C, "mout_apm_i3c", mout_apm_i3c_p, > + CLK_CON_MUX_MUX_CLK_APM_I3C, 0, 1), > +}; > + > +static const struct samsung_div_clock apm_div_clks[] __initconst = { > + DIV(CLK_DOUT_CLKCMU_CHUB_BUS, "dout_clkcmu_chub_bus", > + "gout_clkcmu_chub_bus", > + CLK_CON_DIV_CLKCMU_CHUB_BUS, 0, 3), > + DIV(CLK_DOUT_APM_BUS, "dout_apm_bus", "mout_apm_bus", > + CLK_CON_DIV_DIV_CLK_APM_BUS, 0, 3), > + DIV(CLK_DOUT_APM_I3C, "dout_apm_i3c", "mout_apm_bus", > + CLK_CON_DIV_DIV_CLK_APM_I3C, 0, 3), > +}; > + > +static const struct samsung_gate_clock apm_gate_clks[] __initconst = { > + GATE(CLK_GOUT_CLKCMU_CMGP_BUS, "gout_clkcmu_cmgp_bus", "dout_apm_bus", > + CLK_CON_GAT_CLKCMU_CMGP_BUS, 21, 0, 0), > + GATE(CLK_GOUT_CLKCMU_CHUB_BUS, "gout_clkcmu_chub_bus", > + "mout_clkcmu_chub_bus", > + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, 21, 0, 0), > + GATE(CLK_GOUT_RTC_PCLK, "gout_rtc_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, 21, 0, 0), > + GATE(CLK_GOUT_TOP_RTC_PCLK, "gout_top_rtc_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, 21, 0, 0), > + GATE(CLK_GOUT_I3C_PCLK, "gout_i3c_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, 21, 0, 0), > + GATE(CLK_GOUT_I3C_SCLK, "gout_i3c_sclk", "mout_apm_i3c", > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, 21, 0, 0), > + GATE(CLK_GOUT_SPEEDY_PCLK, "gout_speedy_pclk", "dout_apm_bus", > + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, 21, 0, 0), > +}; > + > +static const struct samsung_cmu_info apm_cmu_info __initconst = { > + .mux_clks = apm_mux_clks, > + .nr_mux_clks = ARRAY_SIZE(apm_mux_clks), > + .div_clks = apm_div_clks, > + .nr_div_clks = ARRAY_SIZE(apm_div_clks), > + .gate_clks = apm_gate_clks, > + .nr_gate_clks = ARRAY_SIZE(apm_gate_clks), > + .fixed_clks = apm_fixed_clks, > + .nr_fixed_clks = ARRAY_SIZE(apm_fixed_clks), > + .nr_clk_ids = APM_NR_CLK, > + .clk_regs = apm_clk_regs, > + .nr_clk_regs = ARRAY_SIZE(apm_clk_regs), > + .clk_name = "dout_clkcmu_apm_bus", > +}; > + > /* ---- CMU_HSI ------------------------------------------------------------- */ > > /* Register Offset definitions for CMU_HSI (0x13400000) */ > @@ -801,9 +939,11 @@ static int __init exynos850_cmu_probe(struct platform_device *pdev) > return 0; > } > > -/* CMUs which belong to Power Domains and need runtime PM to be implemented */ > static const struct of_device_id exynos850_cmu_of_match[] = { > { > + .compatible = "samsung,exynos850-cmu-apm", > + .data = &apm_cmu_info, > + }, { > .compatible = "samsung,exynos850-cmu-hsi", > .data = &hsi_cmu_info, > }, { > diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h > index 8999184f94a2..c65e0300a8d3 100644 > --- a/include/dt-bindings/clock/exynos850.h > +++ b/include/dt-bindings/clock/exynos850.h > @@ -16,46 +16,73 @@ > #define CLK_MOUT_SHARED0_PLL 4 > #define CLK_MOUT_SHARED1_PLL 5 > #define CLK_MOUT_MMC_PLL 6 > -#define CLK_MOUT_CORE_BUS 7 > -#define CLK_MOUT_CORE_CCI 8 > -#define CLK_MOUT_CORE_MMC_EMBD 9 > -#define CLK_MOUT_CORE_SSS 10 > -#define CLK_MOUT_DPU 11 > -#define CLK_MOUT_HSI_BUS 12 > -#define CLK_MOUT_HSI_MMC_CARD 13 > -#define CLK_MOUT_HSI_USB20DRD 14 > -#define CLK_MOUT_PERI_BUS 15 > -#define CLK_MOUT_PERI_UART 16 > -#define CLK_MOUT_PERI_IP 17 > -#define CLK_DOUT_SHARED0_DIV3 18 > -#define CLK_DOUT_SHARED0_DIV2 19 > -#define CLK_DOUT_SHARED1_DIV3 20 > -#define CLK_DOUT_SHARED1_DIV2 21 > -#define CLK_DOUT_SHARED0_DIV4 22 > -#define CLK_DOUT_SHARED1_DIV4 23 > -#define CLK_DOUT_CORE_BUS 24 > -#define CLK_DOUT_CORE_CCI 25 > -#define CLK_DOUT_CORE_MMC_EMBD 26 > -#define CLK_DOUT_CORE_SSS 27 > -#define CLK_DOUT_DPU 28 > -#define CLK_DOUT_HSI_BUS 29 > -#define CLK_DOUT_HSI_MMC_CARD 30 > -#define CLK_DOUT_HSI_USB20DRD 31 > -#define CLK_DOUT_PERI_BUS 32 > -#define CLK_DOUT_PERI_UART 33 > -#define CLK_DOUT_PERI_IP 34 > -#define CLK_GOUT_CORE_BUS 35 > -#define CLK_GOUT_CORE_CCI 36 > -#define CLK_GOUT_CORE_MMC_EMBD 37 > -#define CLK_GOUT_CORE_SSS 38 > -#define CLK_GOUT_DPU 39 > -#define CLK_GOUT_HSI_BUS 40 > -#define CLK_GOUT_HSI_MMC_CARD 41 > -#define CLK_GOUT_HSI_USB20DRD 42 > -#define CLK_GOUT_PERI_BUS 43 > -#define CLK_GOUT_PERI_UART 44 > -#define CLK_GOUT_PERI_IP 45 > -#define TOP_NR_CLK 46 > +#define CLK_MOUT_CLKCMU_APM_BUS 7 > +#define CLK_MOUT_CORE_BUS 8 Basically, you can never change the already defined clock id in nclude/dt-bindings/clock/*.h because of supporting the compatibility of dtb files which were using the already defined clock id instead of changed clock id If you want to add new clock with new clock id, you have to define the new clock id at the end of defined clock like the next of CLK_GOUT_PERI_IP for TOP domain case. (snip) -- Best Regards, Samsung Electronics Chanwoo Choi ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-22 8:58 ` Chanwoo Choi @ 2021-10-22 13:39 ` Sam Protsenko 2021-10-22 14:08 ` Krzysztof Kozlowski 2021-10-22 22:18 ` Chanwoo Choi 0 siblings, 2 replies; 10+ messages in thread From: Sam Protsenko @ 2021-10-22 13:39 UTC (permalink / raw) To: Chanwoo Choi Cc: Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette, Sumit Semwal, devicetree, linux-arm Mailing List, linux-clk, Linux Kernel Mailing List, Linux Samsung SOC On Fri, 22 Oct 2021 at 11:58, Chanwoo Choi <cwchoi00@gmail.com> wrote: > > On 21. 10. 22. 오전 5:31, Sam Protsenko wrote: > > CMU_APM clock domain provides clocks for APM IP-core (Active Power > > Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, > > Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. > > > > This patch adds next clocks: > > - bus clocks in CMU_TOP needed for CMU_APM > > - all internal CMU_APM clocks > > - leaf clocks for I3C, Speedy and RTC IP-cores > > - bus clocks for CMU_CMGP and CMU_CHUB > > > > CMU_APM doesn't belong to Power Domains, but platform driver is used for > > its registration to keep its bus clock always running. Otherwise rtc-s3c > > driver disables that clock and system freezes. > > > > Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > > --- > > drivers/clk/samsung/clk-exynos850.c | 142 +++++++++++++++++++++++++- > > include/dt-bindings/clock/exynos850.h | 107 +++++++++++-------- > > 2 files changed, 208 insertions(+), 41 deletions(-) > > > > diff --git a/drivers/clk/samsung/clk-exynos850.c b/drivers/clk/samsung/clk-exynos850.c > > index 2294989e244c..95e373d17b42 100644 > > --- a/drivers/clk/samsung/clk-exynos850.c > > +++ b/drivers/clk/samsung/clk-exynos850.c > > @@ -72,6 +72,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > > #define PLL_CON3_PLL_SHARED0 0x014c > > #define PLL_CON0_PLL_SHARED1 0x0180 > > #define PLL_CON3_PLL_SHARED1 0x018c > > +#define CLK_CON_MUX_MUX_CLKCMU_APM_BUS 0x1000 > > #define CLK_CON_MUX_MUX_CLKCMU_CORE_BUS 0x1014 > > #define CLK_CON_MUX_MUX_CLKCMU_CORE_CCI 0x1018 > > #define CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD 0x101c > > @@ -83,6 +84,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > > #define CLK_CON_MUX_MUX_CLKCMU_PERI_BUS 0x1070 > > #define CLK_CON_MUX_MUX_CLKCMU_PERI_IP 0x1074 > > #define CLK_CON_MUX_MUX_CLKCMU_PERI_UART 0x1078 > > +#define CLK_CON_DIV_CLKCMU_APM_BUS 0x180c > > #define CLK_CON_DIV_CLKCMU_CORE_BUS 0x1820 > > #define CLK_CON_DIV_CLKCMU_CORE_CCI 0x1824 > > #define CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD 0x1828 > > @@ -100,6 +102,7 @@ static void __init exynos850_init_clocks(struct device_node *np, > > #define CLK_CON_DIV_PLL_SHARED1_DIV2 0x1898 > > #define CLK_CON_DIV_PLL_SHARED1_DIV3 0x189c > > #define CLK_CON_DIV_PLL_SHARED1_DIV4 0x18a0 > > +#define CLK_CON_GAT_GATE_CLKCMU_APM_BUS 0x2008 > > #define CLK_CON_GAT_GATE_CLKCMU_CORE_BUS 0x201c > > #define CLK_CON_GAT_GATE_CLKCMU_CORE_CCI 0x2020 > > #define CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD 0x2024 > > @@ -122,6 +125,7 @@ static const unsigned long top_clk_regs[] __initconst = { > > PLL_CON3_PLL_SHARED0, > > PLL_CON0_PLL_SHARED1, > > PLL_CON3_PLL_SHARED1, > > + CLK_CON_MUX_MUX_CLKCMU_APM_BUS, > > CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, > > CLK_CON_MUX_MUX_CLKCMU_CORE_CCI, > > CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD, > > @@ -133,6 +137,7 @@ static const unsigned long top_clk_regs[] __initconst = { > > CLK_CON_MUX_MUX_CLKCMU_PERI_BUS, > > CLK_CON_MUX_MUX_CLKCMU_PERI_IP, > > CLK_CON_MUX_MUX_CLKCMU_PERI_UART, > > + CLK_CON_DIV_CLKCMU_APM_BUS, > > CLK_CON_DIV_CLKCMU_CORE_BUS, > > CLK_CON_DIV_CLKCMU_CORE_CCI, > > CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD, > > @@ -150,6 +155,7 @@ static const unsigned long top_clk_regs[] __initconst = { > > CLK_CON_DIV_PLL_SHARED1_DIV2, > > CLK_CON_DIV_PLL_SHARED1_DIV3, > > CLK_CON_DIV_PLL_SHARED1_DIV4, > > + CLK_CON_GAT_GATE_CLKCMU_APM_BUS, > > CLK_CON_GAT_GATE_CLKCMU_CORE_BUS, > > CLK_CON_GAT_GATE_CLKCMU_CORE_CCI, > > CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD, > > @@ -183,6 +189,8 @@ static const struct samsung_pll_clock top_pll_clks[] __initconst = { > > PNAME(mout_shared0_pll_p) = { "oscclk", "fout_shared0_pll" }; > > PNAME(mout_shared1_pll_p) = { "oscclk", "fout_shared1_pll" }; > > PNAME(mout_mmc_pll_p) = { "oscclk", "fout_mmc_pll" }; > > +/* List of parent clocks for Muxes in CMU_TOP: for CMU_APM */ > > +PNAME(mout_clkcmu_apm_bus_p) = { "dout_shared0_div4", "pll_shared1_div4" }; > > /* List of parent clocks for Muxes in CMU_TOP: for CMU_CORE */ > > PNAME(mout_core_bus_p) = { "dout_shared1_div2", "dout_shared0_div3", > > "dout_shared1_div3", "dout_shared0_div4" }; > > @@ -222,6 +230,10 @@ static const struct samsung_mux_clock top_mux_clks[] __initconst = { > > MUX(CLK_MOUT_MMC_PLL, "mout_mmc_pll", mout_mmc_pll_p, > > PLL_CON0_PLL_MMC, 4, 1), > > > > + /* APM */ > > + MUX(CLK_MOUT_CLKCMU_APM_BUS, "mout_clkcmu_apm_bus", > > + mout_clkcmu_apm_bus_p, CLK_CON_MUX_MUX_CLKCMU_APM_BUS, 0, 1), > > + > > /* CORE */ > > MUX(CLK_MOUT_CORE_BUS, "mout_core_bus", mout_core_bus_p, > > CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, 0, 2), > > @@ -268,6 +280,10 @@ static const struct samsung_div_clock top_div_clks[] __initconst = { > > DIV(CLK_DOUT_SHARED1_DIV4, "dout_shared1_div4", "dout_shared1_div2", > > CLK_CON_DIV_PLL_SHARED1_DIV4, 0, 1), > > > > + /* APM */ > > + DIV(CLK_DOUT_CLKCMU_APM_BUS, "dout_clkcmu_apm_bus", > > + "gout_clkcmu_apm_bus", CLK_CON_DIV_CLKCMU_APM_BUS, 0, 3), > > + > > /* CORE */ > > DIV(CLK_DOUT_CORE_BUS, "dout_core_bus", "gout_core_bus", > > CLK_CON_DIV_CLKCMU_CORE_BUS, 0, 4), > > @@ -310,6 +326,10 @@ static const struct samsung_gate_clock top_gate_clks[] __initconst = { > > GATE(CLK_GOUT_CORE_SSS, "gout_core_sss", "mout_core_sss", > > CLK_CON_GAT_GATE_CLKCMU_CORE_SSS, 21, 0, 0), > > > > + /* APM */ > > + GATE(CLK_GOUT_CLKCMU_APM_BUS, "gout_clkcmu_apm_bus", > > + "mout_clkcmu_apm_bus", CLK_CON_GAT_GATE_CLKCMU_APM_BUS, 21, 0, 0), > > + > > /* DPU */ > > GATE(CLK_GOUT_DPU, "gout_dpu", "mout_dpu", > > CLK_CON_GAT_GATE_CLKCMU_DPU, 21, 0, 0), > > @@ -354,6 +374,124 @@ static void __init exynos850_cmu_top_init(struct device_node *np) > > CLK_OF_DECLARE(exynos850_cmu_top, "samsung,exynos850-cmu-top", > > exynos850_cmu_top_init); > > > > +/* ---- CMU_APM ------------------------------------------------------------- */ > > + > > +/* Register Offset definitions for CMU_APM (0x11800000) */ > > +#define PLL_CON0_MUX_CLKCMU_APM_BUS_USER 0x0600 > > +#define PLL_CON0_MUX_CLK_RCO_APM_I3C_USER 0x0610 > > +#define PLL_CON0_MUX_CLK_RCO_APM_USER 0x0620 > > +#define PLL_CON0_MUX_DLL_USER 0x0630 > > +#define CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS 0x1000 > > +#define CLK_CON_MUX_MUX_CLK_APM_BUS 0x1004 > > +#define CLK_CON_MUX_MUX_CLK_APM_I3C 0x1008 > > +#define CLK_CON_DIV_CLKCMU_CHUB_BUS 0x1800 > > +#define CLK_CON_DIV_DIV_CLK_APM_BUS 0x1804 > > +#define CLK_CON_DIV_DIV_CLK_APM_I3C 0x1808 > > +#define CLK_CON_GAT_CLKCMU_CMGP_BUS 0x2000 > > +#define CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS 0x2014 > > +#define CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK 0x2024 > > +#define CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK 0x2028 > > +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK 0x2034 > > +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK 0x2038 > > +#define CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK 0x20bc > > + > > +static const unsigned long apm_clk_regs[] __initconst = { > > + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, > > + PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, > > + PLL_CON0_MUX_CLK_RCO_APM_USER, > > + PLL_CON0_MUX_DLL_USER, > > + CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, > > + CLK_CON_MUX_MUX_CLK_APM_BUS, > > + CLK_CON_MUX_MUX_CLK_APM_I3C, > > + CLK_CON_DIV_CLKCMU_CHUB_BUS, > > + CLK_CON_DIV_DIV_CLK_APM_BUS, > > + CLK_CON_DIV_DIV_CLK_APM_I3C, > > + CLK_CON_GAT_CLKCMU_CMGP_BUS, > > + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, > > + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, > > + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, > > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, > > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, > > + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, > > +}; > > + > > +/* List of parent clocks for Muxes in CMU_APM */ > > +PNAME(mout_apm_bus_user_p) = { "oscclk_rco_apm", "dout_clkcmu_apm_bus" }; > > +PNAME(mout_rco_apm_i3c_user_p) = { "oscclk_rco_apm", "clk_rco_i3c_pmic" }; > > +PNAME(mout_rco_apm_user_p) = { "oscclk_rco_apm", "clk_rco_apm__alv" }; > > +PNAME(mout_dll_user_p) = { "oscclk_rco_apm", "clk_dll_dco" }; > > +PNAME(mout_clkcmu_chub_bus_p) = { "mout_apm_bus_user", "mout_dll_user" }; > > +PNAME(mout_apm_bus_p) = { "mout_rco_apm_user", "mout_apm_bus_user", > > + "mout_dll_user", "oscclk_rco_apm" }; > > +PNAME(mout_apm_i3c_p) = { "dout_apm_i3c", "mout_rco_apm_i3c_user" }; > > + > > +static const struct samsung_fixed_rate_clock apm_fixed_clks[] __initconst = { > > + FRATE(CLK_RCO_I3C_PMIC, "clk_rco_i3c_pmic", NULL, 0, 491520000), > > + FRATE(OSCCLK_RCO_APM, "oscclk_rco_apm", NULL, 0, 24576000), > > + FRATE(CLK_RCO_APM__ALV, "clk_rco_apm__alv", NULL, 0, 49152000), > > + FRATE(CLK_DLL_DCO, "clk_dll_dco", NULL, 0, 360000000), > > +}; > > + > > +static const struct samsung_mux_clock apm_mux_clks[] __initconst = { > > + MUX(CLK_MOUT_APM_BUS_USER, "mout_apm_bus_user", mout_apm_bus_user_p, > > + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, 4, 1), > > + MUX(CLK_MOUT_RCO_APM_I3C_USER, "mout_rco_apm_i3c_user", > > + mout_rco_apm_i3c_user_p, PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, 4, 1), > > + MUX(CLK_MOUT_RCO_APM_USER, "mout_rco_apm_user", mout_rco_apm_user_p, > > + PLL_CON0_MUX_CLK_RCO_APM_USER, 4, 1), > > + MUX(CLK_MOUT_DLL_USER, "mout_dll_user", mout_dll_user_p, > > + PLL_CON0_MUX_DLL_USER, 4, 1), > > + MUX(CLK_MOUT_CLKCMU_CHUB_BUS, "mout_clkcmu_chub_bus", > > + mout_clkcmu_chub_bus_p, CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, 0, 1), > > + MUX(CLK_MOUT_APM_BUS, "mout_apm_bus", mout_apm_bus_p, > > + CLK_CON_MUX_MUX_CLK_APM_BUS, 0, 2), > > + MUX(CLK_MOUT_APM_I3C, "mout_apm_i3c", mout_apm_i3c_p, > > + CLK_CON_MUX_MUX_CLK_APM_I3C, 0, 1), > > +}; > > + > > +static const struct samsung_div_clock apm_div_clks[] __initconst = { > > + DIV(CLK_DOUT_CLKCMU_CHUB_BUS, "dout_clkcmu_chub_bus", > > + "gout_clkcmu_chub_bus", > > + CLK_CON_DIV_CLKCMU_CHUB_BUS, 0, 3), > > + DIV(CLK_DOUT_APM_BUS, "dout_apm_bus", "mout_apm_bus", > > + CLK_CON_DIV_DIV_CLK_APM_BUS, 0, 3), > > + DIV(CLK_DOUT_APM_I3C, "dout_apm_i3c", "mout_apm_bus", > > + CLK_CON_DIV_DIV_CLK_APM_I3C, 0, 3), > > +}; > > + > > +static const struct samsung_gate_clock apm_gate_clks[] __initconst = { > > + GATE(CLK_GOUT_CLKCMU_CMGP_BUS, "gout_clkcmu_cmgp_bus", "dout_apm_bus", > > + CLK_CON_GAT_CLKCMU_CMGP_BUS, 21, 0, 0), > > + GATE(CLK_GOUT_CLKCMU_CHUB_BUS, "gout_clkcmu_chub_bus", > > + "mout_clkcmu_chub_bus", > > + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, 21, 0, 0), > > + GATE(CLK_GOUT_RTC_PCLK, "gout_rtc_pclk", "dout_apm_bus", > > + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, 21, 0, 0), > > + GATE(CLK_GOUT_TOP_RTC_PCLK, "gout_top_rtc_pclk", "dout_apm_bus", > > + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, 21, 0, 0), > > + GATE(CLK_GOUT_I3C_PCLK, "gout_i3c_pclk", "dout_apm_bus", > > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, 21, 0, 0), > > + GATE(CLK_GOUT_I3C_SCLK, "gout_i3c_sclk", "mout_apm_i3c", > > + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, 21, 0, 0), > > + GATE(CLK_GOUT_SPEEDY_PCLK, "gout_speedy_pclk", "dout_apm_bus", > > + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, 21, 0, 0), > > +}; > > + > > +static const struct samsung_cmu_info apm_cmu_info __initconst = { > > + .mux_clks = apm_mux_clks, > > + .nr_mux_clks = ARRAY_SIZE(apm_mux_clks), > > + .div_clks = apm_div_clks, > > + .nr_div_clks = ARRAY_SIZE(apm_div_clks), > > + .gate_clks = apm_gate_clks, > > + .nr_gate_clks = ARRAY_SIZE(apm_gate_clks), > > + .fixed_clks = apm_fixed_clks, > > + .nr_fixed_clks = ARRAY_SIZE(apm_fixed_clks), > > + .nr_clk_ids = APM_NR_CLK, > > + .clk_regs = apm_clk_regs, > > + .nr_clk_regs = ARRAY_SIZE(apm_clk_regs), > > + .clk_name = "dout_clkcmu_apm_bus", > > +}; > > + > > /* ---- CMU_HSI ------------------------------------------------------------- */ > > > > /* Register Offset definitions for CMU_HSI (0x13400000) */ > > @@ -801,9 +939,11 @@ static int __init exynos850_cmu_probe(struct platform_device *pdev) > > return 0; > > } > > > > -/* CMUs which belong to Power Domains and need runtime PM to be implemented */ > > static const struct of_device_id exynos850_cmu_of_match[] = { > > { > > + .compatible = "samsung,exynos850-cmu-apm", > > + .data = &apm_cmu_info, > > + }, { > > .compatible = "samsung,exynos850-cmu-hsi", > > .data = &hsi_cmu_info, > > }, { > > diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h > > index 8999184f94a2..c65e0300a8d3 100644 > > --- a/include/dt-bindings/clock/exynos850.h > > +++ b/include/dt-bindings/clock/exynos850.h > > @@ -16,46 +16,73 @@ > > #define CLK_MOUT_SHARED0_PLL 4 > > #define CLK_MOUT_SHARED1_PLL 5 > > #define CLK_MOUT_MMC_PLL 6 > > -#define CLK_MOUT_CORE_BUS 7 > > -#define CLK_MOUT_CORE_CCI 8 > > -#define CLK_MOUT_CORE_MMC_EMBD 9 > > -#define CLK_MOUT_CORE_SSS 10 > > -#define CLK_MOUT_DPU 11 > > -#define CLK_MOUT_HSI_BUS 12 > > -#define CLK_MOUT_HSI_MMC_CARD 13 > > -#define CLK_MOUT_HSI_USB20DRD 14 > > -#define CLK_MOUT_PERI_BUS 15 > > -#define CLK_MOUT_PERI_UART 16 > > -#define CLK_MOUT_PERI_IP 17 > > -#define CLK_DOUT_SHARED0_DIV3 18 > > -#define CLK_DOUT_SHARED0_DIV2 19 > > -#define CLK_DOUT_SHARED1_DIV3 20 > > -#define CLK_DOUT_SHARED1_DIV2 21 > > -#define CLK_DOUT_SHARED0_DIV4 22 > > -#define CLK_DOUT_SHARED1_DIV4 23 > > -#define CLK_DOUT_CORE_BUS 24 > > -#define CLK_DOUT_CORE_CCI 25 > > -#define CLK_DOUT_CORE_MMC_EMBD 26 > > -#define CLK_DOUT_CORE_SSS 27 > > -#define CLK_DOUT_DPU 28 > > -#define CLK_DOUT_HSI_BUS 29 > > -#define CLK_DOUT_HSI_MMC_CARD 30 > > -#define CLK_DOUT_HSI_USB20DRD 31 > > -#define CLK_DOUT_PERI_BUS 32 > > -#define CLK_DOUT_PERI_UART 33 > > -#define CLK_DOUT_PERI_IP 34 > > -#define CLK_GOUT_CORE_BUS 35 > > -#define CLK_GOUT_CORE_CCI 36 > > -#define CLK_GOUT_CORE_MMC_EMBD 37 > > -#define CLK_GOUT_CORE_SSS 38 > > -#define CLK_GOUT_DPU 39 > > -#define CLK_GOUT_HSI_BUS 40 > > -#define CLK_GOUT_HSI_MMC_CARD 41 > > -#define CLK_GOUT_HSI_USB20DRD 42 > > -#define CLK_GOUT_PERI_BUS 43 > > -#define CLK_GOUT_PERI_UART 44 > > -#define CLK_GOUT_PERI_IP 45 > > -#define TOP_NR_CLK 46 > > +#define CLK_MOUT_CLKCMU_APM_BUS 7 > > +#define CLK_MOUT_CORE_BUS 8 > > > Basically, you can never change the already defined clock id > in nclude/dt-bindings/clock/*.h because of supporting > the compatibility of dtb files which were using the > already defined clock id instead of changed clock id > > If you want to add new clock with new clock id, > you have to define the new clock id at the end of defined clock > like the next of CLK_GOUT_PERI_IP for TOP domain case. > Thanks for explaining that in details, Chanwoo. As Krzysztof pointed out, right now there are no dts users of this clock driver in upstream kernel (I didn't submit it yet), so it'd nice if this one can be taken as is. In future I'll increment the last clock ID. Guess it was my OCD talking, trying to keep all clock IDs grouped by clock type :) > (snip) > > -- > Best Regards, > Samsung Electronics > Chanwoo Choi ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-22 13:39 ` Sam Protsenko @ 2021-10-22 14:08 ` Krzysztof Kozlowski 2021-10-22 22:18 ` Chanwoo Choi 1 sibling, 0 replies; 10+ messages in thread From: Krzysztof Kozlowski @ 2021-10-22 14:08 UTC (permalink / raw) To: Sam Protsenko, Chanwoo Choi Cc: Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette, Sumit Semwal, devicetree, linux-arm Mailing List, linux-clk, Linux Kernel Mailing List, Linux Samsung SOC On 22/10/2021 15:39, Sam Protsenko wrote: > On Fri, 22 Oct 2021 at 11:58, Chanwoo Choi <cwchoi00@gmail.com> wrote: >> >> On 21. 10. 22. 오전 5:31, Sam Protsenko wrote: >>> CMU_APM clock domain provides clocks for APM IP-core (Active Power >>> Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, >>> Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. >>> >>> This patch adds next clocks: >>> - bus clocks in CMU_TOP needed for CMU_APM >>> - all internal CMU_APM clocks >>> - leaf clocks for I3C, Speedy and RTC IP-cores >>> - bus clocks for CMU_CMGP and CMU_CHUB >>> >>> CMU_APM doesn't belong to Power Domains, but platform driver is used for >>> its registration to keep its bus clock always running. Otherwise rtc-s3c >>> driver disables that clock and system freezes. >>> >>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> >>> --- >>> drivers/clk/samsung/clk-exynos850.c | 142 +++++++++++++++++++++++++- >>> include/dt-bindings/clock/exynos850.h | 107 +++++++++++-------- >>> 2 files changed, 208 insertions(+), 41 deletions(-) >>> >>> diff --git a/drivers/clk/samsung/clk-exynos850.c b/drivers/clk/samsung/clk-exynos850.c >>> index 2294989e244c..95e373d17b42 100644 >>> --- a/drivers/clk/samsung/clk-exynos850.c >>> +++ b/drivers/clk/samsung/clk-exynos850.c >>> @@ -72,6 +72,7 @@ static void __init exynos850_init_clocks(struct device_node *np, >>> #define PLL_CON3_PLL_SHARED0 0x014c >>> #define PLL_CON0_PLL_SHARED1 0x0180 >>> #define PLL_CON3_PLL_SHARED1 0x018c >>> +#define CLK_CON_MUX_MUX_CLKCMU_APM_BUS 0x1000 >>> #define CLK_CON_MUX_MUX_CLKCMU_CORE_BUS 0x1014 >>> #define CLK_CON_MUX_MUX_CLKCMU_CORE_CCI 0x1018 >>> #define CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD 0x101c >>> @@ -83,6 +84,7 @@ static void __init exynos850_init_clocks(struct device_node *np, >>> #define CLK_CON_MUX_MUX_CLKCMU_PERI_BUS 0x1070 >>> #define CLK_CON_MUX_MUX_CLKCMU_PERI_IP 0x1074 >>> #define CLK_CON_MUX_MUX_CLKCMU_PERI_UART 0x1078 >>> +#define CLK_CON_DIV_CLKCMU_APM_BUS 0x180c >>> #define CLK_CON_DIV_CLKCMU_CORE_BUS 0x1820 >>> #define CLK_CON_DIV_CLKCMU_CORE_CCI 0x1824 >>> #define CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD 0x1828 >>> @@ -100,6 +102,7 @@ static void __init exynos850_init_clocks(struct device_node *np, >>> #define CLK_CON_DIV_PLL_SHARED1_DIV2 0x1898 >>> #define CLK_CON_DIV_PLL_SHARED1_DIV3 0x189c >>> #define CLK_CON_DIV_PLL_SHARED1_DIV4 0x18a0 >>> +#define CLK_CON_GAT_GATE_CLKCMU_APM_BUS 0x2008 >>> #define CLK_CON_GAT_GATE_CLKCMU_CORE_BUS 0x201c >>> #define CLK_CON_GAT_GATE_CLKCMU_CORE_CCI 0x2020 >>> #define CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD 0x2024 >>> @@ -122,6 +125,7 @@ static const unsigned long top_clk_regs[] __initconst = { >>> PLL_CON3_PLL_SHARED0, >>> PLL_CON0_PLL_SHARED1, >>> PLL_CON3_PLL_SHARED1, >>> + CLK_CON_MUX_MUX_CLKCMU_APM_BUS, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_CCI, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD, >>> @@ -133,6 +137,7 @@ static const unsigned long top_clk_regs[] __initconst = { >>> CLK_CON_MUX_MUX_CLKCMU_PERI_BUS, >>> CLK_CON_MUX_MUX_CLKCMU_PERI_IP, >>> CLK_CON_MUX_MUX_CLKCMU_PERI_UART, >>> + CLK_CON_DIV_CLKCMU_APM_BUS, >>> CLK_CON_DIV_CLKCMU_CORE_BUS, >>> CLK_CON_DIV_CLKCMU_CORE_CCI, >>> CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD, >>> @@ -150,6 +155,7 @@ static const unsigned long top_clk_regs[] __initconst = { >>> CLK_CON_DIV_PLL_SHARED1_DIV2, >>> CLK_CON_DIV_PLL_SHARED1_DIV3, >>> CLK_CON_DIV_PLL_SHARED1_DIV4, >>> + CLK_CON_GAT_GATE_CLKCMU_APM_BUS, >>> CLK_CON_GAT_GATE_CLKCMU_CORE_BUS, >>> CLK_CON_GAT_GATE_CLKCMU_CORE_CCI, >>> CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD, >>> @@ -183,6 +189,8 @@ static const struct samsung_pll_clock top_pll_clks[] __initconst = { >>> PNAME(mout_shared0_pll_p) = { "oscclk", "fout_shared0_pll" }; >>> PNAME(mout_shared1_pll_p) = { "oscclk", "fout_shared1_pll" }; >>> PNAME(mout_mmc_pll_p) = { "oscclk", "fout_mmc_pll" }; >>> +/* List of parent clocks for Muxes in CMU_TOP: for CMU_APM */ >>> +PNAME(mout_clkcmu_apm_bus_p) = { "dout_shared0_div4", "pll_shared1_div4" }; >>> /* List of parent clocks for Muxes in CMU_TOP: for CMU_CORE */ >>> PNAME(mout_core_bus_p) = { "dout_shared1_div2", "dout_shared0_div3", >>> "dout_shared1_div3", "dout_shared0_div4" }; >>> @@ -222,6 +230,10 @@ static const struct samsung_mux_clock top_mux_clks[] __initconst = { >>> MUX(CLK_MOUT_MMC_PLL, "mout_mmc_pll", mout_mmc_pll_p, >>> PLL_CON0_PLL_MMC, 4, 1), >>> >>> + /* APM */ >>> + MUX(CLK_MOUT_CLKCMU_APM_BUS, "mout_clkcmu_apm_bus", >>> + mout_clkcmu_apm_bus_p, CLK_CON_MUX_MUX_CLKCMU_APM_BUS, 0, 1), >>> + >>> /* CORE */ >>> MUX(CLK_MOUT_CORE_BUS, "mout_core_bus", mout_core_bus_p, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, 0, 2), >>> @@ -268,6 +280,10 @@ static const struct samsung_div_clock top_div_clks[] __initconst = { >>> DIV(CLK_DOUT_SHARED1_DIV4, "dout_shared1_div4", "dout_shared1_div2", >>> CLK_CON_DIV_PLL_SHARED1_DIV4, 0, 1), >>> >>> + /* APM */ >>> + DIV(CLK_DOUT_CLKCMU_APM_BUS, "dout_clkcmu_apm_bus", >>> + "gout_clkcmu_apm_bus", CLK_CON_DIV_CLKCMU_APM_BUS, 0, 3), >>> + >>> /* CORE */ >>> DIV(CLK_DOUT_CORE_BUS, "dout_core_bus", "gout_core_bus", >>> CLK_CON_DIV_CLKCMU_CORE_BUS, 0, 4), >>> @@ -310,6 +326,10 @@ static const struct samsung_gate_clock top_gate_clks[] __initconst = { >>> GATE(CLK_GOUT_CORE_SSS, "gout_core_sss", "mout_core_sss", >>> CLK_CON_GAT_GATE_CLKCMU_CORE_SSS, 21, 0, 0), >>> >>> + /* APM */ >>> + GATE(CLK_GOUT_CLKCMU_APM_BUS, "gout_clkcmu_apm_bus", >>> + "mout_clkcmu_apm_bus", CLK_CON_GAT_GATE_CLKCMU_APM_BUS, 21, 0, 0), >>> + >>> /* DPU */ >>> GATE(CLK_GOUT_DPU, "gout_dpu", "mout_dpu", >>> CLK_CON_GAT_GATE_CLKCMU_DPU, 21, 0, 0), >>> @@ -354,6 +374,124 @@ static void __init exynos850_cmu_top_init(struct device_node *np) >>> CLK_OF_DECLARE(exynos850_cmu_top, "samsung,exynos850-cmu-top", >>> exynos850_cmu_top_init); >>> >>> +/* ---- CMU_APM ------------------------------------------------------------- */ >>> + >>> +/* Register Offset definitions for CMU_APM (0x11800000) */ >>> +#define PLL_CON0_MUX_CLKCMU_APM_BUS_USER 0x0600 >>> +#define PLL_CON0_MUX_CLK_RCO_APM_I3C_USER 0x0610 >>> +#define PLL_CON0_MUX_CLK_RCO_APM_USER 0x0620 >>> +#define PLL_CON0_MUX_DLL_USER 0x0630 >>> +#define CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS 0x1000 >>> +#define CLK_CON_MUX_MUX_CLK_APM_BUS 0x1004 >>> +#define CLK_CON_MUX_MUX_CLK_APM_I3C 0x1008 >>> +#define CLK_CON_DIV_CLKCMU_CHUB_BUS 0x1800 >>> +#define CLK_CON_DIV_DIV_CLK_APM_BUS 0x1804 >>> +#define CLK_CON_DIV_DIV_CLK_APM_I3C 0x1808 >>> +#define CLK_CON_GAT_CLKCMU_CMGP_BUS 0x2000 >>> +#define CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS 0x2014 >>> +#define CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK 0x2024 >>> +#define CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK 0x2028 >>> +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK 0x2034 >>> +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK 0x2038 >>> +#define CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK 0x20bc >>> + >>> +static const unsigned long apm_clk_regs[] __initconst = { >>> + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, >>> + PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, >>> + PLL_CON0_MUX_CLK_RCO_APM_USER, >>> + PLL_CON0_MUX_DLL_USER, >>> + CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, >>> + CLK_CON_MUX_MUX_CLK_APM_BUS, >>> + CLK_CON_MUX_MUX_CLK_APM_I3C, >>> + CLK_CON_DIV_CLKCMU_CHUB_BUS, >>> + CLK_CON_DIV_DIV_CLK_APM_BUS, >>> + CLK_CON_DIV_DIV_CLK_APM_I3C, >>> + CLK_CON_GAT_CLKCMU_CMGP_BUS, >>> + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, >>> + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, >>> + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, >>> + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, >>> +}; >>> + >>> +/* List of parent clocks for Muxes in CMU_APM */ >>> +PNAME(mout_apm_bus_user_p) = { "oscclk_rco_apm", "dout_clkcmu_apm_bus" }; >>> +PNAME(mout_rco_apm_i3c_user_p) = { "oscclk_rco_apm", "clk_rco_i3c_pmic" }; >>> +PNAME(mout_rco_apm_user_p) = { "oscclk_rco_apm", "clk_rco_apm__alv" }; >>> +PNAME(mout_dll_user_p) = { "oscclk_rco_apm", "clk_dll_dco" }; >>> +PNAME(mout_clkcmu_chub_bus_p) = { "mout_apm_bus_user", "mout_dll_user" }; >>> +PNAME(mout_apm_bus_p) = { "mout_rco_apm_user", "mout_apm_bus_user", >>> + "mout_dll_user", "oscclk_rco_apm" }; >>> +PNAME(mout_apm_i3c_p) = { "dout_apm_i3c", "mout_rco_apm_i3c_user" }; >>> + >>> +static const struct samsung_fixed_rate_clock apm_fixed_clks[] __initconst = { >>> + FRATE(CLK_RCO_I3C_PMIC, "clk_rco_i3c_pmic", NULL, 0, 491520000), >>> + FRATE(OSCCLK_RCO_APM, "oscclk_rco_apm", NULL, 0, 24576000), >>> + FRATE(CLK_RCO_APM__ALV, "clk_rco_apm__alv", NULL, 0, 49152000), >>> + FRATE(CLK_DLL_DCO, "clk_dll_dco", NULL, 0, 360000000), >>> +}; >>> + >>> +static const struct samsung_mux_clock apm_mux_clks[] __initconst = { >>> + MUX(CLK_MOUT_APM_BUS_USER, "mout_apm_bus_user", mout_apm_bus_user_p, >>> + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, 4, 1), >>> + MUX(CLK_MOUT_RCO_APM_I3C_USER, "mout_rco_apm_i3c_user", >>> + mout_rco_apm_i3c_user_p, PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, 4, 1), >>> + MUX(CLK_MOUT_RCO_APM_USER, "mout_rco_apm_user", mout_rco_apm_user_p, >>> + PLL_CON0_MUX_CLK_RCO_APM_USER, 4, 1), >>> + MUX(CLK_MOUT_DLL_USER, "mout_dll_user", mout_dll_user_p, >>> + PLL_CON0_MUX_DLL_USER, 4, 1), >>> + MUX(CLK_MOUT_CLKCMU_CHUB_BUS, "mout_clkcmu_chub_bus", >>> + mout_clkcmu_chub_bus_p, CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, 0, 1), >>> + MUX(CLK_MOUT_APM_BUS, "mout_apm_bus", mout_apm_bus_p, >>> + CLK_CON_MUX_MUX_CLK_APM_BUS, 0, 2), >>> + MUX(CLK_MOUT_APM_I3C, "mout_apm_i3c", mout_apm_i3c_p, >>> + CLK_CON_MUX_MUX_CLK_APM_I3C, 0, 1), >>> +}; >>> + >>> +static const struct samsung_div_clock apm_div_clks[] __initconst = { >>> + DIV(CLK_DOUT_CLKCMU_CHUB_BUS, "dout_clkcmu_chub_bus", >>> + "gout_clkcmu_chub_bus", >>> + CLK_CON_DIV_CLKCMU_CHUB_BUS, 0, 3), >>> + DIV(CLK_DOUT_APM_BUS, "dout_apm_bus", "mout_apm_bus", >>> + CLK_CON_DIV_DIV_CLK_APM_BUS, 0, 3), >>> + DIV(CLK_DOUT_APM_I3C, "dout_apm_i3c", "mout_apm_bus", >>> + CLK_CON_DIV_DIV_CLK_APM_I3C, 0, 3), >>> +}; >>> + >>> +static const struct samsung_gate_clock apm_gate_clks[] __initconst = { >>> + GATE(CLK_GOUT_CLKCMU_CMGP_BUS, "gout_clkcmu_cmgp_bus", "dout_apm_bus", >>> + CLK_CON_GAT_CLKCMU_CMGP_BUS, 21, 0, 0), >>> + GATE(CLK_GOUT_CLKCMU_CHUB_BUS, "gout_clkcmu_chub_bus", >>> + "mout_clkcmu_chub_bus", >>> + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, 21, 0, 0), >>> + GATE(CLK_GOUT_RTC_PCLK, "gout_rtc_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_TOP_RTC_PCLK, "gout_top_rtc_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_I3C_PCLK, "gout_i3c_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_I3C_SCLK, "gout_i3c_sclk", "mout_apm_i3c", >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_SPEEDY_PCLK, "gout_speedy_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, 21, 0, 0), >>> +}; >>> + >>> +static const struct samsung_cmu_info apm_cmu_info __initconst = { >>> + .mux_clks = apm_mux_clks, >>> + .nr_mux_clks = ARRAY_SIZE(apm_mux_clks), >>> + .div_clks = apm_div_clks, >>> + .nr_div_clks = ARRAY_SIZE(apm_div_clks), >>> + .gate_clks = apm_gate_clks, >>> + .nr_gate_clks = ARRAY_SIZE(apm_gate_clks), >>> + .fixed_clks = apm_fixed_clks, >>> + .nr_fixed_clks = ARRAY_SIZE(apm_fixed_clks), >>> + .nr_clk_ids = APM_NR_CLK, >>> + .clk_regs = apm_clk_regs, >>> + .nr_clk_regs = ARRAY_SIZE(apm_clk_regs), >>> + .clk_name = "dout_clkcmu_apm_bus", >>> +}; >>> + >>> /* ---- CMU_HSI ------------------------------------------------------------- */ >>> >>> /* Register Offset definitions for CMU_HSI (0x13400000) */ >>> @@ -801,9 +939,11 @@ static int __init exynos850_cmu_probe(struct platform_device *pdev) >>> return 0; >>> } >>> >>> -/* CMUs which belong to Power Domains and need runtime PM to be implemented */ >>> static const struct of_device_id exynos850_cmu_of_match[] = { >>> { >>> + .compatible = "samsung,exynos850-cmu-apm", >>> + .data = &apm_cmu_info, >>> + }, { >>> .compatible = "samsung,exynos850-cmu-hsi", >>> .data = &hsi_cmu_info, >>> }, { >>> diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h >>> index 8999184f94a2..c65e0300a8d3 100644 >>> --- a/include/dt-bindings/clock/exynos850.h >>> +++ b/include/dt-bindings/clock/exynos850.h >>> @@ -16,46 +16,73 @@ >>> #define CLK_MOUT_SHARED0_PLL 4 >>> #define CLK_MOUT_SHARED1_PLL 5 >>> #define CLK_MOUT_MMC_PLL 6 >>> -#define CLK_MOUT_CORE_BUS 7 >>> -#define CLK_MOUT_CORE_CCI 8 >>> -#define CLK_MOUT_CORE_MMC_EMBD 9 >>> -#define CLK_MOUT_CORE_SSS 10 >>> -#define CLK_MOUT_DPU 11 >>> -#define CLK_MOUT_HSI_BUS 12 >>> -#define CLK_MOUT_HSI_MMC_CARD 13 >>> -#define CLK_MOUT_HSI_USB20DRD 14 >>> -#define CLK_MOUT_PERI_BUS 15 >>> -#define CLK_MOUT_PERI_UART 16 >>> -#define CLK_MOUT_PERI_IP 17 >>> -#define CLK_DOUT_SHARED0_DIV3 18 >>> -#define CLK_DOUT_SHARED0_DIV2 19 >>> -#define CLK_DOUT_SHARED1_DIV3 20 >>> -#define CLK_DOUT_SHARED1_DIV2 21 >>> -#define CLK_DOUT_SHARED0_DIV4 22 >>> -#define CLK_DOUT_SHARED1_DIV4 23 >>> -#define CLK_DOUT_CORE_BUS 24 >>> -#define CLK_DOUT_CORE_CCI 25 >>> -#define CLK_DOUT_CORE_MMC_EMBD 26 >>> -#define CLK_DOUT_CORE_SSS 27 >>> -#define CLK_DOUT_DPU 28 >>> -#define CLK_DOUT_HSI_BUS 29 >>> -#define CLK_DOUT_HSI_MMC_CARD 30 >>> -#define CLK_DOUT_HSI_USB20DRD 31 >>> -#define CLK_DOUT_PERI_BUS 32 >>> -#define CLK_DOUT_PERI_UART 33 >>> -#define CLK_DOUT_PERI_IP 34 >>> -#define CLK_GOUT_CORE_BUS 35 >>> -#define CLK_GOUT_CORE_CCI 36 >>> -#define CLK_GOUT_CORE_MMC_EMBD 37 >>> -#define CLK_GOUT_CORE_SSS 38 >>> -#define CLK_GOUT_DPU 39 >>> -#define CLK_GOUT_HSI_BUS 40 >>> -#define CLK_GOUT_HSI_MMC_CARD 41 >>> -#define CLK_GOUT_HSI_USB20DRD 42 >>> -#define CLK_GOUT_PERI_BUS 43 >>> -#define CLK_GOUT_PERI_UART 44 >>> -#define CLK_GOUT_PERI_IP 45 >>> -#define TOP_NR_CLK 46 >>> +#define CLK_MOUT_CLKCMU_APM_BUS 7 >>> +#define CLK_MOUT_CORE_BUS 8 >> >> >> Basically, you can never change the already defined clock id >> in nclude/dt-bindings/clock/*.h because of supporting >> the compatibility of dtb files which were using the >> already defined clock id instead of changed clock id >> >> If you want to add new clock with new clock id, >> you have to define the new clock id at the end of defined clock >> like the next of CLK_GOUT_PERI_IP for TOP domain case. >> > > Thanks for explaining that in details, Chanwoo. As Krzysztof pointed > out, right now there are no dts users of this clock driver in upstream > kernel (I didn't submit it yet), so it'd nice if this one can be taken > as is. In future I'll increment the last clock ID. Guess it was my OCD > talking, trying to keep all clock IDs grouped by clock type :) Yes, I think now breaking DTBs is allowed because this is a new driver and there are no users of it. Kind of exception. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-22 13:39 ` Sam Protsenko 2021-10-22 14:08 ` Krzysztof Kozlowski @ 2021-10-22 22:18 ` Chanwoo Choi 2021-10-22 22:31 ` Sam Protsenko 1 sibling, 1 reply; 10+ messages in thread From: Chanwoo Choi @ 2021-10-22 22:18 UTC (permalink / raw) To: Sam Protsenko Cc: Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette, Sumit Semwal, devicetree, linux-arm Mailing List, linux-clk, Linux Kernel Mailing List, Linux Samsung SOC On 21. 10. 22. 오후 10:39, Sam Protsenko wrote: > On Fri, 22 Oct 2021 at 11:58, Chanwoo Choi <cwchoi00@gmail.com> wrote: >> >> On 21. 10. 22. 오전 5:31, Sam Protsenko wrote: >>> CMU_APM clock domain provides clocks for APM IP-core (Active Power >>> Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, >>> Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. >>> >>> This patch adds next clocks: >>> - bus clocks in CMU_TOP needed for CMU_APM >>> - all internal CMU_APM clocks >>> - leaf clocks for I3C, Speedy and RTC IP-cores >>> - bus clocks for CMU_CMGP and CMU_CHUB >>> >>> CMU_APM doesn't belong to Power Domains, but platform driver is used for >>> its registration to keep its bus clock always running. Otherwise rtc-s3c >>> driver disables that clock and system freezes. >>> >>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> >>> --- >>> drivers/clk/samsung/clk-exynos850.c | 142 +++++++++++++++++++++++++- >>> include/dt-bindings/clock/exynos850.h | 107 +++++++++++-------- >>> 2 files changed, 208 insertions(+), 41 deletions(-) >>> >>> diff --git a/drivers/clk/samsung/clk-exynos850.c b/drivers/clk/samsung/clk-exynos850.c >>> index 2294989e244c..95e373d17b42 100644 >>> --- a/drivers/clk/samsung/clk-exynos850.c >>> +++ b/drivers/clk/samsung/clk-exynos850.c >>> @@ -72,6 +72,7 @@ static void __init exynos850_init_clocks(struct device_node *np, >>> #define PLL_CON3_PLL_SHARED0 0x014c >>> #define PLL_CON0_PLL_SHARED1 0x0180 >>> #define PLL_CON3_PLL_SHARED1 0x018c >>> +#define CLK_CON_MUX_MUX_CLKCMU_APM_BUS 0x1000 >>> #define CLK_CON_MUX_MUX_CLKCMU_CORE_BUS 0x1014 >>> #define CLK_CON_MUX_MUX_CLKCMU_CORE_CCI 0x1018 >>> #define CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD 0x101c >>> @@ -83,6 +84,7 @@ static void __init exynos850_init_clocks(struct device_node *np, >>> #define CLK_CON_MUX_MUX_CLKCMU_PERI_BUS 0x1070 >>> #define CLK_CON_MUX_MUX_CLKCMU_PERI_IP 0x1074 >>> #define CLK_CON_MUX_MUX_CLKCMU_PERI_UART 0x1078 >>> +#define CLK_CON_DIV_CLKCMU_APM_BUS 0x180c >>> #define CLK_CON_DIV_CLKCMU_CORE_BUS 0x1820 >>> #define CLK_CON_DIV_CLKCMU_CORE_CCI 0x1824 >>> #define CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD 0x1828 >>> @@ -100,6 +102,7 @@ static void __init exynos850_init_clocks(struct device_node *np, >>> #define CLK_CON_DIV_PLL_SHARED1_DIV2 0x1898 >>> #define CLK_CON_DIV_PLL_SHARED1_DIV3 0x189c >>> #define CLK_CON_DIV_PLL_SHARED1_DIV4 0x18a0 >>> +#define CLK_CON_GAT_GATE_CLKCMU_APM_BUS 0x2008 >>> #define CLK_CON_GAT_GATE_CLKCMU_CORE_BUS 0x201c >>> #define CLK_CON_GAT_GATE_CLKCMU_CORE_CCI 0x2020 >>> #define CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD 0x2024 >>> @@ -122,6 +125,7 @@ static const unsigned long top_clk_regs[] __initconst = { >>> PLL_CON3_PLL_SHARED0, >>> PLL_CON0_PLL_SHARED1, >>> PLL_CON3_PLL_SHARED1, >>> + CLK_CON_MUX_MUX_CLKCMU_APM_BUS, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_CCI, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_MMC_EMBD, >>> @@ -133,6 +137,7 @@ static const unsigned long top_clk_regs[] __initconst = { >>> CLK_CON_MUX_MUX_CLKCMU_PERI_BUS, >>> CLK_CON_MUX_MUX_CLKCMU_PERI_IP, >>> CLK_CON_MUX_MUX_CLKCMU_PERI_UART, >>> + CLK_CON_DIV_CLKCMU_APM_BUS, >>> CLK_CON_DIV_CLKCMU_CORE_BUS, >>> CLK_CON_DIV_CLKCMU_CORE_CCI, >>> CLK_CON_DIV_CLKCMU_CORE_MMC_EMBD, >>> @@ -150,6 +155,7 @@ static const unsigned long top_clk_regs[] __initconst = { >>> CLK_CON_DIV_PLL_SHARED1_DIV2, >>> CLK_CON_DIV_PLL_SHARED1_DIV3, >>> CLK_CON_DIV_PLL_SHARED1_DIV4, >>> + CLK_CON_GAT_GATE_CLKCMU_APM_BUS, >>> CLK_CON_GAT_GATE_CLKCMU_CORE_BUS, >>> CLK_CON_GAT_GATE_CLKCMU_CORE_CCI, >>> CLK_CON_GAT_GATE_CLKCMU_CORE_MMC_EMBD, >>> @@ -183,6 +189,8 @@ static const struct samsung_pll_clock top_pll_clks[] __initconst = { >>> PNAME(mout_shared0_pll_p) = { "oscclk", "fout_shared0_pll" }; >>> PNAME(mout_shared1_pll_p) = { "oscclk", "fout_shared1_pll" }; >>> PNAME(mout_mmc_pll_p) = { "oscclk", "fout_mmc_pll" }; >>> +/* List of parent clocks for Muxes in CMU_TOP: for CMU_APM */ >>> +PNAME(mout_clkcmu_apm_bus_p) = { "dout_shared0_div4", "pll_shared1_div4" }; >>> /* List of parent clocks for Muxes in CMU_TOP: for CMU_CORE */ >>> PNAME(mout_core_bus_p) = { "dout_shared1_div2", "dout_shared0_div3", >>> "dout_shared1_div3", "dout_shared0_div4" }; >>> @@ -222,6 +230,10 @@ static const struct samsung_mux_clock top_mux_clks[] __initconst = { >>> MUX(CLK_MOUT_MMC_PLL, "mout_mmc_pll", mout_mmc_pll_p, >>> PLL_CON0_PLL_MMC, 4, 1), >>> >>> + /* APM */ >>> + MUX(CLK_MOUT_CLKCMU_APM_BUS, "mout_clkcmu_apm_bus", >>> + mout_clkcmu_apm_bus_p, CLK_CON_MUX_MUX_CLKCMU_APM_BUS, 0, 1), >>> + >>> /* CORE */ >>> MUX(CLK_MOUT_CORE_BUS, "mout_core_bus", mout_core_bus_p, >>> CLK_CON_MUX_MUX_CLKCMU_CORE_BUS, 0, 2), >>> @@ -268,6 +280,10 @@ static const struct samsung_div_clock top_div_clks[] __initconst = { >>> DIV(CLK_DOUT_SHARED1_DIV4, "dout_shared1_div4", "dout_shared1_div2", >>> CLK_CON_DIV_PLL_SHARED1_DIV4, 0, 1), >>> >>> + /* APM */ >>> + DIV(CLK_DOUT_CLKCMU_APM_BUS, "dout_clkcmu_apm_bus", >>> + "gout_clkcmu_apm_bus", CLK_CON_DIV_CLKCMU_APM_BUS, 0, 3), >>> + >>> /* CORE */ >>> DIV(CLK_DOUT_CORE_BUS, "dout_core_bus", "gout_core_bus", >>> CLK_CON_DIV_CLKCMU_CORE_BUS, 0, 4), >>> @@ -310,6 +326,10 @@ static const struct samsung_gate_clock top_gate_clks[] __initconst = { >>> GATE(CLK_GOUT_CORE_SSS, "gout_core_sss", "mout_core_sss", >>> CLK_CON_GAT_GATE_CLKCMU_CORE_SSS, 21, 0, 0), >>> >>> + /* APM */ >>> + GATE(CLK_GOUT_CLKCMU_APM_BUS, "gout_clkcmu_apm_bus", >>> + "mout_clkcmu_apm_bus", CLK_CON_GAT_GATE_CLKCMU_APM_BUS, 21, 0, 0), >>> + >>> /* DPU */ >>> GATE(CLK_GOUT_DPU, "gout_dpu", "mout_dpu", >>> CLK_CON_GAT_GATE_CLKCMU_DPU, 21, 0, 0), >>> @@ -354,6 +374,124 @@ static void __init exynos850_cmu_top_init(struct device_node *np) >>> CLK_OF_DECLARE(exynos850_cmu_top, "samsung,exynos850-cmu-top", >>> exynos850_cmu_top_init); >>> >>> +/* ---- CMU_APM ------------------------------------------------------------- */ >>> + >>> +/* Register Offset definitions for CMU_APM (0x11800000) */ >>> +#define PLL_CON0_MUX_CLKCMU_APM_BUS_USER 0x0600 >>> +#define PLL_CON0_MUX_CLK_RCO_APM_I3C_USER 0x0610 >>> +#define PLL_CON0_MUX_CLK_RCO_APM_USER 0x0620 >>> +#define PLL_CON0_MUX_DLL_USER 0x0630 >>> +#define CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS 0x1000 >>> +#define CLK_CON_MUX_MUX_CLK_APM_BUS 0x1004 >>> +#define CLK_CON_MUX_MUX_CLK_APM_I3C 0x1008 >>> +#define CLK_CON_DIV_CLKCMU_CHUB_BUS 0x1800 >>> +#define CLK_CON_DIV_DIV_CLK_APM_BUS 0x1804 >>> +#define CLK_CON_DIV_DIV_CLK_APM_I3C 0x1808 >>> +#define CLK_CON_GAT_CLKCMU_CMGP_BUS 0x2000 >>> +#define CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS 0x2014 >>> +#define CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK 0x2024 >>> +#define CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK 0x2028 >>> +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK 0x2034 >>> +#define CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK 0x2038 >>> +#define CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK 0x20bc >>> + >>> +static const unsigned long apm_clk_regs[] __initconst = { >>> + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, >>> + PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, >>> + PLL_CON0_MUX_CLK_RCO_APM_USER, >>> + PLL_CON0_MUX_DLL_USER, >>> + CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, >>> + CLK_CON_MUX_MUX_CLK_APM_BUS, >>> + CLK_CON_MUX_MUX_CLK_APM_I3C, >>> + CLK_CON_DIV_CLKCMU_CHUB_BUS, >>> + CLK_CON_DIV_DIV_CLK_APM_BUS, >>> + CLK_CON_DIV_DIV_CLK_APM_I3C, >>> + CLK_CON_GAT_CLKCMU_CMGP_BUS, >>> + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, >>> + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, >>> + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, >>> + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, >>> +}; >>> + >>> +/* List of parent clocks for Muxes in CMU_APM */ >>> +PNAME(mout_apm_bus_user_p) = { "oscclk_rco_apm", "dout_clkcmu_apm_bus" }; >>> +PNAME(mout_rco_apm_i3c_user_p) = { "oscclk_rco_apm", "clk_rco_i3c_pmic" }; >>> +PNAME(mout_rco_apm_user_p) = { "oscclk_rco_apm", "clk_rco_apm__alv" }; >>> +PNAME(mout_dll_user_p) = { "oscclk_rco_apm", "clk_dll_dco" }; >>> +PNAME(mout_clkcmu_chub_bus_p) = { "mout_apm_bus_user", "mout_dll_user" }; >>> +PNAME(mout_apm_bus_p) = { "mout_rco_apm_user", "mout_apm_bus_user", >>> + "mout_dll_user", "oscclk_rco_apm" }; >>> +PNAME(mout_apm_i3c_p) = { "dout_apm_i3c", "mout_rco_apm_i3c_user" }; >>> + >>> +static const struct samsung_fixed_rate_clock apm_fixed_clks[] __initconst = { >>> + FRATE(CLK_RCO_I3C_PMIC, "clk_rco_i3c_pmic", NULL, 0, 491520000), >>> + FRATE(OSCCLK_RCO_APM, "oscclk_rco_apm", NULL, 0, 24576000), >>> + FRATE(CLK_RCO_APM__ALV, "clk_rco_apm__alv", NULL, 0, 49152000), >>> + FRATE(CLK_DLL_DCO, "clk_dll_dco", NULL, 0, 360000000), >>> +}; >>> + >>> +static const struct samsung_mux_clock apm_mux_clks[] __initconst = { >>> + MUX(CLK_MOUT_APM_BUS_USER, "mout_apm_bus_user", mout_apm_bus_user_p, >>> + PLL_CON0_MUX_CLKCMU_APM_BUS_USER, 4, 1), >>> + MUX(CLK_MOUT_RCO_APM_I3C_USER, "mout_rco_apm_i3c_user", >>> + mout_rco_apm_i3c_user_p, PLL_CON0_MUX_CLK_RCO_APM_I3C_USER, 4, 1), >>> + MUX(CLK_MOUT_RCO_APM_USER, "mout_rco_apm_user", mout_rco_apm_user_p, >>> + PLL_CON0_MUX_CLK_RCO_APM_USER, 4, 1), >>> + MUX(CLK_MOUT_DLL_USER, "mout_dll_user", mout_dll_user_p, >>> + PLL_CON0_MUX_DLL_USER, 4, 1), >>> + MUX(CLK_MOUT_CLKCMU_CHUB_BUS, "mout_clkcmu_chub_bus", >>> + mout_clkcmu_chub_bus_p, CLK_CON_MUX_MUX_CLKCMU_CHUB_BUS, 0, 1), >>> + MUX(CLK_MOUT_APM_BUS, "mout_apm_bus", mout_apm_bus_p, >>> + CLK_CON_MUX_MUX_CLK_APM_BUS, 0, 2), >>> + MUX(CLK_MOUT_APM_I3C, "mout_apm_i3c", mout_apm_i3c_p, >>> + CLK_CON_MUX_MUX_CLK_APM_I3C, 0, 1), >>> +}; >>> + >>> +static const struct samsung_div_clock apm_div_clks[] __initconst = { >>> + DIV(CLK_DOUT_CLKCMU_CHUB_BUS, "dout_clkcmu_chub_bus", >>> + "gout_clkcmu_chub_bus", >>> + CLK_CON_DIV_CLKCMU_CHUB_BUS, 0, 3), >>> + DIV(CLK_DOUT_APM_BUS, "dout_apm_bus", "mout_apm_bus", >>> + CLK_CON_DIV_DIV_CLK_APM_BUS, 0, 3), >>> + DIV(CLK_DOUT_APM_I3C, "dout_apm_i3c", "mout_apm_bus", >>> + CLK_CON_DIV_DIV_CLK_APM_I3C, 0, 3), >>> +}; >>> + >>> +static const struct samsung_gate_clock apm_gate_clks[] __initconst = { >>> + GATE(CLK_GOUT_CLKCMU_CMGP_BUS, "gout_clkcmu_cmgp_bus", "dout_apm_bus", >>> + CLK_CON_GAT_CLKCMU_CMGP_BUS, 21, 0, 0), >>> + GATE(CLK_GOUT_CLKCMU_CHUB_BUS, "gout_clkcmu_chub_bus", >>> + "mout_clkcmu_chub_bus", >>> + CLK_CON_GAT_GATE_CLKCMU_CHUB_BUS, 21, 0, 0), >>> + GATE(CLK_GOUT_RTC_PCLK, "gout_rtc_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_APBIF_RTC_PCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_TOP_RTC_PCLK, "gout_top_rtc_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_APBIF_TOP_RTC_PCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_I3C_PCLK, "gout_i3c_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_PCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_I3C_SCLK, "gout_i3c_sclk", "mout_apm_i3c", >>> + CLK_CON_GAT_GOUT_APM_I3C_APM_PMIC_I_SCLK, 21, 0, 0), >>> + GATE(CLK_GOUT_SPEEDY_PCLK, "gout_speedy_pclk", "dout_apm_bus", >>> + CLK_CON_GAT_GOUT_APM_SPEEDY_APM_PCLK, 21, 0, 0), >>> +}; >>> + >>> +static const struct samsung_cmu_info apm_cmu_info __initconst = { >>> + .mux_clks = apm_mux_clks, >>> + .nr_mux_clks = ARRAY_SIZE(apm_mux_clks), >>> + .div_clks = apm_div_clks, >>> + .nr_div_clks = ARRAY_SIZE(apm_div_clks), >>> + .gate_clks = apm_gate_clks, >>> + .nr_gate_clks = ARRAY_SIZE(apm_gate_clks), >>> + .fixed_clks = apm_fixed_clks, >>> + .nr_fixed_clks = ARRAY_SIZE(apm_fixed_clks), >>> + .nr_clk_ids = APM_NR_CLK, >>> + .clk_regs = apm_clk_regs, >>> + .nr_clk_regs = ARRAY_SIZE(apm_clk_regs), >>> + .clk_name = "dout_clkcmu_apm_bus", >>> +}; >>> + >>> /* ---- CMU_HSI ------------------------------------------------------------- */ >>> >>> /* Register Offset definitions for CMU_HSI (0x13400000) */ >>> @@ -801,9 +939,11 @@ static int __init exynos850_cmu_probe(struct platform_device *pdev) >>> return 0; >>> } >>> >>> -/* CMUs which belong to Power Domains and need runtime PM to be implemented */ >>> static const struct of_device_id exynos850_cmu_of_match[] = { >>> { >>> + .compatible = "samsung,exynos850-cmu-apm", >>> + .data = &apm_cmu_info, >>> + }, { >>> .compatible = "samsung,exynos850-cmu-hsi", >>> .data = &hsi_cmu_info, >>> }, { >>> diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h >>> index 8999184f94a2..c65e0300a8d3 100644 >>> --- a/include/dt-bindings/clock/exynos850.h >>> +++ b/include/dt-bindings/clock/exynos850.h >>> @@ -16,46 +16,73 @@ >>> #define CLK_MOUT_SHARED0_PLL 4 >>> #define CLK_MOUT_SHARED1_PLL 5 >>> #define CLK_MOUT_MMC_PLL 6 >>> -#define CLK_MOUT_CORE_BUS 7 >>> -#define CLK_MOUT_CORE_CCI 8 >>> -#define CLK_MOUT_CORE_MMC_EMBD 9 >>> -#define CLK_MOUT_CORE_SSS 10 >>> -#define CLK_MOUT_DPU 11 >>> -#define CLK_MOUT_HSI_BUS 12 >>> -#define CLK_MOUT_HSI_MMC_CARD 13 >>> -#define CLK_MOUT_HSI_USB20DRD 14 >>> -#define CLK_MOUT_PERI_BUS 15 >>> -#define CLK_MOUT_PERI_UART 16 >>> -#define CLK_MOUT_PERI_IP 17 >>> -#define CLK_DOUT_SHARED0_DIV3 18 >>> -#define CLK_DOUT_SHARED0_DIV2 19 >>> -#define CLK_DOUT_SHARED1_DIV3 20 >>> -#define CLK_DOUT_SHARED1_DIV2 21 >>> -#define CLK_DOUT_SHARED0_DIV4 22 >>> -#define CLK_DOUT_SHARED1_DIV4 23 >>> -#define CLK_DOUT_CORE_BUS 24 >>> -#define CLK_DOUT_CORE_CCI 25 >>> -#define CLK_DOUT_CORE_MMC_EMBD 26 >>> -#define CLK_DOUT_CORE_SSS 27 >>> -#define CLK_DOUT_DPU 28 >>> -#define CLK_DOUT_HSI_BUS 29 >>> -#define CLK_DOUT_HSI_MMC_CARD 30 >>> -#define CLK_DOUT_HSI_USB20DRD 31 >>> -#define CLK_DOUT_PERI_BUS 32 >>> -#define CLK_DOUT_PERI_UART 33 >>> -#define CLK_DOUT_PERI_IP 34 >>> -#define CLK_GOUT_CORE_BUS 35 >>> -#define CLK_GOUT_CORE_CCI 36 >>> -#define CLK_GOUT_CORE_MMC_EMBD 37 >>> -#define CLK_GOUT_CORE_SSS 38 >>> -#define CLK_GOUT_DPU 39 >>> -#define CLK_GOUT_HSI_BUS 40 >>> -#define CLK_GOUT_HSI_MMC_CARD 41 >>> -#define CLK_GOUT_HSI_USB20DRD 42 >>> -#define CLK_GOUT_PERI_BUS 43 >>> -#define CLK_GOUT_PERI_UART 44 >>> -#define CLK_GOUT_PERI_IP 45 >>> -#define TOP_NR_CLK 46 >>> +#define CLK_MOUT_CLKCMU_APM_BUS 7 >>> +#define CLK_MOUT_CORE_BUS 8 >> >> >> Basically, you can never change the already defined clock id >> in nclude/dt-bindings/clock/*.h because of supporting >> the compatibility of dtb files which were using the >> already defined clock id instead of changed clock id >> >> If you want to add new clock with new clock id, >> you have to define the new clock id at the end of defined clock >> like the next of CLK_GOUT_PERI_IP for TOP domain case. >> > > Thanks for explaining that in details, Chanwoo. As Krzysztof pointed > out, right now there are no dts users of this clock driver in upstream > kernel (I didn't submit it yet), so it'd nice if this one can be taken > as is. In future I'll increment the last clock ID. Guess it was my OCD > talking, trying to keep all clock IDs grouped by clock type :) I know that there are no user for this clock. So that it doesn't make the real break for compatibility. But, when some kernel developers might check the kernel history by git command, they never know the history only we know. If there are any critical reason, I don't prefer to break the rule of clock id defintion for patch history. Just I want to keep the original rule of clock id patch in order to remove the potential confusion. It is not a strong objection. But In my case, I cannot reply the ack. Thanks for your work. -- Best Regards, Samsung Electronics Chanwoo Choi ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain 2021-10-22 22:18 ` Chanwoo Choi @ 2021-10-22 22:31 ` Sam Protsenko 0 siblings, 0 replies; 10+ messages in thread From: Sam Protsenko @ 2021-10-22 22:31 UTC (permalink / raw) To: Chanwoo Choi Cc: Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette, Sumit Semwal, devicetree, linux-arm Mailing List, linux-clk, Linux Kernel Mailing List, Linux Samsung SOC On Sat, 23 Oct 2021 at 01:18, Chanwoo Choi <cwchoi00@gmail.com> wrote: > > On 21. 10. 22. 오후 10:39, Sam Protsenko wrote: > > On Fri, 22 Oct 2021 at 11:58, Chanwoo Choi <cwchoi00@gmail.com> wrote: > >> > >> On 21. 10. 22. 오전 5:31, Sam Protsenko wrote: > >>> CMU_APM clock domain provides clocks for APM IP-core (Active Power > >>> Management). According to Exynos850 TRM, CMU_APM generates I3C, Mailbox, > >>> Speedy, Timer, WDT, RTC and PMU clocks for BLK_ALIVE. > >>> > >>> This patch adds next clocks: > >>> - bus clocks in CMU_TOP needed for CMU_APM > >>> - all internal CMU_APM clocks > >>> - leaf clocks for I3C, Speedy and RTC IP-cores > >>> - bus clocks for CMU_CMGP and CMU_CHUB > >>> > >>> CMU_APM doesn't belong to Power Domains, but platform driver is used for > >>> its registration to keep its bus clock always running. Otherwise rtc-s3c > >>> driver disables that clock and system freezes. > >>> > >>> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > >>> --- ...snip... > >> Basically, you can never change the already defined clock id > >> in nclude/dt-bindings/clock/*.h because of supporting > >> the compatibility of dtb files which were using the > >> already defined clock id instead of changed clock id > >> > >> If you want to add new clock with new clock id, > >> you have to define the new clock id at the end of defined clock > >> like the next of CLK_GOUT_PERI_IP for TOP domain case. > >> > > > > Thanks for explaining that in details, Chanwoo. As Krzysztof pointed > > out, right now there are no dts users of this clock driver in upstream > > kernel (I didn't submit it yet), so it'd nice if this one can be taken > > as is. In future I'll increment the last clock ID. Guess it was my OCD > > talking, trying to keep all clock IDs grouped by clock type :) > > I know that there are no user for this clock. So that it doesn't make > the real break for compatibility. But, when some kernel developers might > check the kernel history by git command, they never know the history > only we know. If there are any critical reason, I don't prefer to break > the rule of clock id defintion for patch history. > > Just I want to keep the original rule of clock id patch in order to remove > the potential confusion. It is not a strong objection. But In my case, > I cannot reply the ack. Thanks for your work. > Actually I was thinking about the same -- setting a bad example and stuff. It's not a big deal, I'll send v2 soon :) > -- > Best Regards, > Samsung Electronics > Chanwoo Choi ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM 2021-10-21 20:31 [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Sam Protsenko 2021-10-21 20:31 ` [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain Sam Protsenko @ 2021-10-22 8:50 ` Krzysztof Kozlowski 2021-10-22 22:02 ` Chanwoo Choi 2 siblings, 0 replies; 10+ messages in thread From: Krzysztof Kozlowski @ 2021-10-22 8:50 UTC (permalink / raw) To: Sam Protsenko, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette Cc: Sumit Semwal, devicetree, linux-arm-kernel, linux-clk, linux-kernel, linux-samsung-soc On 21/10/2021 22:31, Sam Protsenko wrote: > CMU_APM generates clocks for APM IP-core (Active Power Management). In > particular it generates RTC clocks, which are needed to enable rtc-s3c > driver on Exynos850 SoC. > > Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > --- > .../clock/samsung,exynos850-clock.yaml | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM 2021-10-21 20:31 [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Sam Protsenko 2021-10-21 20:31 ` [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain Sam Protsenko 2021-10-22 8:50 ` [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Krzysztof Kozlowski @ 2021-10-22 22:02 ` Chanwoo Choi 2 siblings, 0 replies; 10+ messages in thread From: Chanwoo Choi @ 2021-10-22 22:02 UTC (permalink / raw) To: Sam Protsenko, Krzysztof Kozlowski, Sylwester Nawrocki, Paweł Chmiel, Chanwoo Choi, Tomasz Figa, Rob Herring, Stephen Boyd, Michael Turquette Cc: Sumit Semwal, devicetree, linux-arm-kernel, linux-clk, linux-kernel, linux-samsung-soc On 21. 10. 22. 오전 5:31, Sam Protsenko wrote: > CMU_APM generates clocks for APM IP-core (Active Power Management). In > particular it generates RTC clocks, which are needed to enable rtc-s3c > driver on Exynos850 SoC. > > Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > --- > .../clock/samsung,exynos850-clock.yaml | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml b/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml > index 7f8c91a29b91..5618cfa62f80 100644 > --- a/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml > +++ b/Documentation/devicetree/bindings/clock/samsung,exynos850-clock.yaml > @@ -32,6 +32,7 @@ properties: > compatible: > enum: > - samsung,exynos850-cmu-top > + - samsung,exynos850-cmu-apm > - samsung,exynos850-cmu-core > - samsung,exynos850-cmu-dpu > - samsung,exynos850-cmu-hsi > @@ -68,6 +69,24 @@ allOf: > items: > - const: oscclk > > + - if: > + properties: > + compatible: > + contains: > + const: samsung,exynos850-cmu-apm > + > + then: > + properties: > + clocks: > + items: > + - description: External reference clock (26 MHz) > + - description: CMU_APM bus clock (from CMU_TOP) > + > + clock-names: > + items: > + - const: oscclk > + - const: dout_clkcmu_apm_bus > + > - if: > properties: > compatible: > Acked-by: Chanwoo Choi <cw00.choi@samsung.com> Thanks. -- Best Regards, Samsung Electronics Chanwoo Choi ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-10-22 22:31 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-10-21 20:31 [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Sam Protsenko 2021-10-21 20:31 ` [PATCH 2/2] clk: samsung: exynos850: Implement CMU_APM domain Sam Protsenko 2021-10-22 8:54 ` Krzysztof Kozlowski 2021-10-22 8:58 ` Chanwoo Choi 2021-10-22 13:39 ` Sam Protsenko 2021-10-22 14:08 ` Krzysztof Kozlowski 2021-10-22 22:18 ` Chanwoo Choi 2021-10-22 22:31 ` Sam Protsenko 2021-10-22 8:50 ` [PATCH 1/2] dt-bindings: clock: samsung: Document Exynos850 CMU_APM Krzysztof Kozlowski 2021-10-22 22:02 ` Chanwoo Choi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).