diff options
author | Tang Bin <tangbin@cmss.chinamobile.com> | 2020-04-02 15:03:30 +0300 |
---|---|---|
committer | Will Deacon <will@kernel.org> | 2020-04-30 23:49:32 +0300 |
commit | 1f0d97bb7082ead69acc899cd5dae9deb8b91577 (patch) | |
tree | 8327a5af9454b34d31715e79128641eae4fd245a /drivers/perf/arm_spe_pmu.c | |
parent | 5810f00ade49f4710a9e834db5a3ad786a76e880 (diff) | |
download | linux-1f0d97bb7082ead69acc899cd5dae9deb8b91577.tar.xz |
drivers/perf: arm_spe_pmu: Avoid duplicate printouts
platform_get_irq() already screams on failure, so the redundant call to
dev_err() can be removed.
Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
Link: https://lore.kernel.org/r/20200402120330.19468-1-tangbin@cmss.chinamobile.com
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'drivers/perf/arm_spe_pmu.c')
-rw-r--r-- | drivers/perf/arm_spe_pmu.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/perf/arm_spe_pmu.c b/drivers/perf/arm_spe_pmu.c index b72c04852599..58a710eb4e5a 100644 --- a/drivers/perf/arm_spe_pmu.c +++ b/drivers/perf/arm_spe_pmu.c @@ -1133,10 +1133,8 @@ static int arm_spe_pmu_irq_probe(struct arm_spe_pmu *spe_pmu) struct platform_device *pdev = spe_pmu->pdev; int irq = platform_get_irq(pdev, 0); - if (irq < 0) { - dev_err(&pdev->dev, "failed to get IRQ (%d)\n", irq); + if (irq < 0) return -ENXIO; - } if (!irq_is_percpu(irq)) { dev_err(&pdev->dev, "expected PPI but got SPI (%d)\n", irq); |