On Sat, 2006-04-08 at 00:10 +0200, Francois Romieu wrote:
> > + spin_lock_irqsave(&priv->lock, flags);
>
> Interruptions are enabled. No need to save/restore.
Agreed. I have found a few other places where it's also true. I'll fix
it. Actually, I'm going to change the locking to accommodate some USB
devices, so changes will be more widespread.
> > + netif_device_attach(dev);
> > +
> > + priv->hw_unavailable--;
> > +
> > + if (priv->open && (! priv->hw_unavailable)) {
> > + err = __orinoco_up(dev);
>
> I wonder if it would be enough to issue hermes_set_irqmask() later
> in __orinoco_up() to release this irq disabled section.
Maybe, but I prefer not to touch this code, since it's about to undergo
a much more radical rework.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
