[02/15] dt-bindings: timer: add ti,dmtimer compatible for for system timers
diff mbox series

Message ID 20200429215402.18125-3-tony@atomide.com
State New
Headers show
Series
  • Update omaps to use drivers/clocksource timers
Related show

Commit Message

Tony Lindgren April 29, 2020, 9:53 p.m. UTC
The TI dual-mode timer can be used for both clocksource and clockevent
system timers. We need a way to specify which dual-mode timers are
reserved for system timers as there are multiple instances available
that may require a board specific configuration.

Let's add a generic compatible "ti,dmtimer" that TIMER_OF_DECLARE can
use as suggested by Daniel Lezcano <daniel.lezcano@linaro.org>.

Suggested-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Signed-off-by: Tony Lindgren <tony@atomide.com>
---
 Documentation/devicetree/bindings/timer/ti,timer.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Rob Herring May 12, 2020, 2:52 p.m. UTC | #1
On Wed, 29 Apr 2020 14:53:49 -0700, Tony Lindgren wrote:
> The TI dual-mode timer can be used for both clocksource and clockevent
> system timers. We need a way to specify which dual-mode timers are
> reserved for system timers as there are multiple instances available
> that may require a board specific configuration.
> 
> Let's add a generic compatible "ti,dmtimer" that TIMER_OF_DECLARE can
> use as suggested by Daniel Lezcano <daniel.lezcano@linaro.org>.
> 
> Suggested-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Signed-off-by: Tony Lindgren <tony@atomide.com>
> ---
>  Documentation/devicetree/bindings/timer/ti,timer.txt | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>
Tony Lindgren May 12, 2020, 2:56 p.m. UTC | #2
* Rob Herring <robh@kernel.org> [200512 14:53]:
> On Wed, 29 Apr 2020 14:53:49 -0700, Tony Lindgren wrote:
> > The TI dual-mode timer can be used for both clocksource and clockevent
> > system timers. We need a way to specify which dual-mode timers are
> > reserved for system timers as there are multiple instances available
> > that may require a board specific configuration.
> > 
> > Let's add a generic compatible "ti,dmtimer" that TIMER_OF_DECLARE can
> > use as suggested by Daniel Lezcano <daniel.lezcano@linaro.org>.
> > 
> > Suggested-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> > Signed-off-by: Tony Lindgren <tony@atomide.com>
> > ---
> >  Documentation/devicetree/bindings/timer/ti,timer.txt | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> 
> Acked-by: Rob Herring <robh@kernel.org>

Looks like this is no longer needed with v3 set in the repost
thread:

"[PATCHv3 00/14] Update omaps to use drivers/clocksource timers"

It now does detection for the preferred system timers based on
the various timer features like you suggested.

Regards,

Tony

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/timer/ti,timer.txt b/Documentation/devicetree/bindings/timer/ti,timer.txt
--- a/Documentation/devicetree/bindings/timer/ti,timer.txt
+++ b/Documentation/devicetree/bindings/timer/ti,timer.txt
@@ -14,6 +14,7 @@  Required properties:
 			ti,omap5430-timer (applicable to OMAP543x devices)
 			ti,am335x-timer	(applicable to AM335x devices)
 			ti,am335x-timer-1ms (applicable to AM335x devices)
+			ti,dmtimer (applicable to system timers)
 
 - reg:			Contains timer register address range (base address and
 			length).