diff options
| author | Jonathan Austin <jonathan.austin@arm.com> | 2013-07-23 17:42:18 +0200 |
|---|---|---|
| committer | Mike Turquette <mturquette@linaro.org> | 2013-10-08 08:01:07 +0200 |
| commit | 2f9f64bc5aa31836810cd25301aa4772ad73ebab (patch) | |
| tree | 9ea2482804a3a5705023615e8e012c500d2f7141 /drivers | |
| parent | clk: socfpga: Fix incorrect sdmmc clock name (diff) | |
| download | linux-2f9f64bc5aa31836810cd25301aa4772ad73ebab.tar.xz linux-2f9f64bc5aa31836810cd25301aa4772ad73ebab.zip | |
clk: fixup argument order when setting VCO parameters
The order of arguments in the call to vco_set() for the ICST clocks appears to
have been switched in error, which results in the VCO not being initialised
correctly. This in turn stops the integrated LCD on things like Integrator/CP
from working correctly.
This patch fixes the order and restores the expected functionality.
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Jonathan Austin <jonathan.austin@arm.com>
Signed-off-by: Mike Turquette <mturquette@linaro.org>
Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/clk/versatile/clk-icst.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/clk/versatile/clk-icst.c b/drivers/clk/versatile/clk-icst.c index 67ccf4aa7277..f5e4c21b301f 100644 --- a/drivers/clk/versatile/clk-icst.c +++ b/drivers/clk/versatile/clk-icst.c @@ -107,7 +107,7 @@ static int icst_set_rate(struct clk_hw *hw, unsigned long rate, vco = icst_hz_to_vco(icst->params, rate); icst->rate = icst_hz(icst->params, vco); - vco_set(icst->vcoreg, icst->lockreg, vco); + vco_set(icst->lockreg, icst->vcoreg, vco); return 0; } |