diff options
author | Nishanth Menon <nm@ti.com> | 2014-08-18 18:56:55 +0200 |
---|---|---|
committer | Tero Kristo <t-kristo@ti.com> | 2014-08-21 17:04:16 +0200 |
commit | 20411dad75ece9a613af715df4489e60990c4017 (patch) | |
tree | a9d8dfbb92732c28b24737916f2aaf84bb703aa0 /drivers/clk/ti | |
parent | clk: ti: divider: Provide error check for incoming parameters in set_rate (diff) | |
download | linux-20411dad75ece9a613af715df4489e60990c4017.tar.xz linux-20411dad75ece9a613af715df4489e60990c4017.zip |
clk: ti: dra7-atl: Provide error check for incoming parameters in set_rate
Check for valid parameters in check rate. Else, we end up getting
errors.
This occurs as part of the inital clock tree update of child clock
nodes where new_rate could be 0 for non functional clocks.
Fixes: 9ac33b0ce81fa48 (" CLK: TI: Driver for DRA7 ATL (Audio Tracking Logic)")
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Tero Kristo <t-kristo@ti.com>
Diffstat (limited to 'drivers/clk/ti')
-rw-r--r-- | drivers/clk/ti/clk-dra7-atl.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c index 4a65b410e4d5..af29359677da 100644 --- a/drivers/clk/ti/clk-dra7-atl.c +++ b/drivers/clk/ti/clk-dra7-atl.c @@ -139,9 +139,13 @@ static long atl_clk_round_rate(struct clk_hw *hw, unsigned long rate, static int atl_clk_set_rate(struct clk_hw *hw, unsigned long rate, unsigned long parent_rate) { - struct dra7_atl_desc *cdesc = to_atl_desc(hw); + struct dra7_atl_desc *cdesc; u32 divider; + if (!hw || !rate) + return -EINVAL; + + cdesc = to_atl_desc(hw); divider = ((parent_rate + rate / 2) / rate) - 1; if (divider > DRA7_ATL_DIVIDER_MASK) divider = DRA7_ATL_DIVIDER_MASK; |