[PATCH 2/6] clk: divider: add error code propagation
Jules Maselbas
jmaselbas at zdiv.net
Thu Nov 7 06:57:18 PST 2024
divider_get_val can return a negative error code and should not be used
as a valid divider.
Signed-off-by: Jules Maselbas <jmaselbas at zdiv.net>
---
drivers/clk/clk-divider.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/clk-divider.c b/drivers/clk/clk-divider.c
index ccab70aecc..b05b042da5 100644
--- a/drivers/clk/clk-divider.c
+++ b/drivers/clk/clk-divider.c
@@ -308,6 +308,7 @@ static int clk_divider_set_rate(struct clk_hw *hw, unsigned long rate,
struct clk *clk = clk_hw_to_clk(hw);
struct clk_divider *divider = to_clk_divider(hw);
unsigned int value;
+ int ret;
u32 val;
if (divider->flags & CLK_DIVIDER_READ_ONLY)
@@ -320,8 +321,10 @@ static int clk_divider_set_rate(struct clk_hw *hw, unsigned long rate,
clk_set_rate(clk_get_parent(clk), parent_rate);
}
- value = divider_get_val(rate, parent_rate, divider->table,
+ value = ret = divider_get_val(rate, parent_rate, divider->table,
divider->width, divider->flags);
+ if (ret < 0)
+ return ret;
val = readl(divider->reg);
val &= ~(clk_div_mask(divider->width) << divider->shift);
--
2.46.2
More information about the barebox
mailing list