From: alexandre.belloni@bootlin.com (Alexandre Belloni)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/6] clocksource: rework Atmel TCB timer driver
Date: Thu, 29 Mar 2018 13:42:55 +0200 [thread overview]
Message-ID: <20180329114255.GL13942@piout.net> (raw)
In-Reply-To: <3103570.fW0XyqvKVi@ada>
On 29/03/2018 at 13:31:18 +0200, Alexander Dahl wrote:
> Pretty sure. I rebuilt the whole BSP and added another line to the kernel
> source to see if the tree I applied the patches to, was actually built:
>
>
> diff --git a/drivers/clocksource/timer-atmel-tcb.c b/drivers/clocksource/timer-atmel-tcb.c
> index 7fde9cfbf203..f85affc74a86 100644
> --- a/drivers/clocksource/timer-atmel-tcb.c
> +++ b/drivers/clocksource/timer-atmel-tcb.c
> @@ -222,7 +222,8 @@ static int __init tc_clkevt_register(struct device_node *node,
> goto err_slow;
> clk_disable(tce.clk);
>
> - clockevents_config_and_register(&tce.clkevt, 32768, 1, bits - 1);
> + pr_info( "*** bits: 0x%x, BIT(bits): 0x%lx\n", bits, BIT(bits) );
> + clockevents_config_and_register(&tce.clkevt, 32768, 1, BIT(bits) - 1);
>
> ret = request_irq(tce.irq, tc_clkevt2_irq, IRQF_TIMER | IRQF_SHARED,
> tce.clkevt.name, &tce);
>
>
I've just tested on a g20, old driver:
INT NAME RATE MAX
16 [vel at91_tick,] 175 Ints/s (max: 231)
19 [ vel tc_clkevt] 129 Ints/s (max: 129)
new driver:
INT NAME RATE MAX
17 [vel timer at fffa] 129 Ints/s (max: 129)
18 [ vel ttyS0] 175 Ints/s (max: 231)
--
Alexandre Belloni, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2018-03-29 11:42 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-23 17:15 [PATCH v3 0/6] clocksource: rework Atmel TCB timer driver Alexandre Belloni
2018-02-23 17:15 ` [PATCH v3 1/6] ARM: at91: add TCB registers definitions Alexandre Belloni
2018-02-23 17:15 ` [PATCH v3 2/6] clocksource/drivers: Add a new driver for the Atmel ARM TC blocks Alexandre Belloni
2018-02-23 17:15 ` [PATCH v3 3/6] clocksource/drivers: atmel-pit: make option silent Alexandre Belloni
2018-02-23 17:15 ` [PATCH v3 4/6] ARM: at91: Implement clocksource selection Alexandre Belloni
2018-02-23 17:15 ` [PATCH v3 5/6] ARM: configs: at91: use new TCB timer driver Alexandre Belloni
2018-02-23 17:15 ` [PATCH v3 6/6] ARM: configs: at91: unselect PIT Alexandre Belloni
2018-03-22 15:33 ` [PATCH v3 0/6] clocksource: rework Atmel TCB timer driver Alexandre Belloni
2018-03-27 10:41 ` Alexander Dahl
2018-03-27 11:30 ` Daniel Lezcano
2018-03-28 10:29 ` Alexander Dahl
2018-03-28 13:03 ` Daniel Lezcano
2018-03-28 14:16 ` Alexandre Belloni
2018-03-28 14:36 ` Daniel Lezcano
2018-03-28 15:31 ` Alexandre Belloni
2018-03-28 15:50 ` Alexandre Belloni
2018-03-29 8:01 ` Alexander Dahl
2018-03-29 10:45 ` Alexandre Belloni
2018-03-29 11:31 ` Alexander Dahl
2018-03-29 11:42 ` Alexandre Belloni [this message]
2018-03-29 12:07 ` Daniel Lezcano
2018-03-29 13:02 ` Alexandre Belloni
2018-03-29 15:11 ` Alexander Dahl
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=20180329114255.GL13942@piout.net \
--to=alexandre.belloni@bootlin.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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).