Commit 57c624ae authored by Wei Yongjun's avatar Wei Yongjun Committed by Mark Brown

spi: orion: Fix return value check in orion_spi_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: default avatarWei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent b3c195b3
......@@ -655,8 +655,8 @@ static int orion_spi_probe(struct platform_device *pdev)
spi->direct_access[cs].vaddr = devm_ioremap(&pdev->dev,
r->start,
PAGE_SIZE);
if (IS_ERR(spi->direct_access[cs].vaddr)) {
status = PTR_ERR(spi->direct_access[cs].vaddr);
if (!spi->direct_access[cs].vaddr) {
status = -ENOMEM;
goto out_rel_clk;
}
spi->direct_access[cs].size = PAGE_SIZE;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment