[coreboot] [PATCH] v3: rename mainboard_part_number to mainboard_name

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Sat Jul 19 15:09:29 CEST 2008


Rename mainboard_part_number to mainboard_name in various places. This
is the logical continuation of r416 which happened a year ago and was
> Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
> Acked-by: Ronald G. Minnich <rminnich at gmail.com>

As an added bonus, we now have consistent naming again, making grepping
the source for dts properties possible.

Build tested on all targets. Patch attached for Gmail users.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Index: corebootv3-mainboard_name/mainboard/adl/msm800sev/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/adl/msm800sev/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/adl/msm800sev/dts	(Arbeitskopie)
@@ -22,7 +22,7 @@
 
 /{
 	mainboard_vendor = "Advanced Digital Logic";
-	mainboard-name = "MSM800SEV";
+	mainboard_name = "MSM800SEV";
 	mainboard_pci_subsystem_vendor = "0x1022";
 	mainboard_pci_subsystem_device = "0x2323";
 	cpus { };
Index: corebootv3-mainboard_name/mainboard/amd/norwich/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/amd/norwich/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/amd/norwich/dts	(Arbeitskopie)
@@ -20,7 +20,7 @@
 
 /{
 	mainboard_vendor = "AMD";
-	mainboard-name = "NORWICH";
+	mainboard_name = "NORWICH";
 	mainboard_pci_subsystem_vendor = "0x1022";
 	mainboard_pci_subsystem_device = "0x2323";
 	cpus { };
Index: corebootv3-mainboard_name/mainboard/amd/db800/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/amd/db800/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/amd/db800/dts	(Arbeitskopie)
@@ -20,7 +20,7 @@
 
 /{
 	mainboard_vendor = "AMD";
-	mainboard-name = "DB800";
+	mainboard_name = "DB800";
 	mainboard_pci_subsystem_vendor = "0x1022";
 	mainboard_pci_subsystem_device = "0x2323";
 	cpus { };
Index: corebootv3-mainboard_name/mainboard/artecgroup/dbe61/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/artecgroup/dbe61/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/artecgroup/dbe61/dts	(Arbeitskopie)
@@ -74,7 +74,7 @@
 
 /{
 	mainboard_vendor = "Artec Group";
-	mainboard-name = "DBE61";
+	mainboard_name = "DBE61";
 	mainboard_pci_subsystem_vendor = "0x1022";
 	mainboard_pci_subsystem_device = "0x2323";
 	cpus { };
Index: corebootv3-mainboard_name/mainboard/artecgroup/dbe62/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/artecgroup/dbe62/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/artecgroup/dbe62/dts	(Arbeitskopie)
@@ -20,7 +20,7 @@
 
 /{
 	mainboard_vendor = "Artec";
-	mainboard-name = "DBE62";
+	mainboard_name = "DBE62";
 	cpus { };
 	apic at 0 {
 		/config/("northbridge/amd/geodelx/apic");
Index: corebootv3-mainboard_name/mainboard/pcengines/alix1c/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/pcengines/alix1c/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/pcengines/alix1c/dts	(Arbeitskopie)
@@ -20,7 +20,7 @@
 
 /{
 	mainboard_vendor = "PC Engines";
-	mainboard-name = "ALIX1.C";
+	mainboard_name = "ALIX1.C";
 	cpus { };
 	apic at 0 {
 		/config/("northbridge/amd/geodelx/apic");
Index: corebootv3-mainboard_name/mainboard/pcengines/alix2c3/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/pcengines/alix2c3/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/pcengines/alix2c3/dts	(Arbeitskopie)
@@ -20,7 +20,7 @@
 
 /{
 	mainboard_vendor = "PC Engines";
-	mainboard-name = "ALIX.2C3";
+	mainboard_name = "ALIX.2C3";
 	cpus { };
 	apic at 0 {
 		/config/("northbridge/amd/geodelx/apic");
Index: corebootv3-mainboard_name/mainboard/emulation/qemu-x86/dts
===================================================================
--- corebootv3-mainboard_name/mainboard/emulation/qemu-x86/dts	(Revision 696)
+++ corebootv3-mainboard_name/mainboard/emulation/qemu-x86/dts	(Arbeitskopie)
@@ -20,7 +20,7 @@
 
 /{
 	mainboard_vendor = "Emulation";
-	mainboard-name = "QEMU x86";
+	mainboard_name = "QEMU x86";
 	mainboard_pci_subsystem_vendor = "0x15ad";
 	mainboard_pci_subsystem_device = "0x1976";
 	device_operations = "qemuvga_pci_ops_dev";
Index: corebootv3-mainboard_name/util/dtc/flattree.c
===================================================================
--- corebootv3-mainboard_name/util/dtc/flattree.c	(Revision 696)
+++ corebootv3-mainboard_name/util/dtc/flattree.c	(Arbeitskopie)
@@ -1328,7 +1328,7 @@
 	extern char *code;
 	struct node *next;
 	extern struct node *first_node;
-	int found_mainboard_vendor = 0, found_mainboard_partnumber = 0, found_mainboard_subsys = 0;
+	int found_mainboard_vendor = 0, found_mainboard_name = 0, found_mainboard_subsys = 0;
 
 	labeltree(bi->dt);
 
@@ -1357,9 +1357,9 @@
 			found_mainboard_vendor = 1;
 			fprintf(f, "const char *mainboard_vendor = \"%s\";\n", prop->val.val);
 		}
-		if (streq(prop->name, "mainboard-name")){
-			found_mainboard_partnumber = 1;
-			fprintf(f, "const char *mainboard_part_number = \"%s\";\n", prop->val.val);
+		if (streq(prop->name, "mainboard_name")){
+			found_mainboard_name = 1;
+			fprintf(f, "const char *mainboard_name = \"%s\";\n", prop->val.val);
 		}
 		if (streq(prop->name, "mainboard_pci_subsystem_vendor")){
 			found_mainboard_subsys++;
@@ -1373,11 +1373,11 @@
 
 	if (! 	found_mainboard_vendor){
 		die("There is no mainboard_vendor property in the root. Please add one."
-			"(and make sure there is a mainboard-name property too");
+			"(and make sure there is a mainboard_name property too");
 	}
 
-	if (! 	found_mainboard_partnumber){
-		die("There is no mainboard-name property in the root. "
+	if (! 	found_mainboard_name){
+		die("There is no mainboard_name property in the root. "
 			"Please add one."
 			"(and make sure there is a mainboard_vendor property too");
 	}
@@ -1436,7 +1436,7 @@
 	fix_next(bi->dt);
 	/* emit any includes that we need  -- TODO: ONLY ONCE PER TYPE*/
 	fprintf(f, "#include <device/device.h>\n#include <device/pci.h>\n");
-	fprintf(f, "extern const char *mainboard_vendor, *mainboard_part_number;\n");
+	fprintf(f, "extern const char *mainboard_vendor, *mainboard_name;\n");
 
 	for_each_property(bi->dt, prop) {
 		if (streq(prop->name, "mainboard_pci_subsystem_vendor")){
Index: corebootv3-mainboard_name/arch/x86/coreboot_table.c
===================================================================
--- corebootv3-mainboard_name/arch/x86/coreboot_table.c	(Revision 696)
+++ corebootv3-mainboard_name/arch/x86/coreboot_table.c	(Arbeitskopie)
@@ -142,14 +142,14 @@
 {
 	struct lb_record *rec;
 	struct lb_mainboard *mainboard;
-	extern char *mainboard_vendor, *mainboard_part_number;
+	extern char *mainboard_vendor, *mainboard_name;
 	rec = lb_new_record(header);
 	mainboard = (struct lb_mainboard *)rec;
 	mainboard->tag = LB_TAG_MAINBOARD;
 
 	mainboard->size = (sizeof(*mainboard) +
 		strlen(mainboard_vendor) + 1 + 
-		strlen(mainboard_part_number) + 1 +
+		strlen(mainboard_name) + 1 +
 		3) & ~3;
 
 	mainboard->vendor_idx = 0;
@@ -158,7 +158,7 @@
 	memcpy(mainboard->strings + mainboard->vendor_idx,
 		mainboard_vendor,      strlen(mainboard_vendor) + 1);
 	memcpy(mainboard->strings + mainboard->part_number_idx,
-		mainboard_part_number, strlen(mainboard_part_number) + 1);
+		mainboard_name, strlen(mainboard_name) + 1);
 
 	return mainboard;
 }


-- 
http://www.hailfinger.org/

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: linuxbios3_mainboard_name_dts.diff
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080719/6b7cdbe6/attachment.ksh>


More information about the coreboot mailing list