From: Thomas Gleixner <tglx@linutronix.de>
To: Biju Das <biju.das.jz@bp.renesas.com>,
Geert Uytterhoeven <geert@linux-m68k.org>,
William Breathitt Gray <william.gray@linaro.org>,
Rob Herring <robh@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
William Breathitt Gray <william.gray@linaro.org>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Magnus Damm <magnus.damm@gmail.com>,
"linux-renesas-soc@vger.kernel.org"
<linux-renesas-soc@vger.kernel.org>,
"linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>,
Fabrizio Castro <fabrizio.castro.jz@renesas.com>
Subject: RE: [PATCH 0/6] Add RZ/V2M Compare-Match Timer (TIM) support
Date: Wed, 07 Dec 2022 11:53:55 +0100 [thread overview]
Message-ID: <878rjjfprw.ffs@tglx> (raw)
In-Reply-To: <OS0PR01MB59228CED6187C7B19776CE22861A9@OS0PR01MB5922.jpnprd01.prod.outlook.com>
Biju!
On Wed, Dec 07 2022 at 07:52, Biju Das wrote:
>> On Tue, Dec 06 2022 at 09:40, Geert Uytterhoeven wrote:
>> > When multiple clocksources are registered, the clocksource subsystems
>> > picks the best one anyway, right?
>>
>> As it does for the clock event devices. If there is an architected timer
>> then that should be always preferred.
>>
>> No idea why there is a need for the extra hardware and the drivers which
>> are both never utilized.
>
> I got feedback from BSP team for the actual usage of this timer.
>
> Basically, this HW timer is used for measuring the processing time
> of DRP-AI accurately compared to the CPU timer normally we use.
How is a slow to access timer with a lower clock frequency more
accurate?
> The example use cases,
> Timer in FREERUN mode, Check the timer value after the restart(1usec)"
> Timer in FREERUN mode, Check the timer value after the restart(10000000usec)"
>
> What is the model to be used for this kind of HW usage? Counter or Timer?
>
> I can think of one possible HW usage by using Counter model.
> Not sure how timer model can be used for this kind of HW usage??
>
> Eg: we can set ceiling values 1usec and 10000000usec using counter framework
> And that will trigger interrupt events corresponding to the ceiling values
> to user space and user space app can accurately measure the DRP-AI processing time.
>
> Also counter model exposes count values to user space from the counter HW.
Counter subsystem != clocksource/event subsystem.
We are debating a clocksource/clockevent driver and not a counter
driver, right?
Thanks,
tglx
next prev parent reply other threads:[~2022-12-07 10:54 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-05 14:59 [PATCH 0/6] Add RZ/V2M Compare-Match Timer (TIM) support Biju Das
2022-12-05 14:59 ` [PATCH 1/6] clk: renesas: r9a09g011: Add TIM clock and reset entries Biju Das
2022-12-21 14:47 ` Geert Uytterhoeven
2022-12-05 14:59 ` [PATCH 2/6] dt-bindings: timer: Add RZ/V2M TIM binding Biju Das
2022-12-05 14:59 ` [PATCH 3/6] clocksource/drivers/rzv2m-tim: Add Renesas RZ/V2M compare match timer(TIM) driver Biju Das
2022-12-05 14:59 ` [PATCH 4/6] dt-bindings: counter: Add RZ/V2M TIM counter binding Biju Das
2022-12-05 14:59 ` [PATCH 5/6] counter: Add Renesas RZ/V2M TIM counter driver Biju Das
2022-12-05 14:59 ` [PATCH 6/6] arm64: dts: renesas: r9a09g011: Add tim nodes Biju Das
2022-12-05 22:50 ` [PATCH 0/6] Add RZ/V2M Compare-Match Timer (TIM) support Rob Herring
2022-12-06 8:13 ` Biju Das
2022-12-06 8:40 ` Geert Uytterhoeven
2022-12-06 8:57 ` Thomas Gleixner
2022-12-06 9:45 ` Biju Das
2022-12-07 7:52 ` Biju Das
2022-12-07 10:53 ` Thomas Gleixner [this message]
2022-12-07 11:35 ` Biju Das
2022-12-07 16:49 ` Thomas Gleixner
2022-12-09 22:24 ` William Breathitt Gray
2022-12-10 7:52 ` Biju Das
2022-12-10 10:59 ` Thomas Gleixner
2022-12-06 8:59 ` Biju Das
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=878rjjfprw.ffs@tglx \
--to=tglx@linutronix.de \
--cc=biju.das.jz@bp.renesas.com \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=fabrizio.castro.jz@renesas.com \
--cc=geert+renesas@glider.be \
--cc=geert@linux-m68k.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=mturquette@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=prabhakar.mahadev-lad.rj@bp.renesas.com \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
--cc=william.gray@linaro.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.