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

Patrice CHOTARD patrice.chotard at foss.st.com
Wed Sep 17 09:26:57 CEST 2025



On 9/8/25 09:02, Patrice CHOTARD wrote:
> 
> 
> 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
Applied to u-boot-stm32/master

Thanks
Patrice


More information about the U-Boot mailing list