Re: [Ltib] Zaptel enchancements

[Stuart Hughes]

Hi Vadim,

I've uploaded your patch (with __powerpc__) to the GPP as:
zaptel-1.4.11-hrt-arch.patch

I've also added this to zaptel.spec and updated CVS.

Regards, Stuart

On Tue, 2008-12-02 at 16:33 +0100, Vadim Lebedev wrote:
> Stuart
> 
> I've no idea about PPC, never worked with it.
> But IMO your modification is OK, because if HRTs are not avail on
> given platform the config var will not be defined.
> 
> Thanks
> Vadim
> Stuart Hughes wrote: 
> > Hi Vadim,
> > 
> > One more question, do you know Hi Res timers work on powerpc (ppc).  I'm
> > wondering whether while I'm at it to change it to:
> > 
> >  +#if defined(__i386__) || defined(__x86_64__) || defined(__arm__) || 
> > defined(__powerpc__)
> > 
> > Regards, Stuart
> > 
> > On Tue, 2008-12-02 at 10:29 +0100, Vadim Lebedev wrote:
> >   
> > > Yes, it is correct
> > > 
> > > 
> > > Thanks
> > > Vadim
> > > Le 2 déc. 08 à 09:59, Stuart Hughes a écrit :
> > > 
> > >     
> > > > Hi Vadim,
> > > > 
> > > > I just wanted to check.  Your new patch is much smaller, just:
> > > > 
> > > > diff --exclude CVS -uNr zaptel-1.4.11/kernel/ztdummy.c  
> > > > zaptel-1.4.11.modified/kernel/ztdummy.c
> > > > --- zaptel-1.4.11/kernel/ztdummy.c      2008-02-18  
> > > > 18:32:26.000000000 +0100
> > > > +++ zaptel-1.4.11.modified/kernel/ztdummy.c     2008-10-22  
> > > > 16:36:26.000000000 +0200
> > > > @@ -54,7 +54,7 @@
> > > >  * If using an i386 architecture without a PC real-time clock,
> > > >  * the #define USE_RTC should be commented out.
> > > >  */
> > > > -#if defined(__i386__) || defined(__x86_64__)
> > > > +#if defined(__i386__) || defined(__x86_64__) || defined(__arm__)
> > > > #if LINUX_VERSION_CODE >= VERSION_CODE(2,6,13)
> > > > /* The symbol hrtimer_forward is only exported as of 2.6.22: */
> > > > #if defined(CONFIG_HIGH_RES_TIMERS) && LINUX_VERSION_CODE >=  
> > > > VERSION_CODE(2,6,22)
> > > > 
> > > > Is that all that you need patched?
> > > > 
> > > > Regards, Stuart
> > > > 
> > > > On Mon, 2008-12-01 at 20:48 +0100, Vadim Lebedev wrote:
> > > >       
> > > > > Please disregard the patch file i've sent in previous message...
> > > > > the only file which should be patched is ztdummy.c
> > > > > 
> > > > > I've screwed something just before publishing it
> > > > > 
> > > > > I'm attaching the new version
> > > > > 
> > > > > Thanks
> > > > > Vadim
> > > > > 
> > > > > Vadim Lebedev wrote:
> > > > >         
> > > > > > Hello,
> > > > > > 
> > > > > > I'm submitting two enchancements to zaptel for ltib,
> > > > > > 
> > > > > > 1) patch allowing usage of Hi res timers on non-intel platforms
> > > > > > 
> > > > > > 2) possibility to build in a setup where kernel kbuild output dir is
> > > > > > different  from  kernel  source dir
> > > > > > 
> > > > > > 
> > > > > > (The suggested name for the patch  file would be  :
> > > > > > zaptel-1.4.11-nonintel-hrt.patch
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > Thanks
> > > > > > Vadim
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > ____________________________________________________________________
> > > > > > 
> > > > > > _______________________________________________
> > > > > > LTIB home page: http://bitshrine.org
> > > > > > 
> > > > > > Ltib mailing list
> > > > > > address@hidden
> > > > > > http://lists.nongnu.org/mailman/listinfo/ltib
> > > > > >           
> > > > > _______________________________________________
> > > > > LTIB home page: http://bitshrine.org
> > > > > 
> > > > > Ltib mailing list
> > > > > address@hidden
> > > > > http://lists.nongnu.org/mailman/listinfo/ltib
> > > > >         
> > 
> > 
> >   
>