clk: fixup argument order when setting VCO parameters

2f9f64bc · Jonathan Austin <[email protected]> · 2013-07-23 16:42:18 +01:00

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 <[email protected]>
Signed-off-by: Jonathan Austin <[email protected]>
Signed-off-by: Mike Turquette <[email protected]>
Cc: [email protected]

Commit: 2f9f64bc5aa31836810cd25301aa4772ad73ebab
Parents: 79a2e998

Changes

1 files changed · +1 −1

File	Status	Add	Del	Content
drivers/clk/versatile/clk-icst.c	modified	+1	−1	View file

drivers/clk/versatile/clk-icst.c

View file

diff --git a/drivers/clk/versatile/clk-icst.c b/drivers/clk/versatile/clk-icst.c
index 67ccf4a..f5e4c21 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;
 }
 