[coreboot] [PATCH] remove annoying noisy message

Myles Watson mylesgw at gmail.com
Fri Apr 2 18:19:24 CEST 2010

> I'm mixed on this one, but at the same time, I bet there are few
> broken devices that return 0 as there were in the old days.

In that case the check could be split, so that the message is only printed
in the cases where it isn't 0xffffffff.  Then only the broken boards would
print the message.

		dummy.path.pci.devfn = devfn;
 		id = pci_read_config32(&dummy, PCI_VENDOR_ID);
 		/* Have we found something?
		 * Some broken boards return 0 if a slot is empty, but
		 * the expected answer is 0xffffffff
		if (id == 0xffffffff) {
 			return NULL;
		if ((id == 0x00000000) || (id == 0x0000ffff) ||
		    (id == 0xffff0000)) {
			printk(BIOS_SPEW, "%s, bad id 0x%x\n",
dev_path(&dummy), id);
 			return NULL;
 		dev = alloc_dev(bus, &dummy.path);


More information about the coreboot mailing list