From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Jonas Jensen <jonas.jensen@gmail.com>
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, arm@kernel.org,
john.stultz@linaro.org, tglx@linutronix.de,
u.kleine-koenig@pengutronix.de, tomasz.figa@gmail.com,
linus.walleij@linaro.org, thomas.petazzoni@free-electrons.com,
arnd@arndb.de
Subject: Re: [PATCH v7] ARM: clocksource: add support for MOXA ART SoCs
Date: Tue, 16 Jul 2013 15:52:26 +0200 [thread overview]
Message-ID: <51E5501A.6070403@linaro.org> (raw)
In-Reply-To: <1373024773-24574-1-git-send-email-jonas.jensen@gmail.com>
On 07/05/2013 01:46 PM, Jonas Jensen wrote:
> This patch adds an clocksource driver for the main timer(s)
> found on MOXA ART SoCs.
>
> Signed-off-by: Jonas Jensen <jonas.jensen@gmail.com>
> ---
>
> Notes:
> Applies to next-20130703
>
> Changes since v6:
>
> 1. remove TIMER_CR read back
> 2. set TIMER_CR from constants
>
Except for the few comments below, it looks ok for me.
[ ... ]
> + */
> +#define TIMEREG_CR_1_ENABLE (1 << 0)
> +#define TIMEREG_CR_1_CLOCK (1 << 1)
> +#define TIMEREG_CR_1_INT (1 << 2)
> +#define TIMEREG_CR_2_ENABLE (1 << 3)
> +#define TIMEREG_CR_2_CLOCK (1 << 4)
> +#define TIMEREG_CR_2_INT (1 << 5)
> +#define TIMEREG_CR_3_ENABLE (1 << 6)
> +#define TIMEREG_CR_3_CLOCK (1 << 7)
> +#define TIMEREG_CR_3_INT (1 << 8)
> +#define TIMEREG_CR_COUNT_UP (1 << 9)
> +#define TIMEREG_CR_COUNT_DOWN (0 << 9)
Could you replace this by the BIT macro ?
[ ... ]
> +
> + irq = irq_of_parse_and_map(node, 0);
> + if (irq <= 0)
> + panic("%s: irq_of_parse_and_map failed\n", node->full_name);
> +
> + ret = setup_irq(irq, &moxart_timer_irq);
> + if (ret)
> + panic("%s: setup_irq failed\n", node->full_name);
> +
> + clk = of_clk_get(node, 0);
> + if (IS_ERR(clk))
> + panic("%s: of_clk_get failed\n", node->full_name);
> +
> + pclk = clk_get_rate(clk);
> +
> + if (clocksource_mmio_init(base + TIMER2_BASE + REG_COUNT,
> + "moxart_timer", pclk, 200, 32,
> + clocksource_mmio_readl_down))
> + panic("%s: clocksource_mmio_init failed\n", node->full_name);
> +
> + clock_count_per_tick = DIV_ROUND_CLOSEST(pclk, HZ);
> +
> + writel(~0, base + TIMER2_BASE + REG_LOAD);
> + writel(TIMEREG_CR_2_ENABLE, base + TIMER_CR);
> +
> + moxart_clockevent.cpumask = cpumask_of(0);
moxart_clockevent.irq = irq;
> +
> + /*
> + * documentation is not publicly available:
> + * min_delta / max_delta obtained by trial-and-error,
> + * max_delta 0xfffffffe should be ok because count
> + * register size is u32
> + */
> + clockevents_config_and_register(&moxart_clockevent, pclk,
> + 0x4, 0xfffffffe);
> +}
> +CLOCKSOURCE_OF_DECLARE(moxart, "moxa,moxart-timer", moxart_timer_init);
Thanks
-- Daniel
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
next prev parent reply other threads:[~2013-07-16 13:52 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-18 10:00 [PATCH] ARM: clocksource: add support for MOXA ART SoCs Jonas Jensen
2013-06-18 15:14 ` Thomas Petazzoni
2013-06-18 15:28 ` Arnd Bergmann
2013-06-26 14:53 ` [PATCH v2] " Jonas Jensen
2013-06-26 14:59 ` Jonas Jensen
2013-06-26 16:10 ` Uwe Kleine-König
2013-06-26 19:15 ` Linus Walleij
2013-06-27 11:23 ` [PATCH v3] " Jonas Jensen
2013-06-28 13:34 ` Thomas Gleixner
2013-07-01 14:02 ` [PATCH v4] " Jonas Jensen
2013-07-01 17:55 ` Thomas Gleixner
2013-07-02 20:19 ` Linus Walleij
2013-07-04 12:19 ` [PATCH v5] " Jonas Jensen
2013-07-04 21:42 ` Thomas Gleixner
2013-07-05 10:05 ` Jonas Jensen
2013-07-05 10:21 ` Thomas Gleixner
2013-07-05 11:48 ` Jonas Jensen
2013-07-05 10:04 ` [PATCH v6] " Jonas Jensen
2013-07-05 11:46 ` [PATCH v7] " Jonas Jensen
2013-07-16 13:52 ` Daniel Lezcano [this message]
2013-07-16 14:44 ` [PATCH v8] " Jonas Jensen
2013-07-16 15:01 ` Daniel Lezcano
2013-07-17 8:14 ` Jonas Jensen
2013-07-17 12:13 ` Daniel Lezcano
2013-07-17 8:04 ` [PATCH v9] " Jonas Jensen
2013-07-19 11:12 ` [PATCH] ARM: clocksource: moxart: documentation: update device tree bindings document Jonas Jensen
2013-07-19 12:16 ` Daniel Lezcano
2013-07-19 12:50 ` Jonas Jensen
2013-07-20 20:45 ` [PATCH v9] ARM: clocksource: add support for MOXA ART SoCs Linus Walleij
2013-07-20 21:34 ` Daniel Lezcano
2013-07-26 14:03 ` [PATCH] ARM: clocksource: moxart: add bitops.h include Jonas Jensen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51E5501A.6070403@linaro.org \
--to=daniel.lezcano@linaro.org \
--cc=arm@kernel.org \
--cc=arnd@arndb.de \
--cc=john.stultz@linaro.org \
--cc=jonas.jensen@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=thomas.petazzoni@free-electrons.com \
--cc=tomasz.figa@gmail.com \
--cc=u.kleine-koenig@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).