Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 22 Sep 2014 15:20:05 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Warner Losh <imp@bsdimp.com>
Cc:        freebsd-current@freebsd.org, imp@freebsd.org
Subject:   Re: [PATCH] Various fixes to wl(4)
Message-ID:  <2930763.7FbKMgYveJ@ralph.baldwin.cx>
In-Reply-To: <4D1FD834-2861-4D16-9ABF-21B8B07561F8@bsdimp.com>
References:  <1815820.OytfPhNq3X@ralph.baldwin.cx> <4D1FD834-2861-4D16-9ABF-21B8B07561F8@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, September 19, 2014 03:47:16 PM Warner Losh wrote:
> I got rid of my pre-802.11 WaveLAN cards about 8 years go after not h=
aving
> them in a system at all for 8 years. And they were about 4 years obso=
lete
> when I took them out of service=E2=80=A6 I=E2=80=99m not even sure I =
have a machine with an
> ISA slot to test the card, even if I still had it.
>=20
> Sorry I can=E2=80=99t help more :)

If no one tests them then I will happily toss the driver into the dustb=
in,=20
just giving folks a chance to test patches before I do so.  (I'm secret=
ly=20
hoping I can purge several old drivers during the current round of time=
out()=20
purging.)

> Warner
>=20
> On Sep 19, 2014, at 2:07 PM, John Baldwin <jhb@FreeBSD.org> wrote:
> > This patch fixes various issues in wl(4) including:
> >=20
> > - Use bus_space instead of inb/outb.
> > - Use device_printf() and if_printf()
> > - Use callout(9) instead of timeout(9)
> > - Don't hold the driver lock across sleeping actions like
> >=20
> >   bus_teardown_intr(), subyte(), etc.
> >=20
> > - Don't recurse on the driver lock.
> >=20
> > The patch is against HEAD but probably applies to 9 and 10 as well.=

> >=20
> > http://people.freebsd.org/~jhb/patches/wl_cleanup.patch

--=20
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2930763.7FbKMgYveJ>