diff options
| author | Yann Gautier <yann.gautier@st.com> | 2026-05-27 15:25:23 +0200 |
|---|---|---|
| committer | Yann Gautier <yann.gautier@st.com> | 2026-06-29 15:31:56 +0200 |
| commit | ddcc8db85b1660c8234be0f26776c471c030cae3 (patch) | |
| tree | c6b9f2d1eff36b73f9c2ba6f13b2413fbd2de992 | |
| parent | cc050178952812ffa541a232f775b74b4c2b56d5 (diff) | |
| download | arm-trusted-firmware-ddcc8db85b1660c8234be0f26776c471c030cae3.tar.gz arm-trusted-firmware-ddcc8db85b1660c8234be0f26776c471c030cae3.zip | |
fix(st-clock): correct some STM32MP15 clock driver function
The function clk_mux_get_parent() is not used in this driver, remove it.
Set clk_stm32_set_div() static as it is only used in this file for
STM32MP15.
This corrects the following warnings, when compiling with W=2:
drivers/st/clk/stm32mp1_clk.c:214:5: warning: no previous prototype
for 'clk_mux_get_parent' [-Wmissing-prototypes]
214 | int clk_mux_get_parent(struct stm32_clk_priv *priv, uint32_t
mux_id)
| ^~~~~~~~~~~~~~~~~~
drivers/st/clk/stm32mp1_clk.c:267:5: warning: no previous prototype for
'clk_stm32_set_div' [-Wmissing-prototypes]
267 | int clk_stm32_set_div(struct stm32_clk_priv *priv,
uint32_t div_id, uint32_t value)
| ^~~~~~~~~~~~~~~~~
Change-Id: I60f76ff0c7c591c5907caa08260987139bb57b7e
Signed-off-by: Yann Gautier <yann.gautier@st.com>
| -rw-r--r-- | drivers/st/clk/stm32mp1_clk.c | 18 |
1 files changed, 1 insertions, 17 deletions
diff --git a/drivers/st/clk/stm32mp1_clk.c b/drivers/st/clk/stm32mp1_clk.c index 45bdc88ba..1c50d75da 100644 --- a/drivers/st/clk/stm32mp1_clk.c +++ b/drivers/st/clk/stm32mp1_clk.c @@ -211,22 +211,6 @@ static const struct mux_cfg parent_mp15[MUX_NB] = { #define MASK_WIDTH_SHIFT(_width, _shift) \ GENMASK(((_width) + (_shift) - 1U), (_shift)) -int clk_mux_get_parent(struct stm32_clk_priv *priv, uint32_t mux_id) -{ - const struct mux_cfg *mux; - uint32_t mask; - - if (mux_id >= priv->nb_parents) { - panic(); - } - - mux = &priv->parents[mux_id]; - - mask = MASK_WIDTH_SHIFT(mux->width, mux->shift); - - return (mmio_read_32(priv->base + mux->offset) & mask) >> mux->shift; -} - static int clk_mux_set_parent(struct stm32_clk_priv *priv, uint16_t pid, uint8_t sel) { const struct mux_cfg *mux = &priv->parents[pid]; @@ -264,7 +248,7 @@ static int stm32_clk_configure_mux(struct stm32_clk_priv *priv, uint32_t val) return clk_mux_set_parent(priv, mux, sel); } -int clk_stm32_set_div(struct stm32_clk_priv *priv, uint32_t div_id, uint32_t value) +static int clk_stm32_set_div(struct stm32_clk_priv *priv, uint32_t div_id, uint32_t value) { const struct div_cfg *divider; uintptr_t address; |
