[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