On Wed, Feb 06, 2008 at 07:05:46PM +0100, Uwe Hermann wrote: > On Tue, Feb 05, 2008 at 09:47:18PM +0100, Luc Verhaegen wrote: > > + dev = pci_dev_find(0x1106, 0x3227); /* VT8237 ISA bridge */ > > Is this really meant to be VT8237 or should it be VT8237R?
Hrm... Took the disk out already :( Chip itself is covered by a heatsink. The machine i needed to flash the bios for has a VT8237A, and this has 0x3337 for the ISA bridge. pciids.sf.net has the SP device down for a plain VT8237, confirms the VT8237A and has 0x3372 for VT8237S. I'll guess i'll just give in and install a new suse for the p4m900 and return the other disk to its original owner (the SP). > > + val &= ~0x80; > > Just some cosmetics: I personally find the > > val &= ~(1 << 7); > > notation clearer for manipulating bits. Taste :) > If nobody complains I'll commit with the changes above. Let me get back to you on the identification of the devices. Luc Verhaegen. SUSE/Novell X Driver Developer. -- coreboot mailing list [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
