On Sun, Sep 04, 2005 Nish Aravamudan wrote: > On 9/4/05, Johannes Stezenbach <[EMAIL PROTECTED]> wrote: > > On Sun, Sep 04, 2005 Nish Aravamudan wrote: > > > On 9/4/05, Johannes Stezenbach <[EMAIL PROTECTED]> wrote: > > > > > > > > -static int numpkt = 0, lastj, numts, numstuff, numsec, numinvalid; > > > > +static int numpkt = 0, numts, numstuff, numsec, numinvalid; > > > > +static unsigned long lastj; > > > > > > > > > > I think you actually want: > > > > > > static void ttusb_iso_irq(....) > > > { > > > unsigned long lastj; > > > > > > > The code in question is used to print a one-per-second debug output, > > and lastj is assigned after every print. > > Ah yes, didn't see that in the current code, sorry for that noise. > Still, lastj is local to ttusb_iso_irq(), right?
Yes, but it needs to be static. Johannes - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/