u-boot.git/include/clk.h, branch v2022.07

clk: Make clk_free return void

2022-03-30T17:02:55+00:00

Most callers of this function do not check the return value, and it is
unclear what action they should take if it fails. If a function is freeing
multiple clocks, it should not stop just because the first one failed.
Since the callbacks can no longer fail, just convert the return type to
void.

Signed-off-by: Sean Anderson 
Link: https://lore.kernel.org/r/20220115222504.617013-8-seanga2@gmail.com

clk: Add clk_get_by_name_optional

2022-02-25T06:41:04+00:00

This adds a helper function for clk_get_by_name in cases where the clock is
optional. Hopefully this helps point driver writers in the right direction.
Also convert some existing users.

Signed-off-by: Sean Anderson 
Reviewed-by: Neil Armstrong 
Reviewed-by: Simon Glass 
Link: https://lore.kernel.org/r/20220115205247.566210-2-seanga2@gmail.com

clk: Add client API to HTML docs

2022-02-25T06:41:04+00:00

This converts the existing client (aka clk.h) documentation to kernel doc
format, and adds it to the HTML docs. I have tried to preserve existing
comments as much as possible, refraining from semantic changes.

Signed-off-by: Sean Anderson 
Tested-by: Heinrich Schuchardt 
Reviewed-by: Simon Glass 
Link: https://lore.kernel.org/r/20211222171114.3091780-4-seanga2@gmail.com
[rebased onto u-boot/master and resolved conflicts]
Signed-off-by: Sean Anderson

clk: Inline clk_get_*_optional

2022-02-25T04:58:13+00:00

The optional varients of clk_get_* functions are just simple wrappers.
Reduce code size a bit by inlining them. On platforms where it is not used
(most of them), it will not be compiled in any more. On platforms where
they are used, the inlined branch should not cause any significant growth.

Signed-off-by: Sean Anderson 
Reviewed-by: Simon Glass 
Link: https://lore.kernel.org/r/20211222171114.3091780-3-seanga2@gmail.com

clk: Rename clk_get_optional_nodev

2022-02-25T04:58:13+00:00

This normalizes the name of this accessor function to put "_optional" last.

Signed-off-by: Sean Anderson 
Reviewed-by: Simon Glass 
Link: https://lore.kernel.org/r/20211222171114.3091780-2-seanga2@gmail.com

doc: replace @return by Return:

2022-01-19T17:11:34+00:00

Sphinx expects Return: and not @return to indicate a return value.

find . -name '*.c' -exec \
sed -i 's/^\(\s\)\*\(\s*\)@return\(\s\)/\1*\2Return:\3/' {} \;

find . -name '*.h' -exec \
sed -i 's/^\(\s\)\*\(\s*\)@return\(\s\)/\1*\2Return:\3/' {} \;

Signed-off-by: Heinrich Schuchardt

clk: fix clk_get_rate() documentation

2021-12-15T17:16:15+00:00

Improve clk_get_rate() @return documentation that otherwise is a bit
ambiguous. At the moment I expect to return 0 as error since the return
type is 'ulong', instead the function really returns negative value in
case the corresponding function pointer is null and returns 0 if the clock
is invalid.

Signed-off-by: Giulio Benetti 
Reviewed-by: Sean Anderson

clk: Rename clk_get_by_driver_info()

2021-09-25T15:46:15+00:00

This is actually a misnomer now, since the phandle info may contain
a driver_info index or a udevice index. Rename it to use the word
'phandle', which seems more accurate. Add a comment while we are here.

Also add a test for this function.

Signed-off-by: Simon Glass

treewide: Simply conditions with the new OF_REAL

2021-09-25T15:46:15+00:00

Use this new Kconfig to simplify the compilation conditions where
appropriate.

Signed-off-by: Simon Glass

clk: Allow force setting clock defaults before relocation

2021-06-17T01:40:57+00:00

Since 291da96b8e ("clk: Allow clock defaults to be set during re-reloc
state for SPL only") it has been impossible to set clock defaults before
relocation. This is annoying on boards without SPL, since there is no way
to set clock defaults before U-Boot proper. In particular, the aisram rate
must be changed before relocation on the K210, since U-Boot will hang if we
try and change the rate while we are using aisram.

To get around this, extend the stage parameter to allow force setting
defaults, even if they would be otherwise postponed for later. A device
tree property was decided against because of the concerns in the original
commit thread about the overhead of repeatedly parsing the device tree.

Signed-off-by: Sean Anderson 
Reviewed-by: Simon Glass