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@hermann-uwe.de Acked-by: Ronald G. Minnich rminnich@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@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@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@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@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; }
On 20.07.2008 22:27, ron minnich wrote:
Acked-by: Ronald G. Minnich rminnich@gmail.com
Thanks! Committed in revision 697.
Regards, Carl-Daniel
-
Carl-Daniel Hailfinger -
ron minnich