[coreboot] r844 - in coreboot-v3: mainboard/emulation/qemu-x86 util/dtc

svn at coreboot.org svn at coreboot.org
Fri Aug 29 10:49:24 CEST 2008


Author: hailfinger
Date: 2008-08-29 10:49:24 +0200 (Fri, 29 Aug 2008)
New Revision: 844

Modified:
   coreboot-v3/mainboard/emulation/qemu-x86/dts
   coreboot-v3/util/dtc/flattree.c
Log:
Prepare for per-device subsystem IDs.
Fix the bus location for Qemu IDE.

This patch only provides the needed infrastructure for per-device
subsystem IDs, it does not hook them up to the PCI core yet, so this
patch is a no-op.
By the way, the on_mainboard property is activating lots of completely
untested code paths in v3, so someone might want to audit them.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Acked-by: Ronald G. Minnich <rminnich at gmail.com>
Acked-by: Stefan Reinauer <stepan at coresystems.de>


Modified: coreboot-v3/mainboard/emulation/qemu-x86/dts
===================================================================
--- coreboot-v3/mainboard/emulation/qemu-x86/dts	2008-08-29 04:33:56 UTC (rev 843)
+++ coreboot-v3/mainboard/emulation/qemu-x86/dts	2008-08-29 08:49:24 UTC (rev 844)
@@ -30,8 +30,11 @@
 		bus at 0 {
 			pci at 0,0 {
 			};
-			pci at 1,0 {
+			pci at 1,1 {
 				/config/("southbridge/intel/i82371eb/ide");
+				subsystem_vendor = "0x15ad";
+				subsystem_device = "0x1976";
+				on_mainboard;
 			};
 		};
 	};

Modified: coreboot-v3/util/dtc/flattree.c
===================================================================
--- coreboot-v3/util/dtc/flattree.c	2008-08-29 04:33:56 UTC (rev 843)
+++ coreboot-v3/util/dtc/flattree.c	2008-08-29 08:49:24 UTC (rev 844)
@@ -622,6 +622,12 @@
 		if (streq(prop->name, "on_mainboard")){
 			fprintf(f, "\t.on_mainboard = 1,\n");
 		}
+		if (streq(prop->name, "subsystem_vendor")){
+			fprintf(f, "\t.subsystem_vendor = %s,\n", prop->val.val);
+		}
+		if (streq(prop->name, "subsystem_device")){
+			fprintf(f, "\t.subsystem_device = %s,\n", prop->val.val);
+		}
 		if (streq(prop->name, "enabled")){
 			enabled = 1;
 		}





More information about the coreboot mailing list