u-boot.git/drivers/clk/clk-uclass.c, branch master

clk: clk-uclass: used dev name in debug message

2026-01-21T19:25:40+00:00

Consistently use the device name in debug messages. The clk-uclass file
previously had a mix of printing the dev pointer and the device name.
Changing all to use the device name makes the debug messages more
useful.

Signed-off-by: David Lechner

clk: clk-uclass: fix format specifier for ofnode name

2026-01-21T19:25:40+00:00

Change the format specifier from %p to %s when printing the ofnode name
so that the actual name is printed instead of the pointer address.

Signed-off-by: David Lechner

Revert "clk: Return value calculated by ERR_PTR"

2025-12-12T14:00:54+00:00

While this change was intended to fix a mistake in the code, of calling
the ERR_PTR macro but not making use of the result, it seems that
functionally platforms depend on the loop not existing here. The TI K3
families of platforms for example were broken by this commit.

This reverts commit fe780310cfa8bf5a093894b5cd7fe85c6b02fd91.

Reported-by: Nishanth Menon 
Reviewed-by: Andrew Goodbody 
Reviewed-by: Nishanth Menon 
Signed-off-by: Tom Rini

Merge tag 'v2026.01-rc4' into next

2025-12-08T19:17:27+00:00

Prepare v2026.01-rc4

Merge patch series "clk: Return value calculated by ERR_PTR"

2025-12-05T23:03:36+00:00

Andrew Goodbody  says:

Smatch reported an error where a value calculated by ERR_PTR was not
used. Fixing this to return the generated value led to a test failure
which meant updating the sandbox clock code so that it would still cause
the tests to pass with the above correction.
Debugging this problem led to a SIGSEGV which is addressed in 1/3.
Possible memory leaks noticed are addressed in 3/3.

Link: https://lore.kernel.org/r/20251121-clk_uclass_fix-v2-0-74f4ea10e194@linaro.org

clk: Prevent memory leak on error

2025-12-05T23:03:31+00:00

In clk_set_default_rates() memory is allocated to store the clock rates
that are read. Direct returns fail to free this memory leading to a
memory leak so instead use 'goto fail;' which will then perform the free
before exiting the function.

Signed-off-by: Andrew Goodbody

clk: Return value calculated by ERR_PTR

2025-12-05T23:03:31+00:00

In clk_set_default_get_by_id ret is passed to ERR_PTR but nothing is
done with the value that this calculates which is obviously not the
intention of the code. This is confirmed by the code around where this
function is called.
Instead return the value from ERR_PTR.

Then fixup the sandbox code so that the test dm_test_clk does not fail
as it relied on the broken behaviour.

Finally disable part of the test that does not work correctly with
CLK_AUTO_ID

This issue found by Smatch.

Signed-off-by: Andrew Goodbody

clk: Prevent SIGSEGV on debug

2025-12-05T23:03:31+00:00

If LOG_DEBUG is defined and a NULL clk is passed to clk_enable or
clk_disable then an attempt is made to dereference NULL in the debug
statement. Guard against this.

Signed-off-by: Andrew Goodbody 
Reviewed-by: Patrick Delaunay

clk: Only enable the parent clock if the clock was enabled before reparenting

2025-12-04T15:39:26+00:00

The current implementation of clk_set_parent() unconditionally enables
the new parent clock, even if the target clock was not previously enabled.

To avoid this implicit behavior, this patch adds a check for whether
the target clock has been enabled before parent enabling..

Fixes: ac30d90f336 ("clk: Ensure the parent clocks are enabled while reparenting")
Signed-off-by: Maksim Kiselev 
Reviewed-by: Miquel Raynal

clk: use private clk struct to access clock flags

2025-11-12T04:56:12+00:00

There may be cases where the flags set for a clock is not available.
This is usually the case with clocks which have been retrieved using
clk_request(). However, clock flags are found in their respective
private clock struct, so use that instead.

Signed-off-by: Kaustabh Chakraborty 
Signed-off-by: Minkyu Kang