Re: [PATCHv4] x86, mrst: share APB timer code with other platforms

[john stultz <johnstul@us.ibm.com>]

On Tue, 2011-06-21 at 11:14 +0100, Jamie Iles wrote:
> On Mon, Jun 06, 2011 at 12:43:07PM +0100, Jamie Iles wrote:
> > The APB timers are an IP block from Synopsys (DesignWare APB timers)
> > and are also found in other systems including ARM SoC's.  This patch
> > adds functions for creating clock_event_devices and clocksources from
> > APB timers but does not do the resource allocation.  This is handled
> > in a higher layer to allow the timers to be created from multiple
> > methods such as platform_devices.
> > 
> > Changes since v3:
> > 	- Add dw_apb_clocksource_unregister()
> > 	- Move kerneldoc to the implementation file
> > Changes since v2:
> > 	- Make eoi() for clock_event_device optional for x86 where eoi                                                                                                                                                                                                                                                         
> > 	  is handled in firmware.
> > Changes since v1:
> > 	- Use the correct timer for clocksource on x86
> > 	- Select the correct timer rating for x86
> > 	- Restore freerunning timer behaviour for oneshot event devices
> > 	- Reenable event irq correctly for hotplug
> > 
> > Cc: John Stultz <johnstul@us.ibm.com>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Ingo Molnar <mingo@redhat.com>
> > Cc: "H. Peter Anvin" <hpa@zytor.com>
> > Cc: Jacob Pan <jacob.jun.pan@linux.intel.com>
> > Signed-off-by: Jamie Iles <jamie@jamieiles.com>
> > ---
> 
> Ping?

Sorry for being slow here. Looks ok to me, so I've queued this for 3.1.

thanks
-john