Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-15 Thread Mark Rutland
On Tue, Jan 15, 2013 at 11:24:53AM +, Thomas Gleixner wrote: > On Mon, 14 Jan 2013, Mark Rutland wrote: > > On Mon, Jan 14, 2013 at 02:17:26PM +, Thomas Gleixner wrote: > > > > I thought this previously also [1], but I couldn't find any path such > > > > that a > > > > tick_cpu_device woul

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-15 Thread Thomas Gleixner
On Mon, 14 Jan 2013, Mark Rutland wrote: > On Mon, Jan 14, 2013 at 02:17:26PM +, Thomas Gleixner wrote: > > > I thought this previously also [1], but I couldn't find any path such > > > that a > > > tick_cpu_device would have an evtdev without an event_handler. We always > > > set the > > > h

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Santosh Shilimkar
On Monday 14 January 2013 09:06 PM, Mark Rutland wrote: On Mon, Jan 14, 2013 at 02:17:26PM +, Thomas Gleixner wrote: On Mon, 14 Jan 2013, Mark Rutland wrote: On Mon, Jan 14, 2013 at 11:50:55AM +, Thomas Gleixner wrote: On Mon, 14 Jan 2013, Mark Rutland wrote: On Mon, Jan 14, 2013 at

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Mark Rutland
On Mon, Jan 14, 2013 at 02:17:26PM +, Thomas Gleixner wrote: > On Mon, 14 Jan 2013, Mark Rutland wrote: > > On Mon, Jan 14, 2013 at 11:50:55AM +, Thomas Gleixner wrote: > > > On Mon, 14 Jan 2013, Mark Rutland wrote: > > > > > > > On Mon, Jan 14, 2013 at 11:06:31AM +, Thomas Gleixner wr

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Thomas Gleixner
On Mon, 14 Jan 2013, Mark Rutland wrote: > On Mon, Jan 14, 2013 at 11:50:55AM +, Thomas Gleixner wrote: > > On Mon, 14 Jan 2013, Mark Rutland wrote: > > > > > On Mon, Jan 14, 2013 at 11:06:31AM +, Thomas Gleixner wrote: > > > > On Wed, 9 Jan 2013, Mark Rutland wrote: > > > > > +#ifdef CONF

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Mark Rutland
On Mon, Jan 14, 2013 at 11:50:55AM +, Thomas Gleixner wrote: > On Mon, 14 Jan 2013, Mark Rutland wrote: > > > On Mon, Jan 14, 2013 at 11:06:31AM +, Thomas Gleixner wrote: > > > On Wed, 9 Jan 2013, Mark Rutland wrote: > > > > +#ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST > > > > +extern int

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Thomas Gleixner
On Mon, 14 Jan 2013, Mark Rutland wrote: > On Mon, Jan 14, 2013 at 11:06:31AM +, Thomas Gleixner wrote: > > On Wed, 9 Jan 2013, Mark Rutland wrote: > > > +#ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST > > > +extern int tick_receive_broadcast(void); > > > +#else > > > +static inline int tick_rece

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Mark Rutland
On Mon, Jan 14, 2013 at 11:06:31AM +, Thomas Gleixner wrote: > On Wed, 9 Jan 2013, Mark Rutland wrote: > > +#ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST > > +extern int tick_receive_broadcast(void); > > +#else > > +static inline int tick_receive_broadcast(void) > > +{ > > + return 0; > > +} >

Re: [PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-14 Thread Thomas Gleixner
On Wed, 9 Jan 2013, Mark Rutland wrote: > +#ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST > +extern int tick_receive_broadcast(void); > +#else > +static inline int tick_receive_broadcast(void) > +{ > + return 0; > +} What's the inline function for? If an arch does not have broadcasting support it

[PATCHv2 1/4] clockevents: Add generic timer broadcast receiver

2013-01-09 Thread Mark Rutland
Currently the broadcast mechanism used for timers is abstracted by a function pointer on struct clock_event_device. As the fundamental mechanism for broadcast is architecture-specific, this ties each clock_event_device driver to a single architecture, even where the driver is otherwise generic. Th