1 From f98eded9e9ab048c88ff59c5523e703a6ced5523 Mon Sep 17 00:00:00 2001
2 From: Lorenzo Bianconi <lorenzo@kernel.org>
3 Date: Tue, 12 Nov 2024 01:08:52 +0100
4 Subject: [PATCH 4/6] clk: en7523: fix estimation of fixed rate for EN7581
6 Introduce en7581_base_clks array in order to define per-SoC fixed-rate
7 clock parameters and fix wrong parameters for emi, npu and crypto EN7581
10 Fixes: 66bc47326ce2 ("clk: en7523: Add EN7581 support")
11 Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
12 Link: https://lore.kernel.org/r/20241112-clk-en7581-syscon-v2-5-8ada5e394ae4@kernel.org
13 Signed-off-by: Stephen Boyd <sboyd@kernel.org>
15 drivers/clk/clk-en7523.c | 105 ++++++++++++++++++++++++++++++++++++++-
16 1 file changed, 103 insertions(+), 2 deletions(-)
18 --- a/drivers/clk/clk-en7523.c
19 +++ b/drivers/clk/clk-en7523.c
21 #define REG_NP_SCU_SSTR 0x9c
22 #define REG_PCIE_XSI0_SEL_MASK GENMASK(14, 13)
23 #define REG_PCIE_XSI1_SEL_MASK GENMASK(12, 11)
24 +#define REG_CRYPTO_CLKSRC2 0x20c
26 #define REG_RST_CTRL2 0x00
27 #define REG_RST_CTRL1 0x04
28 @@ -89,6 +90,10 @@ static const u32 emi_base[] = { 33300000
29 static const u32 bus_base[] = { 500000000, 540000000 };
30 static const u32 slic_base[] = { 100000000, 3125000 };
31 static const u32 npu_base[] = { 333000000, 400000000, 500000000 };
33 +static const u32 emi7581_base[] = { 540000000, 480000000, 400000000, 300000000 };
34 +static const u32 npu7581_base[] = { 800000000, 750000000, 720000000, 600000000 };
35 +static const u32 crypto_base[] = { 540000000, 480000000 };
37 static const struct en_clk_desc en7523_base_clks[] = {
39 @@ -186,6 +191,102 @@ static const struct en_clk_desc en7523_b
43 +static const struct en_clk_desc en7581_base_clks[] = {
45 + .id = EN7523_CLK_GSW,
48 + .base_reg = REG_GSW_CLK_DIV_SEL,
51 + .base_values = gsw_base,
52 + .n_base_values = ARRAY_SIZE(gsw_base),
59 + .id = EN7523_CLK_EMI,
62 + .base_reg = REG_EMI_CLK_DIV_SEL,
65 + .base_values = emi7581_base,
66 + .n_base_values = ARRAY_SIZE(emi7581_base),
73 + .id = EN7523_CLK_BUS,
76 + .base_reg = REG_BUS_CLK_DIV_SEL,
79 + .base_values = bus_base,
80 + .n_base_values = ARRAY_SIZE(bus_base),
87 + .id = EN7523_CLK_SLIC,
90 + .base_reg = REG_SPI_CLK_FREQ_SEL,
93 + .base_values = slic_base,
94 + .n_base_values = ARRAY_SIZE(slic_base),
96 + .div_reg = REG_SPI_CLK_DIV_SEL,
102 + .id = EN7523_CLK_SPI,
105 + .base_reg = REG_SPI_CLK_DIV_SEL,
107 + .base_value = 400000000,
114 + .id = EN7523_CLK_NPU,
117 + .base_reg = REG_NPU_CLK_DIV_SEL,
120 + .base_values = npu7581_base,
121 + .n_base_values = ARRAY_SIZE(npu7581_base),
128 + .id = EN7523_CLK_CRYPTO,
131 + .base_reg = REG_CRYPTO_CLKSRC2,
134 + .base_values = crypto_base,
135 + .n_base_values = ARRAY_SIZE(crypto_base),
139 static const u16 en7581_rst_ofs[] = {
142 @@ -457,8 +558,8 @@ static void en7581_register_clocks(struc
146 - for (i = 0; i < ARRAY_SIZE(en7523_base_clks); i++) {
147 - const struct en_clk_desc *desc = &en7523_base_clks[i];
148 + for (i = 0; i < ARRAY_SIZE(en7581_base_clks); i++) {
149 + const struct en_clk_desc *desc = &en7581_base_clks[i];
150 u32 val, reg = desc->div_reg ? desc->div_reg : desc->base_reg;