diff options
| author | Vasiliy Kulikov <segoon@openwall.com> | 2010-11-22 18:59:13 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2010-11-22 20:26:44 +0000 |
| commit | b3915d1fb6557dda206f4644ba9aa96ffd9a99d2 (patch) | |
| tree | e65a256f822fe3774d9a714aef55734c1a9fa53a /sound/soc/atmel | |
| parent | 13a2e06c5898d27aadabfdb9830169101b21432f (diff) | |
| download | talos-op-linux-b3915d1fb6557dda206f4644ba9aa96ffd9a99d2.tar.gz talos-op-linux-b3915d1fb6557dda206f4644ba9aa96ffd9a99d2.zip | |
ASoC: atmel: test wrong variable
After clk_get() mclk is checked second time instead of pllb check.
In patch v1 Jarkko Nikula noticed that PTR_ERR() is also has wrong argument.
Signed-off-by: Vasiliy Kulikov <segoon@openwall.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/atmel')
| -rw-r--r-- | sound/soc/atmel/sam9g20_wm8731.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/soc/atmel/sam9g20_wm8731.c b/sound/soc/atmel/sam9g20_wm8731.c index 293569dfd0ed..032e17dd8fdb 100644 --- a/sound/soc/atmel/sam9g20_wm8731.c +++ b/sound/soc/atmel/sam9g20_wm8731.c @@ -222,9 +222,9 @@ static int __init at91sam9g20ek_init(void) } pllb = clk_get(NULL, "pllb"); - if (IS_ERR(mclk)) { + if (IS_ERR(pllb)) { printk(KERN_ERR "ASoC: Failed to get PLLB\n"); - ret = PTR_ERR(mclk); + ret = PTR_ERR(pllb); goto err_mclk; } ret = clk_set_parent(mclk, pllb); |
