[PATCH] clk: Do not return signed value from clk_get_rate()

Michal Simek michal.simek at amd.com
Wed Dec 17 12:37:27 CET 2025


As it is visible clk_get_rate() is returning ulong:
ulong clk_get_rate(struct clk *clk);

Also struct clk_ops is defining it as
ulong (*get_rate)(struct clk *clk);

That's why return 0 rate when ops->get_rate is not implemented which is
the same value as is return when clock is not valid.

Signed-off-by: Michal Simek <michal.simek at amd.com>
---

Another alternative is obviously also return only long and fix all
get_rate calls.
---
 drivers/clk/clk-uclass.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c
index 4420651f765c..bca818530d4c 100644
--- a/drivers/clk/clk-uclass.c
+++ b/drivers/clk/clk-uclass.c
@@ -490,7 +490,7 @@ ulong clk_get_rate(struct clk *clk)
 	ops = clk_dev_ops(clk->dev);
 
 	if (!ops->get_rate)
-		return -ENOSYS;
+		return 0;
 
 	return ops->get_rate(clk);
 }
-- 
2.43.0

base-commit: e1082054610fe73d4487d12f4274315030592c77
branch: debian-sent3


More information about the U-Boot mailing list