lkml.org 
[lkml]   [2022]   [Jun]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 129/172] clocksource/drivers/sp804: Avoid error on multiple instances
    Date
    From: Andre Przywara <andre.przywara@arm.com>

    [ Upstream commit a98399cbc1e05f7b977419f03905501d566cf54e ]

    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 | 10 +++++-----
    1 file changed, 5 insertions(+), 5 deletions(-)

    diff --git a/drivers/clocksource/timer-sp804.c b/drivers/clocksource/timer-sp804.c
    index 6e8ad4a4ea3c..bedd3570474b 100644
    --- a/drivers/clocksource/timer-sp804.c
    +++ b/drivers/clocksource/timer-sp804.c
    @@ -274,6 +274,11 @@ static int __init sp804_of_init(struct device_node *np, struct sp804_timer *time
    struct clk *clk1, *clk2;
    const char *name = of_get_property(np, "compatible", NULL);

    + if (initialized) {
    + pr_debug("%pOF: skipping further SP804 timer device\n", np);
    + return 0;
    + }
    +
    base = of_iomap(np, 0);
    if (!base)
    return -ENXIO;
    @@ -285,11 +290,6 @@ static int __init sp804_of_init(struct device_node *np, struct sp804_timer *time
    writel(0, timer1_base + timer->ctrl);
    writel(0, timer2_base + timer->ctrl);

    - if (initialized || !of_device_is_available(np)) {
    - ret = -EINVAL;
    - goto err;
    - }
    -
    clk1 = of_clk_get(np, 0);
    if (IS_ERR(clk1))
    clk1 = NULL;
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-06-13 14:49    [W:2.803 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site