clocksource/drivers/sp804: Avoid error on multiple instances
authorAndre Przywara <andre.przywara@arm.com>
Fri, 6 May 2022 16:25:22 +0000 (17:25 +0100)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 18 May 2022 09:08:52 +0000 (11:08 +0200)
commita98399cbc1e05f7b977419f03905501d566cf54e
treee2848fa6b718e7b38b2e315ef9ea5c986917bcfb
parent232ccac1bd9b5bfe73895f527c08623e7fa0752d
clocksource/drivers/sp804: Avoid error on multiple instances

When a machine sports more than one SP804 timer instance, we only bring
up the first one, since multiple timers of the same kind are not useful
to Linux. As this is intentional behaviour, we should not return an
error message, as we do today:
===============
[    0.000800] Failed to initialize '/bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/timer@120000': -22
===============

Replace the -EINVAL return with a debug message and return 0 instead.

Also we do not reach the init function anymore if the DT node is
disabled (as this is now handled by OF_DECLARE), so remove the explicit
check for that case.

This fixes a long standing bogus error when booting ARM's fastmodels.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Robin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/20220506162522.3675399-1-andre.przywara@arm.com
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
drivers/clocksource/timer-sp804.c