Commit 2da56881 authored by Dan Carpenter's avatar Dan Carpenter Committed by Will Deacon
Browse files

drivers: perf: marvell_cn10k: fix an IS_ERR() vs NULL check 



The devm_ioremap() function does not return error pointers.  It returns
NULL.

Fixes: 036a7584 ("drivers: perf: Add LLC-TAD perf counter support")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20211217145907.GA16611@kili


Signed-off-by: default avatarWill Deacon <will@kernel.org>
parent 527a7f52

drivers/perf/marvell_cn10k_tad_pmu.c

+1 −1
Original line number Diff line number Diff line
@@ -312,7 +312,7 @@ static int tad_pmu_probe(struct platform_device *pdev) 
		regions[i].base = devm_ioremap(&pdev->dev,

					       res->start,

					       tad_pmu_page_size);

		if (IS_ERR(regions[i].base)) {

		if (!regions[i].base) {

			dev_err(&pdev->dev, "TAD%d ioremap fail\n", i);

			return -ENOMEM;

		}