Re: [PATCHv3/RESEND 4/4] clocksource: arch_timer: Add support for memory mapped timers

2013-06-21 Thread Thomas Gleixner
On Fri, 21 Jun 2013, Stephen Boyd wrote: > On 06/21/13 14:18, Thomas Gleixner wrote: > > Something in my little brain yells: function pointer > > > > You can't be serious about hacking nested if/else/switch constructs > > into a hot path. > > > > Why not making your cpu data: > > > > struct arch_ti

Re: [PATCHv3/RESEND 4/4] clocksource: arch_timer: Add support for memory mapped timers

2013-06-21 Thread Stephen Boyd
On 06/21/13 14:18, Thomas Gleixner wrote: > On Fri, 21 Jun 2013, Stephen Boyd wrote: >> /* >> * Architected system timer support. >> @@ -46,13 +73,69 @@ static bool arch_timer_use_virtual = true; >> static inline void arch_timer_reg_write(int access, int reg, u32 val, >>

Re: [PATCHv3/RESEND 4/4] clocksource: arch_timer: Add support for memory mapped timers

2013-06-21 Thread Thomas Gleixner
On Fri, 21 Jun 2013, Stephen Boyd wrote: > /* > * Architected system timer support. > @@ -46,13 +73,69 @@ static bool arch_timer_use_virtual = true; > static inline void arch_timer_reg_write(int access, int reg, u32 val, > struct clock_event_device *clk) >

[PATCHv3/RESEND 4/4] clocksource: arch_timer: Add support for memory mapped timers

2013-06-21 Thread Stephen Boyd
Add support for the memory mapped timers by filling in the read/write functions and adding some parsing code. Note that we only register one clocksource, preferring the cp15 based clocksource over the mmio one. To keep things simple we register one global clockevent. This covers the case of UP and