On Thu, Jun 02, 2011 at 11:55:54PM +0200, Mark Kettenis wrote:
> > Date: Thu, 2 Jun 2011 22:49:43 +0100
> > From: Owain Ainsworth <[email protected]>
> >
> > Been running with this for a couple of weeks, seems to work fine.
> >
> > ok?
> >
> > diff --git dev/pci/if_iwn.c dev/pci/if_iwn.c
> > index 16765f4..a54556b 100644
> > --- dev/pci/if_iwn.c
> > +++ dev/pci/if_iwn.c
> > @@ -338,7 +338,8 @@ iwn_attach(struct device *parent, struct device *self,
> > void *aux)
> > }
> >
> > /* Install interrupt handler. */
> > - if (pci_intr_map(pa, &ih) != 0) {
> > + if (pci_intr_map_msi(pa, &ih) != 0 &&
> > + pci_intr_map(pa, &ih) != 0) {
> > printf(": can't map interrupt\n");
> > return;
> > }
>
> Please keep those calls on a single line, to keep this consistent with
> the other wireless drivers.
I forgot about this diff for a while, but sure:
ok?
diff --git dev/pci/if_iwn.c dev/pci/if_iwn.c
index 16765f4..4ebcb95 100644
--- dev/pci/if_iwn.c
+++ dev/pci/if_iwn.c
@@ -338,7 +338,7 @@ iwn_attach(struct device *parent, struct device *self, void
*aux)
}
/* Install interrupt handler. */
- if (pci_intr_map(pa, &ih) != 0) {
+ if (pci_intr_map_msi(pa, &ih) != 0 && pci_intr_map(pa, &ih) != 0) {
printf(": can't map interrupt\n");
return;
}
--
There is no such thing as fortune. Try again.
