Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 7 Jun 2006 11:23:51 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Gleb Smirnoff <glebius@freebsd.org>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/dev/dc if_dc.c if_dcreg.h
Message-ID:  <200606071123.52816.jhb@freebsd.org>
In-Reply-To: <200606071045.39131.john@baldwin.cx>
References:  <200606071226.k57CQNvK018193@repoman.freebsd.org> <200606071045.39131.john@baldwin.cx>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday 07 June 2006 10:45, John Baldwin wrote:
> On Wednesday 07 June 2006 08:26, Gleb Smirnoff wrote:
> > glebius     2006-06-07 12:26:23 UTC
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/dev/dc           if_dc.c if_dcreg.h 
> >   Log:
> >   Add device IDs for Linksys PCMPC200 Cardbus card.
> >   
> >   PR:             kern/75582
> >   Submitted by:   Gary Palmer
> 
> This doesn't work because you didn't handle the card in attach() and it 
> probably worked by accident prior to my latest rounds of dc cleanups because 
> attach() only checked half the PCI device ID before I fixed it.  The patches 
> on hackers@ for this that I planned to fixup to do this add two new device 
> IDs not just one. :)
> 
> I think you just need to add a 'case DC_DEVID(..., ...):' line to the 
> appropriate entry.  The patches on hackers let you know which entry in 
> attach() they need to be added to.

Specifically, look at www.freebsd.org/~jhb/patches/dc_linksys.patch which is 
relative to HEAD.

-- 
John Baldwin



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