diff options
author | Boris Brezillon <boris.brezillon@free-electrons.com> | 2016-07-18 09:49:12 +0200 |
---|---|---|
committer | Michael Turquette <mturquette@baylibre.com> | 2016-07-18 17:45:41 -0700 |
commit | f96423f483b1a7854270335b319e8d1cdd6f3585 (patch) | |
tree | 15cbcba2a4262a9c78953ce9a2cb9bd8322bedfc /drivers/clk/at91 | |
parent | 9d6928639134cff896a92f8ff7fad14370e88d3c (diff) | |
download | op-kernel-dev-f96423f483b1a7854270335b319e8d1cdd6f3585.zip op-kernel-dev-f96423f483b1a7854270335b319e8d1cdd6f3585.tar.gz |
clk: at91: fix clk_programmable_set_parent()
Since commit 1bdf02326b71e ("clk: at91: make use of syscon/regmap
internally"), clk_programmable_set_parent() is always selecting the
first parent (AKA slow_clk), no matter what's passed in the 'index'
parameter.
Fix that by initializing the pckr variable to the index value.
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Reported-by: Hans Verkuil <hans.verkuil@cisco.com>
Fixes: 1bdf02326b71e ("clk: at91: make use of syscon/regmap internally")
Cc: <stable@vger.kernel.org>
Signed-off-by: Michael Turquette <mturquette@baylibre.com>
Link: lkml.kernel.org/r/1468828152-18389-1-git-send-email-boris.brezillon@free-electrons.com
Diffstat (limited to 'drivers/clk/at91')
-rw-r--r-- | drivers/clk/at91/clk-programmable.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/clk/at91/clk-programmable.c b/drivers/clk/at91/clk-programmable.c index 10f846c..25d5906 100644 --- a/drivers/clk/at91/clk-programmable.c +++ b/drivers/clk/at91/clk-programmable.c @@ -99,7 +99,7 @@ static int clk_programmable_set_parent(struct clk_hw *hw, u8 index) struct clk_programmable *prog = to_clk_programmable(hw); const struct clk_programmable_layout *layout = prog->layout; unsigned int mask = layout->css_mask; - unsigned int pckr = 0; + unsigned int pckr = index; if (layout->have_slck_mck) mask |= AT91_PMC_CSSMCK_MCK; |