Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 4 Feb 2014 14:24:53 -0500
From:      John Baldwin <jhb@freebsd.org>
To:        Gavin Atkinson <gavin@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, imp@freebsd.org
Subject:   Re: svn commit: r261216 - head/sys/dev/pccbb
Message-ID:  <201402041424.54122.jhb@freebsd.org>
In-Reply-To: <alpine.BSF.2.00.1402022216230.33534@ury.york.ac.uk>
References:  <201401271949.s0RJnr7t067977@svn.freebsd.org> <alpine.BSF.2.00.1402022216230.33534@ury.york.ac.uk>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sunday, February 02, 2014 5:34:58 pm Gavin Atkinson wrote:
> On Mon, 27 Jan 2014, John Baldwin wrote:
> > Author: jhb
> > Date: Mon Jan 27 19:49:52 2014
> > New Revision: 261216
> > URL: http://svnweb.freebsd.org/changeset/base/261216
> > 
> > Log:
> >   Explicitly enable I/O and memory decoding in the bridge's command register
> >   when activating an I/O or memory window on the CardBus bridge.
> 
> This fixes some, but not all of my machines.  One in particular, a Toshiba 
> M5 laptop, remains broken by r254263 even with this change.  Specificaly, 
> the laptop does not notice when a card is inserted.
> 
> The attached minimal patch gets things working again, though I don't know 
> if is the correct fix or if a more involved fix is required.
> 
> dmesg before and after that patch:
> 
> http://people.freebsd.org/~gavin/m5-dmesg-before.txt
> http://people.freebsd.org/~gavin/m5-dmesg-after.txt
> 
> The only difference is the cbb register dump, the one bit that I am 
> setting in the patch.

Your patch effectively reverts r254263.  It may be the correct thing to do,
but the question is why. :)  Can you provide 'pciconf -lbc' output for this
device?  (You can just do 'pciconf -lbc pccbb0' in HEAD now)

-- 
John Baldwin



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