[PATCH] clk: stm32: Pass udevice pointer to clk_register_composite()

Patrice CHOTARD patrice.chotard at foss.st.com
Mon Sep 8 09:02:35 CEST 2025



On 9/7/25 01:00, Marek Vasut wrote:
> The clk_register_composite() does clk_resolve_parent_clk() look up,
> which requires valid udevice pointer. Do not pass NULL, pass a valid
> device pointer to prevent hang on registering ck_usbo_48m clock on
> STM32MP13xx.
> 
> Signed-off-by: Marek Vasut <marek.vasut at mailbox.org>
> ---
> Cc: Gabriel Fernandez <gabriel.fernandez at foss.st.com>
> Cc: Patrice Chotard <patrice.chotard at foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay at foss.st.com>
> Cc: Sean Anderson <seanga2 at gmail.com>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: u-boot at lists.denx.de
> Cc: uboot-stm32 at st-md-mailman.stormreply.com
> ---
>  drivers/clk/stm32/clk-stm32-core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/stm32/clk-stm32-core.c b/drivers/clk/stm32/clk-stm32-core.c
> index a0ae89d0912..858f122db1a 100644
> --- a/drivers/clk/stm32/clk-stm32-core.c
> +++ b/drivers/clk/stm32/clk-stm32-core.c
> @@ -309,7 +309,7 @@ clk_stm32_register_composite(struct udevice *dev,
>  		gate_ops = &clk_stm32_gate_ops;
>  	}
>  
> -	clk = clk_register_composite(NULL, cfg->name,
> +	clk = clk_register_composite(dev, cfg->name,
>  				     parent_names, num_parents,
>  				     mux_clk, mux_ops,
>  				     div_clk, div_ops,

Reviewed-by: Patrice Chotard <patrice.chotard at foss.st.com>

Thanks
Patrice


More information about the U-Boot mailing list