diff options
author | Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> | 2017-03-29 21:49:06 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-03-30 17:22:32 -0400 |
commit | d7f298197a22f11b38059f257842dac7c30a564c (patch) | |
tree | 999c45b884228b6f9601e92e71e0e1d94219a76d | |
parent | 9986943ef5d61a9bea3c86000d91d3b789f0060e (diff) |
ASoC: rcar: fixup of_clk_add_provider() usage for multi clkout
Current adg is calling of_clk_add_povider() multiple times,
but it is not correct usage. This patch fixup its parameter
and call it once.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | sound/soc/sh/rcar/adg.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/sound/soc/sh/rcar/adg.c b/sound/soc/sh/rcar/adg.c index 85a33ac0a5c4..56107454bdb3 100644 --- a/sound/soc/sh/rcar/adg.c +++ b/sound/soc/sh/rcar/adg.c | |||
@@ -537,16 +537,14 @@ static void rsnd_adg_get_clkout(struct rsnd_priv *priv, | |||
537 | clk = clk_register_fixed_rate(dev, clkout_name[i], | 537 | clk = clk_register_fixed_rate(dev, clkout_name[i], |
538 | parent_clk_name, 0, | 538 | parent_clk_name, 0, |
539 | req_rate); | 539 | req_rate); |
540 | if (!IS_ERR(clk)) { | 540 | adg->clkout[i] = ERR_PTR(-ENOENT); |
541 | adg->onecell.clks = adg->clkout; | 541 | if (!IS_ERR(clk)) |
542 | adg->onecell.clk_num = CLKOUTMAX; | ||
543 | |||
544 | adg->clkout[i] = clk; | 542 | adg->clkout[i] = clk; |
545 | |||
546 | of_clk_add_provider(np, of_clk_src_onecell_get, | ||
547 | &adg->onecell); | ||
548 | } | ||
549 | } | 543 | } |
544 | adg->onecell.clks = adg->clkout; | ||
545 | adg->onecell.clk_num = CLKOUTMAX; | ||
546 | of_clk_add_provider(np, of_clk_src_onecell_get, | ||
547 | &adg->onecell); | ||
550 | } | 548 | } |
551 | 549 | ||
552 | adg->ckr = ckr; | 550 | adg->ckr = ckr; |