diff options
author | 2020-05-27 03:02:10 +0000 | |
---|---|---|
committer | 2020-05-27 14:48:56 +0100 | |
commit | 185457632ba344d3100e6bdd8ba839b959521813 (patch) | |
tree | d98a69bd4547ddc201317a732135a391265ff123 /sound/soc/pxa | |
parent | ASoC: tlv320adcx140: Fix warnings when using W=1 (diff) | |
download | linux-185457632ba344d3100e6bdd8ba839b959521813.tar.gz linux-185457632ba344d3100e6bdd8ba839b959521813.tar.bz2 linux-185457632ba344d3100e6bdd8ba839b959521813.zip |
ASoC: mmp-sspa: Fix return value check in asoc_mmp_sspa_probe()
In case of error, the function devm_ioremap() returns NULL pointer not
ERR_PTR(). The IS_ERR() test in the return value check should be
replaced with NULL test.
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Link: https://lore.kernel.org/r/20200527030210.124393-1-weiyongjun1@huawei.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/pxa')
-rw-r--r-- | sound/soc/pxa/mmp-sspa.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sound/soc/pxa/mmp-sspa.c b/sound/soc/pxa/mmp-sspa.c index 3e37ab625f8d..4255851c71c1 100644 --- a/sound/soc/pxa/mmp-sspa.c +++ b/sound/soc/pxa/mmp-sspa.c @@ -493,13 +493,13 @@ static int asoc_mmp_sspa_probe(struct platform_device *pdev) return -ENODEV; sspa->rx_base = devm_ioremap(&pdev->dev, res->start, 0x30); - if (IS_ERR(sspa->rx_base)) - return PTR_ERR(sspa->rx_base); + if (!sspa->rx_base) + return -ENOMEM; sspa->tx_base = devm_ioremap(&pdev->dev, res->start + 0x80, 0x30); - if (IS_ERR(sspa->tx_base)) - return PTR_ERR(sspa->tx_base); + if (!sspa->tx_base) + return -ENOMEM; sspa->clk = devm_clk_get(&pdev->dev, NULL); if (IS_ERR(sspa->clk)) |