On 6/23/23 15:05, Patrick Delaunay wrote:

To disabled a clock in clock tree initialization for a mux of STM32MP15, the selected clock source index is set with the latest possible index for the number of bit used. Today this valid configuration cause a error in U-Boot messages, for example with CLK_ETH_DISABLED, when this clock is not needed for the used ETH PHY without crystal:

no parents defined for clk id 123

This patch change the level of this message to avoid this trace for valid clock tree.

Signed-off-by: Patrick Delaunay patrick.delaunay@foss.st.com

drivers/clk/stm32/clk-stm32mp1.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/stm32/clk-stm32mp1.c b/drivers/clk/stm32/clk-stm32mp1.c index 4f4524fcb2c6..615028769495 100644 --- a/drivers/clk/stm32/clk-stm32mp1.c +++ b/drivers/clk/stm32/clk-stm32mp1.c @@ -881,7 +881,8 @@ static int stm32mp1_clk_get_parent(struct stm32mp1_clk_priv *priv, return sel[s].parent[p]; }

• log_err("no parents defined for clk id %d\n", (u32)id);

• /* clock is DISABLED when the clock src is not in clk_parent[] range */

• log_debug("no parents defined for clk id %d\n", (u32)id);

  return -EINVAL;

}

Reviewed-by: Patrice Chotard patrice.chotard@foss.st.com

Thanks Patrice