From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stuffed Crust Subject: Re: [Linux-fbdev-devel] Fwd: [Suspend-devel] resume not working on acer ferrari 4005 with radeonfb enabled Date: Fri, 17 Nov 2006 09:36:58 -0500 Message-ID: <20061117143658.GB5158@shaftnet.org> References: <200611151109.06956.rjw@sisk.pl> <200611162317.30880.chrmhoffmann@gmail.com> <200611162344.41622.rjw@sisk.pl> <20061117052755.GA23831@shaftnet.org> <1163744220.5940.443.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="oLBj+sq0vYjzfsbl" Return-path: Content-Disposition: inline In-Reply-To: <1163744220.5940.443.camel@localhost.localdomain> Sender: linux-kernel-owner@vger.kernel.org List-Id: Cc: linux-fbdev-devel@lists.sourceforge.net, Christian Hoffmann , Andrew Morton , Christian Hoffmann , LKML , Pavel Machek --oLBj+sq0vYjzfsbl Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Nov 17, 2006 at 05:17:00PM +1100, Benjamin Herrenschmidt wrote: > > radeonfb is still using its own code for saving and restoring PCI=20 > > registers; I'm in the process of fixing it up to use proper PCI > > subsystem calls. That will hopefully work better. =20 > >=20 > > It's possible there's a good reason (other than "nobody's ported it ove= r=20 > > yet") that the radeonfb driver is doing it manually, but I don't know= =20 > > why that would be the case. =20 >=20 > Well, radeonfb has code to bring back some cards from D2 or D3 cold (or > hard reset). It differenciates those states by checking if the config > space has been trashed. We should try to find out some better way. The d2 vs d3 is determined by chipset in advance -- powermacs and some=20 thinkpads use d2, and everyone else uses d3. On resume, we check that same flag, and restore differently. We only=20 checked the config space on D3 resume, and restored everything if the=20 first byte was trashed.. =20 If I understand what you're saying correctly, if we re-write a valid set of pci registers, we'll trash the radeon state? Why _wouldn't_ a D3=20 resume be trashed? - Solomon --=20 Solomon Peachy pizza at shaftnet dot org =20 Melbourne, FL ^^ (mail/jabber/gtalk) ^^ Quidquid latine dictum sit, altum viditur. ICQ: 1318344 --oLBj+sq0vYjzfsbl Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.7 (GNU/Linux) iD8DBQFFXckKPuLgii2759ARAlPdAKCMAaSNaASscvIbRz8/d7rIKiJlJQCeOegj /hGsErmJP2Rhbp7xs/SjXHM= =NqTo -----END PGP SIGNATURE----- --oLBj+sq0vYjzfsbl--