> > Workqueue updates for the Atari EtherNEC driver (does this make sense?)
>
> These changes yes, but while I looked at the driver I noticed a few other
> things.

I'm not sure whether this could introduce more delays - OTOH the work
queue is just a fallback in case I cannot get a timer interrupt.

> > @@ -222,7 +223,7 @@ static int use_poll;
> >   */
> >  static wait_queue_head_t WaitQ;
> >
> > -static struct work_struct tqueue;
> > +static struct delayed_work tqueue;
> >
> >  #ifdef ETHERNEC_USE_POLL
> >  static struct {
>
> Within ETHERNEC_USE_POLL is another work_struct, which is basically
> another delayed work event also calling basically ei_interrupt. Two worker
> doing the same thing?

Right - that could in fact use the same work_struct if I can make sure
they don't step on each other's toes. Or the ETHERNEC_USE_POLL stuff can
just disappear.

> It also would be nice if the driver could be converted to
> module_init/module_exit before it gets merged.

Remotely related - does any m68k driver use the driver model framework
yet? Where would we plug in the initialization of hardware data?

        Michael
-
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to