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 <[email protected]>
> ---
> Cc: Gabriel Fernandez <[email protected]>
> Cc: Patrice Chotard <[email protected]>
> Cc: Patrick Delaunay <[email protected]>
> Cc: Sean Anderson <[email protected]>
> Cc: Tom Rini <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> ---
> 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 <[email protected]>
Thanks
Patrice
