From patchwork Sun Oct 28 12:55:00 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Blumenstingl X-Patchwork-Id: 10658595 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id A4A2613A9 for ; Sun, 28 Oct 2018 12:56:14 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9282028387 for ; Sun, 28 Oct 2018 12:56:14 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 8633C28505; Sun, 28 Oct 2018 12:56:14 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FROM,MAILING_LIST_MULTI,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 9D75628387 for ; Sun, 28 Oct 2018 12:56:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=7ZR21yOVjAPeAUCj8Cnc6PNvJV2GGlTITVpXTXUK7a8=; b=i72LkSOgxd396J hNrtXfw1AHyJfd4XNJVeRSFOVILMCMpYE1oWQH05tah06ekEtGGfLN51au9fK8nGRMPGE+M/eabK3 nanXzic30DT+79KQ7RHti3+xXe487gZd8GLgn0kJQBsPKdq+IjA0P4FYiJtWe+PBbjvrG9taGv4yz 39P6QuEGxmo7mla4Orm/zIpvC2lFvR6w5fS9FiJtU9PRo6cOvImBl/4GjAm8o1a+swCqF6YHerxQS xZW4BM+0blDws/tmYW5jANBzO7H8CQ9uePDYQOyWFP9kThQM90xZMhZzmAbQAcW5rGUZZYwBEO1Yr VHrMMhU+frpnaZp+6e2A==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gGkbw-00038E-Um; Sun, 28 Oct 2018 12:56:09 +0000 Received: from mail-wm1-x342.google.com ([2a00:1450:4864:20::342]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gGkbG-0002E9-Ge; Sun, 28 Oct 2018 12:55:28 +0000 Received: by mail-wm1-x342.google.com with SMTP id y144-v6so5369332wmd.4; Sun, 28 Oct 2018 05:55:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=+uso9Ozi2wPAuGZmooAksNGFtRAXXFWFpj8R8QZQcQY=; b=rHY3Tter6Fb0Dwp8XneSYCUsQpNLF15iKBiXqtoaHKN+4jevMyuREpxcXKF9uYcG4/ a3Rgp+rky6z8v9w4TUv/3GQlHXT/vi1BsS1eTwafvQfRD8cBnyQR+Ne2dQQoECxj7t4R B0Z2wzceoS/MZYsdumf3ly3lSRnOxms4sBxtFyR80+SstzQzVnLnB69LKZE84jVm/HqE DrnTCydS82xPpW8JwRdVPRJfXJBQHltEtGnaourwVzPztJy6vjtWkxciIeVnc5R7Od1i ctemHSLZLJmlmKtZ9qtsRBD5v5oKFeyAXa5O+2LNdYyDKPCaOLqb0OshGKkdoQqnrTIr kdTA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=+uso9Ozi2wPAuGZmooAksNGFtRAXXFWFpj8R8QZQcQY=; b=Hg7gcczpnaZMOi4Uz83/Zbgrk5PMxnUQSF/z1SRcJBsPy4goMKymI0RKlZ07afNyc3 8gOl1+8xR4aOtMeahhMdrQPZmBjcUo79qN1urwxC6hJxr897cQEP28XH/+u6acy5tzDq 5yGTC8UaX5fFd1zQhdk0DbonlvJ3zT1yIbNRL1z3JagXGu5pd/vC8PD5jgtjPThnt3a2 ctnlFVZZYUVmLc/jt2ZmAjSLEXNIHzISZTMxOJhSoxaFLfgN8zPAYzYm/KuNqBipKjMg 9ZbHQych4kbCFheL4qYnz3x6+kFgWgBNIq6lqDfANHyXMO7Q3e5znIsAXulXQwTg0BEn oc6g== X-Gm-Message-State: AGRZ1gLD23rDaW6uReqzQs+5w1oTjBB39J0MN1TI4bLuRWS0GQJas9rQ BavALNU6FHb9YPehaBInQA3WLU8Z X-Google-Smtp-Source: AJdET5e3x0NXEiTH9nbYaj1j08PjQrqKi0tyZgbYHohu4PUwv+a8+zUwvEBDBpmxJDUiMqmAbigobQ== X-Received: by 2002:a1c:ce0e:: with SMTP id e14-v6mr10474129wmg.45.1540731315393; Sun, 28 Oct 2018 05:55:15 -0700 (PDT) Received: from blackbox.darklights.net (p200300DCD7072200691979D1FB980407.dip0.t-ipconnect.de. [2003:dc:d707:2200:6919:79d1:fb98:407]) by smtp.googlemail.com with ESMTPSA id u10-v6sm13759388wrt.59.2018.10.28.05.55.14 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 28 Oct 2018 05:55:14 -0700 (PDT) From: Martin Blumenstingl To: linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, tglx@linutronix.de, daniel.lezcano@linaro.org Subject: [PATCH 1/2] clocksource: meson6_timer: use register names from the datasheet Date: Sun, 28 Oct 2018 13:55:00 +0100 Message-Id: <20181028125501.17336-2-martin.blumenstingl@googlemail.com> X-Mailer: git-send-email 2.19.1 In-Reply-To: <20181028125501.17336-1-martin.blumenstingl@googlemail.com> References: <20181028125501.17336-1-martin.blumenstingl@googlemail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181028_055526_599543_0D97D547 X-CRM114-Status: GOOD ( 16.09 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: carlo@caione.org, khilman@baylibre.com, Martin Blumenstingl , linux-arm-kernel@lists.infradead.org Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+patchwork-linux-amlogic=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP This makes the driver use the names from S805 datasheet for the preprocessor #defines. This makes it easier to spot that the driver currently only supports Timer A (as clockevent with interrupt support) and Timer E (as clocksource without interrupts). Timer B, C and D (which are similar to Timer A) are currently not supported by the driver. While here, this also removes the internal "CED_ID" and "CSD_ID" defines which are used to identify the timer. These IDs are not described in the datasheet and thus make it harder to compare the code to what's written in the datasheet. Signed-off-by: Martin Blumenstingl --- drivers/clocksource/meson6_timer.c | 110 ++++++++++++++++++----------- 1 file changed, 68 insertions(+), 42 deletions(-) diff --git a/drivers/clocksource/meson6_timer.c b/drivers/clocksource/meson6_timer.c index 92f20991a937..c622135bee9d 100644 --- a/drivers/clocksource/meson6_timer.c +++ b/drivers/clocksource/meson6_timer.c @@ -10,6 +10,8 @@ * warranty of any kind, whether express or implied. */ +#include +#include #include #include #include @@ -20,80 +22,102 @@ #include #include -#define CED_ID 0 -#define CSD_ID 4 - -#define TIMER_ISA_MUX 0 -#define TIMER_ISA_VAL(t) (((t) + 1) << 2) - -#define TIMER_INPUT_BIT(t) (2 * (t)) -#define TIMER_ENABLE_BIT(t) (16 + (t)) -#define TIMER_PERIODIC_BIT(t) (12 + (t)) +enum meson6_timera_input_clock { + MESON_TIMERA_CLOCK_1US = 0x0, + MESON_TIMERA_CLOCK_10US = 0x1, + MESON_TIMERA_CLOCK_100US = 0x2, + MESON_TIMERA_CLOCK_1MS = 0x3, +}; -#define TIMER_CED_INPUT_MASK (3UL << TIMER_INPUT_BIT(CED_ID)) -#define TIMER_CSD_INPUT_MASK (7UL << TIMER_INPUT_BIT(CSD_ID)) +enum meson6_timere_input_clock { + MESON_TIMERE_CLOCK_SYSTEM_CLOCK = 0x0, + MESON_TIMERE_CLOCK_1US = 0x1, + MESON_TIMERE_CLOCK_10US = 0x2, + MESON_TIMERE_CLOCK_100US = 0x3, + MESON_TIMERE_CLOCK_1MS = 0x4, +}; -#define TIMER_CED_UNIT_1US 0 -#define TIMER_CSD_UNIT_1US 1 +#define MESON_ISA_TIMER_MUX 0x00 +#define MESON_ISA_TIMER_MUX_TIMERD_EN BIT(19) +#define MESON_ISA_TIMER_MUX_TIMERC_EN BIT(18) +#define MESON_ISA_TIMER_MUX_TIMERB_EN BIT(17) +#define MESON_ISA_TIMER_MUX_TIMERA_EN BIT(16) +#define MESON_ISA_TIMER_MUX_TIMERD_MODE BIT(15) +#define MESON_ISA_TIMER_MUX_TIMERC_MODE BIT(14) +#define MESON_ISA_TIMER_MUX_TIMERB_MODE BIT(13) +#define MESON_ISA_TIMER_MUX_TIMERA_MODE BIT(12) +#define MESON_ISA_TIMER_MUX_TIMERE_INPUT_CLOCK_MASK GENMASK(10, 8) +#define MESON_ISA_TIMER_MUX_TIMERD_INPUT_CLOCK_MASK GENMASK(7, 6) +#define MESON_ISA_TIMER_MUX_TIMERC_INPUT_CLOCK_MASK GENMASK(5, 4) +#define MESON_ISA_TIMER_MUX_TIMERB_INPUT_CLOCK_MASK GENMASK(3, 2) +#define MESON_ISA_TIMER_MUX_TIMERA_INPUT_CLOCK_MASK GENMASK(1, 0) + +#define MESON_ISA_TIMERA 0x04 +#define MESON_ISA_TIMERB 0x08 +#define MESON_ISA_TIMERC 0x0c +#define MESON_ISA_TIMERD 0x10 +#define MESON_ISA_TIMERE 0x14 static void __iomem *timer_base; static u64 notrace meson6_timer_sched_read(void) { - return (u64)readl(timer_base + TIMER_ISA_VAL(CSD_ID)); + return (u64)readl(timer_base + MESON_ISA_TIMERE); } -static void meson6_clkevt_time_stop(unsigned char timer) +static void meson6_clkevt_time_stop(void) { - u32 val = readl(timer_base + TIMER_ISA_MUX); + u32 val = readl(timer_base + MESON_ISA_TIMER_MUX); - writel(val & ~TIMER_ENABLE_BIT(timer), timer_base + TIMER_ISA_MUX); + writel(val & ~MESON_ISA_TIMER_MUX_TIMERA_EN, + timer_base + MESON_ISA_TIMER_MUX); } -static void meson6_clkevt_time_setup(unsigned char timer, unsigned long delay) +static void meson6_clkevt_time_setup(unsigned long delay) { - writel(delay, timer_base + TIMER_ISA_VAL(timer)); + writel(delay, timer_base + MESON_ISA_TIMERA); } -static void meson6_clkevt_time_start(unsigned char timer, bool periodic) +static void meson6_clkevt_time_start(bool periodic) { - u32 val = readl(timer_base + TIMER_ISA_MUX); + u32 val = readl(timer_base + MESON_ISA_TIMER_MUX); if (periodic) - val |= TIMER_PERIODIC_BIT(timer); + val |= MESON_ISA_TIMER_MUX_TIMERA_MODE; else - val &= ~TIMER_PERIODIC_BIT(timer); + val &= ~MESON_ISA_TIMER_MUX_TIMERA_MODE; - writel(val | TIMER_ENABLE_BIT(timer), timer_base + TIMER_ISA_MUX); + writel(val | MESON_ISA_TIMER_MUX_TIMERA_EN, + timer_base + MESON_ISA_TIMER_MUX); } static int meson6_shutdown(struct clock_event_device *evt) { - meson6_clkevt_time_stop(CED_ID); + meson6_clkevt_time_stop(); return 0; } static int meson6_set_oneshot(struct clock_event_device *evt) { - meson6_clkevt_time_stop(CED_ID); - meson6_clkevt_time_start(CED_ID, false); + meson6_clkevt_time_stop(); + meson6_clkevt_time_start(false); return 0; } static int meson6_set_periodic(struct clock_event_device *evt) { - meson6_clkevt_time_stop(CED_ID); - meson6_clkevt_time_setup(CED_ID, USEC_PER_SEC / HZ - 1); - meson6_clkevt_time_start(CED_ID, true); + meson6_clkevt_time_stop(); + meson6_clkevt_time_setup(USEC_PER_SEC / HZ - 1); + meson6_clkevt_time_start(true); return 0; } static int meson6_clkevt_next_event(unsigned long evt, struct clock_event_device *unused) { - meson6_clkevt_time_stop(CED_ID); - meson6_clkevt_time_setup(CED_ID, evt); - meson6_clkevt_time_start(CED_ID, false); + meson6_clkevt_time_stop(); + meson6_clkevt_time_setup(evt); + meson6_clkevt_time_start(false); return 0; } @@ -144,22 +168,24 @@ static int __init meson6_timer_init(struct device_node *node) } /* Set 1us for timer E */ - val = readl(timer_base + TIMER_ISA_MUX); - val &= ~TIMER_CSD_INPUT_MASK; - val |= TIMER_CSD_UNIT_1US << TIMER_INPUT_BIT(CSD_ID); - writel(val, timer_base + TIMER_ISA_MUX); + val = readl(timer_base + MESON_ISA_TIMER_MUX); + val &= ~MESON_ISA_TIMER_MUX_TIMERE_INPUT_CLOCK_MASK; + val |= FIELD_PREP(MESON_ISA_TIMER_MUX_TIMERE_INPUT_CLOCK_MASK, + MESON_TIMERE_CLOCK_1US); + writel(val, timer_base + MESON_ISA_TIMER_MUX); sched_clock_register(meson6_timer_sched_read, 32, USEC_PER_SEC); - clocksource_mmio_init(timer_base + TIMER_ISA_VAL(CSD_ID), node->name, + clocksource_mmio_init(timer_base + MESON_ISA_TIMERE, node->name, 1000 * 1000, 300, 32, clocksource_mmio_readl_up); /* Timer A base 1us */ - val &= ~TIMER_CED_INPUT_MASK; - val |= TIMER_CED_UNIT_1US << TIMER_INPUT_BIT(CED_ID); - writel(val, timer_base + TIMER_ISA_MUX); + val &= ~MESON_ISA_TIMER_MUX_TIMERA_INPUT_CLOCK_MASK; + val |= FIELD_PREP(MESON_ISA_TIMER_MUX_TIMERA_INPUT_CLOCK_MASK, + MESON_TIMERA_CLOCK_1US); + writel(val, timer_base + MESON_ISA_TIMER_MUX); /* Stop the timer A */ - meson6_clkevt_time_stop(CED_ID); + meson6_clkevt_time_stop(); ret = setup_irq(irq, &meson6_timer_irq); if (ret) { From patchwork Sun Oct 28 12:55:01 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Blumenstingl X-Patchwork-Id: 10658599 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id B646813A4 for ; Sun, 28 Oct 2018 12:57:07 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A0B6028387 for ; Sun, 28 Oct 2018 12:57:07 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 9093028505; Sun, 28 Oct 2018 12:57:07 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FROM,MAILING_LIST_MULTI,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 7188128387 for ; Sun, 28 Oct 2018 12:57:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=23LJpJhiHRrbjNloWyKGF/jva6YRiVeMDkzRiqTIIGQ=; b=teuqPSp3wf7sJ7 E4cIxC+1ELdkzjIsa6Vu08eaww0musJJJPZoHO5uU/XfI5Mw0m3G+zuTUEmMSQhes3fw8zW2LB5Ny nRykwUr+rsOPWEul40l/kyOLbzzVtHl0OHHKNCduDZ9UIZNv10AsAe+Yjh+tkw5OHQ/Fv/tZo2lJJ UTCpxhMPUp0ia/5hjZgSlKmcLnYRsdpeoxVxM8Zafw0G+7193FWzwRchfGaHGQmx5kZIZCjrQPi/r 7noKgG4CHiLMxJjrxy4GjsMFBv0yiMjA5B3mM+eMFsw0CpKCislPfkhc6fFuekh3s3gS0l+l0dcqs ipYUDmUH927GqlXczDbQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gGkcn-0003Vj-JY; Sun, 28 Oct 2018 12:57:01 +0000 Received: from mail-wm1-x344.google.com ([2a00:1450:4864:20::344]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gGkbI-0002Ln-JH; Sun, 28 Oct 2018 12:55:30 +0000 Received: by mail-wm1-x344.google.com with SMTP id l26-v6so5556016wmh.3; Sun, 28 Oct 2018 05:55:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=ZkqiSK0foukgf/wyUrP43W1JLOOI7NiMe4dW1YSf01k=; b=Z7IWfZcPcB/PNywoYxW/PWEV03Ypu5mWL0eY1bCC+X3r+BP99BfoGGhcKQA7pl7Q7y /5lVMTeT4PokHF3gUNtwNJsRtm89a5ctfXv9b+U7YyAB+9UWDmYzg9RtgOQbbPpLReRD K/5Y5mEtemcD5PeIvgFTBiCNGzm3R8BBgJhlF8vaBqP7BnXPDpzNDkR7FgCbeCm+Mkt/ u7W3Xt3vsfyGtDjUCpDXMADD186INcbV85TSXjIlN9iWsK90Xb2/RCthxxkScwrnFVea b1wqhdPm7AsucPhx4JwhdyvS+CtNlks54LE/eYtEaVc0e0vpUGzfmtBsZI0oUVJbyWmW mmbw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=ZkqiSK0foukgf/wyUrP43W1JLOOI7NiMe4dW1YSf01k=; b=paSuXcY293gebf3nv2i1FwI24+tAkSiaBfvMOYIZF/UrZjKsxxc+6VHErjUoQil38s dBNva25ICTdZkYSAEwH8M9DSYIRzTw1Z1lG/gvBeKmkKJq2LVLpWlhtcx8QIR817Z8Tr 8MyW60qQDcAHwPVSY200MzWy2vIDFiWoFHUC/7NNxAX/EiI5AEIiAM4BHGc9WlcqF42X QRbYUK2jCCpgpEeBgwfP298AzrN2dxn8CsFI84TBQWPduN9cx3InFJ3s12jWoAEzNVAh /VIlpwPEVX9AKhc9SQ0odnAomRe2N6xWv9zCJ9EfuTvn7/mobFJj8x3+pWy4uWu9aFN4 rrgg== X-Gm-Message-State: AGRZ1gLzjnZ5gfWeAc19GSpVakf2n0XMFUaUDslqH+NagW8ARRSNSMSk OhDDS004bLg1YoOtJmdRlgBIgbz5 X-Google-Smtp-Source: AJdET5fNaBy49wFoVlhlhHSMueD4TpIheu+8OV97VsJV+9DxdBtIU+NJ0UVQP41SqmDOxXSw0prkjg== X-Received: by 2002:a1c:a794:: with SMTP id q142-v6mr11517090wme.77.1540731316600; Sun, 28 Oct 2018 05:55:16 -0700 (PDT) Received: from blackbox.darklights.net (p200300DCD7072200691979D1FB980407.dip0.t-ipconnect.de. [2003:dc:d707:2200:6919:79d1:fb98:407]) by smtp.googlemail.com with ESMTPSA id u10-v6sm13759388wrt.59.2018.10.28.05.55.15 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 28 Oct 2018 05:55:15 -0700 (PDT) From: Martin Blumenstingl To: linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, tglx@linutronix.de, daniel.lezcano@linaro.org Subject: [PATCH 2/2] clocksource: meson6_timer: implement ARM delay timer Date: Sun, 28 Oct 2018 13:55:01 +0100 Message-Id: <20181028125501.17336-3-martin.blumenstingl@googlemail.com> X-Mailer: git-send-email 2.19.1 In-Reply-To: <20181028125501.17336-1-martin.blumenstingl@googlemail.com> References: <20181028125501.17336-1-martin.blumenstingl@googlemail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181028_055528_665461_87663DAE X-CRM114-Status: GOOD ( 12.38 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: carlo@caione.org, khilman@baylibre.com, Martin Blumenstingl , linux-arm-kernel@lists.infradead.org Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+patchwork-linux-amlogic=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Implement an ARM delay timer to be used for udelay(). This allows us to skip the delay loop calibration at boot. With this patch udelay() is now independent of CPU frequency changes. This is a good thing on Meson8, Meson8b and Meson8m2 because changing the CPU frequency requires running the CPU clock off the XTAL while changing the PLL or it's dividers. After changing the CPU clocks we need to wait a few usecs for the clock to become stable. So having an udelay() implementation that doesn't depend on the CPU frequency is beneficial. Suggested-by: Jianxin Pan Signed-off-by: Martin Blumenstingl --- drivers/clocksource/meson6_timer.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/drivers/clocksource/meson6_timer.c b/drivers/clocksource/meson6_timer.c index c622135bee9d..5ee2a575069b 100644 --- a/drivers/clocksource/meson6_timer.c +++ b/drivers/clocksource/meson6_timer.c @@ -22,6 +22,10 @@ #include #include +#ifdef CONFIG_ARM +#include +#endif + enum meson6_timera_input_clock { MESON_TIMERA_CLOCK_1US = 0x0, MESON_TIMERA_CLOCK_10US = 0x1, @@ -60,6 +64,18 @@ enum meson6_timere_input_clock { static void __iomem *timer_base; +#ifdef CONFIG_ARM +static unsigned long meson6_read_current_timer(void) +{ + return readl_relaxed(timer_base + MESON_ISA_TIMERE); +} + +static struct delay_timer meson6_delay_timer = { + .read_current_timer = meson6_read_current_timer, + .freq = 1000 * 1000, +}; +#endif + static u64 notrace meson6_timer_sched_read(void) { return (u64)readl(timer_base + MESON_ISA_TIMERE); @@ -198,6 +214,12 @@ static int __init meson6_timer_init(struct device_node *node) clockevents_config_and_register(&meson6_clockevent, USEC_PER_SEC, 1, 0xfffe); + +#ifdef CONFIG_ARM + /* Also use MESON_ISA_TIMERE for delays */ + register_current_timer_delay(&meson6_delay_timer); +#endif + return 0; } TIMER_OF_DECLARE(meson6, "amlogic,meson6-timer",