diff options
| author | Martin Blumenstingl <martin.blumenstingl@googlemail.com> | 2020-12-26 13:15:56 +0100 |
|---|---|---|
| committer | Jerome Brunet <jbrunet@baylibre.com> | 2021-01-04 11:42:43 +0100 |
| commit | ccdc1f0836f8e37b558a424f1e491f929b2e7ede (patch) | |
| tree | 4d24d76e03369dfe664e2638b3903717f184f840 | |
| parent | 9e717285f0bd591d716fa0e7418f2cdaf756dd25 (diff) | |
clk: meson: clk-pll: propagate the error from meson_clk_pll_set_rate()
Popagate the error code from meson_clk_pll_set_rate() when the PLL does
not lock with the new settings.
Fixes: 722825dcd54b2e ("clk: meson: migrate plls clocks to clk_regmap")
Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Link: https://lore.kernel.org/r/20201226121556.975418-4-martin.blumenstingl@googlemail.com
| -rw-r--r-- | drivers/clk/meson/clk-pll.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/clk/meson/clk-pll.c b/drivers/clk/meson/clk-pll.c index 5b932976483f..49f27fe53213 100644 --- a/drivers/clk/meson/clk-pll.c +++ b/drivers/clk/meson/clk-pll.c @@ -394,7 +394,8 @@ static int meson_clk_pll_set_rate(struct clk_hw *hw, unsigned long rate, if (!enabled) return 0; - if (meson_clk_pll_enable(hw)) { + ret = meson_clk_pll_enable(hw); + if (ret) { pr_warn("%s: pll did not lock, trying to restore old rate %lu\n", __func__, old_rate); /* @@ -406,7 +407,7 @@ static int meson_clk_pll_set_rate(struct clk_hw *hw, unsigned long rate, meson_clk_pll_set_rate(hw, old_rate, parent_rate); } - return 0; + return ret; } /* |