Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-21 Thread Grant Likely
On Mon, Apr 21, 2008 at 12:39 PM, Scott Wood <[EMAIL PROTECTED]> wrote: > On Mon, Apr 21, 2008 at 08:03:31AM -0600, Grant Likely wrote: > > > > +r) Freescale General-purpose Timers Module > > > + > > > +Required properties: > > > + - compatible : should be "fsl,gtm" ("fsl,qe-gtm

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-21 Thread Scott Wood
On Mon, Apr 21, 2008 at 08:03:31AM -0600, Grant Likely wrote: > > +r) Freescale General-purpose Timers Module > > + > > +Required properties: > > + - compatible : should be "fsl,gtm" ("fsl,qe-gtm" in addition for QE > > + GTMs or "fsl,cpm2-gtm" for CPM2 GTMs).

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-21 Thread Segher Boessenkool
+static inline void gtm_ack_timer16(struct gtm_timer *tmr, u16 events) +{ + out_be16(tmr->gtevr, events); +} Drop 'inline' and expect gcc to do the right thing. Not sure about newer gccs, but older complained about "defined but not used" functions w/o inline keyword. I'm almost sure

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-21 Thread Anton Vorontsov
On Mon, Apr 21, 2008 at 08:03:31AM -0600, Grant Likely wrote: > 2008/4/18 Anton Vorontsov <[EMAIL PROTECTED]>: > > GTM stands for General-purpose Timers Module and able to generate > > timer{1,2,3,4} interrupts. These timers are used by the drivers that > > need time precise interrupts (like for

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-21 Thread Grant Likely
2008/4/18 Anton Vorontsov <[EMAIL PROTECTED]>: > GTM stands for General-purpose Timers Module and able to generate > timer{1,2,3,4} interrupts. These timers are used by the drivers that > need time precise interrupts (like for USB transactions scheduling for > the Freescale USB Host controller a

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-18 Thread Anton Vorontsov
On Thu, Apr 17, 2008 at 11:19:39PM -0500, Kumar Gala wrote: [...] >>> + * interval value, and fires the interrupt when the value is >>> reached. This >>> + * function will reduce the precision of the timer as needed in >>> order for the >>> + * requested timeout to fit in a 16-bit register. >>>

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-17 Thread Kumar Gala
On Apr 17, 2008, at 5:47 PM, Anton Vorontsov wrote: On Thu, Apr 17, 2008 at 11:28:32PM +0400, Anton Vorontsov wrote: GTM stands for General-purpose Timers Module and able to generate timer{1,2,3,4} interrupts. These timers are used by the drivers that need time precise interrupts (like for USB

Re: [PATCH 1/5] [POWERPC] sysdev: implement FSL GTM support

2008-04-17 Thread Anton Vorontsov
On Thu, Apr 17, 2008 at 11:28:32PM +0400, Anton Vorontsov wrote: > GTM stands for General-purpose Timers Module and able to generate > timer{1,2,3,4} interrupts. These timers are used by the drivers that > need time precise interrupts (like for USB transactions scheduling for > the Freescale USB Ho