OSDN Git Service

clk: tegra: Remove needless initializations
authorThierry Reding <treding@nvidia.com>
Wed, 18 Feb 2015 15:25:16 +0000 (16:25 +0100)
committerThierry Reding <treding@nvidia.com>
Fri, 10 Apr 2015 14:04:18 +0000 (16:04 +0200)
The ret variable is often explicitly initialized to 0, but there is no
need to do so in many cases because it will immediately be overwritten
with the return value from a function.

Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/clk/tegra/clk-pll.c

index f9950dd..05c6d08 100644 (file)
@@ -981,7 +981,7 @@ static int clk_pllxc_set_rate(struct clk_hw *hw, unsigned long rate,
        struct tegra_clk_pll *pll = to_clk_pll(hw);
        struct tegra_clk_pll_freq_table cfg, old_cfg;
        unsigned long flags = 0;
-       int ret = 0;
+       int ret;
 
        ret = _pll_ramp_calc_pll(hw, &cfg, rate, parent_rate);
        if (ret < 0)
@@ -1005,7 +1005,7 @@ static long clk_pll_ramp_round_rate(struct clk_hw *hw, unsigned long rate,
                                unsigned long *prate)
 {
        struct tegra_clk_pll_freq_table cfg;
-       int ret = 0, p_div;
+       int ret, p_div;
        u64 output_rate = *prate;
 
        ret = _pll_ramp_calc_pll(hw, &cfg, rate, *prate);
@@ -1073,7 +1073,7 @@ static int clk_pllc_enable(struct clk_hw *hw)
 {
        struct tegra_clk_pll *pll = to_clk_pll(hw);
        u32 val;
-       int ret = 0;
+       int ret;
        unsigned long flags = 0;
 
        if (pll->lock)