| From fa3c1846eb14660b82c1b44745066c6635716135 Mon Sep 17 00:00:00 2001 |
| From: Wei Yongjun <yongjun_wei@trendmicro.com.cn> |
| Date: Tue, 30 Jul 2013 07:51:37 +0800 |
| Subject: ASoC: rcar: fix return value check in rsnd_gen1_probe() |
| |
| In case of error, the function devm_ioremap_resource() returns ERR_PTR() |
| and never returns NULL. The NULL test in the return value check should be |
| replaced with IS_ERR(), and also remove the dev_err call to avoid redundant |
| error message. |
| |
| Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> |
| Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> |
| Signed-off-by: Mark Brown <broonie@linaro.org> |
| (cherry picked from commit 70263cb474853c116f80713d468f3c17d805921c) |
| Signed-off-by: Simon Horman <horms+renesas@verge.net.au> |
| --- |
| sound/soc/sh/rcar/gen.c | 17 ++++------------- |
| 1 file changed, 4 insertions(+), 13 deletions(-) |
| |
| diff --git a/sound/soc/sh/rcar/gen.c b/sound/soc/sh/rcar/gen.c |
| index 5e4ae0da4352..61232cd9908f 100644 |
| --- a/sound/soc/sh/rcar/gen.c |
| +++ b/sound/soc/sh/rcar/gen.c |
| @@ -150,25 +150,16 @@ static int rsnd_gen1_probe(struct platform_device *pdev, |
| sru_res = platform_get_resource(pdev, IORESOURCE_MEM, RSND_GEN1_SRU); |
| adg_res = platform_get_resource(pdev, IORESOURCE_MEM, RSND_GEN1_ADG); |
| ssi_res = platform_get_resource(pdev, IORESOURCE_MEM, RSND_GEN1_SSI); |
| - if (!sru_res || |
| - !adg_res || |
| - !ssi_res) { |
| - dev_err(dev, "Not enough SRU/SSI/ADG platform resources.\n"); |
| - return -ENODEV; |
| - } |
| - |
| - gen->ops = &rsnd_gen1_ops; |
| |
| gen->base[RSND_GEN1_SRU] = devm_ioremap_resource(dev, sru_res); |
| gen->base[RSND_GEN1_ADG] = devm_ioremap_resource(dev, adg_res); |
| gen->base[RSND_GEN1_SSI] = devm_ioremap_resource(dev, ssi_res); |
| - if (!gen->base[RSND_GEN1_SRU] || |
| - !gen->base[RSND_GEN1_ADG] || |
| - !gen->base[RSND_GEN1_SSI]) { |
| - dev_err(dev, "SRU/SSI/ADG ioremap failed\n"); |
| + if (IS_ERR(gen->base[RSND_GEN1_SRU]) || |
| + IS_ERR(gen->base[RSND_GEN1_ADG]) || |
| + IS_ERR(gen->base[RSND_GEN1_SSI])) |
| return -ENODEV; |
| - } |
| |
| + gen->ops = &rsnd_gen1_ops; |
| rsnd_gen1_reg_map_init(gen); |
| |
| dev_dbg(dev, "Gen1 device probed\n"); |
| -- |
| 1.8.5.rc3 |
| |