Commit 58c53d43 authored by Lorenzo Bianconi's avatar Lorenzo Bianconi Committed by Stephen Boyd
Browse files

clk: en7523: fix rate divider for slic and spi clocks 



Introduce div_offset field in en_clk_desc struct in order to fix rate
divider estimation in en7523_get_div routine for slic and spi fixed
rate clocks.
Moreover, fix base_shift for crypto clock.

Fixes: 1e627317 ("clk: en7523: Add clock driver for Airoha EN7523 SoC")
Signed-off-by: default avatarLorenzo Bianconi <lorenzo@kernel.org>
Link: https://lore.kernel.org/r/c491bdea05d847f1f1294b94f14725d292eb95d0.1718615934.git.lorenzo@kernel.org


Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: default avatarStephen Boyd <sboyd@kernel.org>
parent bf288bd2

drivers/clk/clk-en7523.c

+7 −2
Original line number Diff line number Diff line
@@ -62,6 +62,7 @@ struct en_clk_desc { 
	u8 div_shift;

	u16 div_val0;

	u8 div_step;

	u8 div_offset;

};



struct en_clk_gate {
@@ -106,6 +107,7 @@ static const struct en_clk_desc en7523_base_clks[] = { 
		.div_bits = 3,

		.div_shift = 0,

		.div_step = 1,

		.div_offset = 1,

	}, {

		.id = EN7523_CLK_EMI,

		.name = "emi",
@@ -119,6 +121,7 @@ static const struct en_clk_desc en7523_base_clks[] = { 
		.div_bits = 3,

		.div_shift = 0,

		.div_step = 1,

		.div_offset = 1,

	}, {

		.id = EN7523_CLK_BUS,

		.name = "bus",
@@ -132,6 +135,7 @@ static const struct en_clk_desc en7523_base_clks[] = { 
		.div_bits = 3,

		.div_shift = 0,

		.div_step = 1,

		.div_offset = 1,

	}, {

		.id = EN7523_CLK_SLIC,

		.name = "slic",
@@ -172,13 +176,14 @@ static const struct en_clk_desc en7523_base_clks[] = { 
		.div_bits = 3,

		.div_shift = 0,

		.div_step = 1,

		.div_offset = 1,

	}, {

		.id = EN7523_CLK_CRYPTO,

		.name = "crypto",



		.base_reg = REG_CRYPTO_CLKSRC,

		.base_bits = 1,

		.base_shift = 8,

		.base_shift = 0,

		.base_values = emi_base,

		.n_base_values = ARRAY_SIZE(emi_base),

	}
@@ -281,7 +286,7 @@ static u32 en7523_get_div(void __iomem *base, int i) 
	if (!val && desc->div_val0)

		return desc->div_val0;



	return (val + 1) * desc->div_step;

	return (val + desc->div_offset) * desc->div_step;

}



static int en7523_pci_is_enabled(struct clk_hw *hw)