]> git.baikalelectronics.ru Git - kernel.git/commit
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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Jun 2022 16:36:22 +0000 (18:36 +0200)
commit576bfe3b40f56b71f8a077e2f7133b46ce7a4d8f
tree7a58a129f9950177d20b83d6290fe10dccd9ae9d
parent916d2ba846cc728b77ee713de15c16475c72341e
clocksource/drivers/sp804: Avoid error on multiple instances

[ Upstream commit c6e9b96210c807c2949466e7fd5898fbde198d8f ]

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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/clocksource/timer-sp804.c