[coreboot-gerrit] Patch set updated for coreboot: mainboard/emulation: Use C89 comments style & remove commented code

HAOUAS Elyes (ehaouas@noos.fr) gerrit at coreboot.org
Mon Oct 17 20:58:17 CEST 2016


HAOUAS Elyes (ehaouas at noos.fr) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16916

-gerrit

commit 6349c2b0f5ddef28eaab63b2d6b7c602c6b80bca
Author: Elyes HAOUAS <ehaouas at noos.fr>
Date:   Fri Oct 7 13:02:31 2016 +0200

    mainboard/emulation: Use C89 comments style & remove commented code
    
    Change-Id: I627338505fe1273366bc8f6f528d829b3162b371
    Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
 src/mainboard/emulation/qemu-i440fx/romstage.c  | 3 ---
 src/mainboard/emulation/qemu-power8/bootblock.c | 5 +++--
 src/mainboard/emulation/qemu-power8/mainboard.c | 2 +-
 src/mainboard/emulation/qemu-q35/acpi_tables.c  | 2 +-
 src/mainboard/emulation/qemu-q35/romstage.c     | 3 ---
 5 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/src/mainboard/emulation/qemu-i440fx/romstage.c b/src/mainboard/emulation/qemu-i440fx/romstage.c
index bc23929..1676fb0 100644
--- a/src/mainboard/emulation/qemu-i440fx/romstage.c
+++ b/src/mainboard/emulation/qemu-i440fx/romstage.c
@@ -40,9 +40,6 @@ void * asmlinkage romstage_main(unsigned long bist)
 	/* Halt if there was a built in self test failure */
 	report_bist_failure(bist);
 
-	//print_pci_devices();
-	//dump_pci_devices();
-
 	cbmem_was_initted = !cbmem_recovery(0);
 
 	timestamp_init(timestamp_get());
diff --git a/src/mainboard/emulation/qemu-power8/bootblock.c b/src/mainboard/emulation/qemu-power8/bootblock.c
index d92c169..5cfb7c1 100644
--- a/src/mainboard/emulation/qemu-power8/bootblock.c
+++ b/src/mainboard/emulation/qemu-power8/bootblock.c
@@ -17,8 +17,9 @@
 #include <console/console.h>
 #include <program_loading.h>
 
-// the qemu part of all this is very, very non-hardware like.
-// so it gets its own bootblock.
+/* The qemu part of all this is very, very non-hardware like.
+ * So it gets its own bootblock.
+ */
 void main(void)
 {
 	if (IS_ENABLED(CONFIG_BOOTBLOCK_CONSOLE)) {
diff --git a/src/mainboard/emulation/qemu-power8/mainboard.c b/src/mainboard/emulation/qemu-power8/mainboard.c
index 6413f29..70b858e 100644
--- a/src/mainboard/emulation/qemu-power8/mainboard.c
+++ b/src/mainboard/emulation/qemu-power8/mainboard.c
@@ -24,7 +24,7 @@ static void mainboard_enable(device_t dev)
 		die("No dev0; die\n");
 	}
 
-	// Where does RAM live?
+	/* Where does RAM live? */
 	ram_resource(dev, 0, 2048, 32768);
 	cbmem_recovery(0);
 }
diff --git a/src/mainboard/emulation/qemu-q35/acpi_tables.c b/src/mainboard/emulation/qemu-q35/acpi_tables.c
index 07589e1..1b32c02 100644
--- a/src/mainboard/emulation/qemu-q35/acpi_tables.c
+++ b/src/mainboard/emulation/qemu-q35/acpi_tables.c
@@ -204,7 +204,7 @@ unsigned long acpi_fill_mcfg(unsigned long current)
 		return current;
 
 	reg = pci_read_config32(dev, 0x60);
-	if ((reg & 0x07) != 0x01)  // require enabled + 256MB size
+	if ((reg & 0x07) != 0x01)  /* require enabled + 256MB size */
 		return current;
 
 	current += acpi_create_mcfg_mmconfig((acpi_mcfg_mmconfig_t *) current,
diff --git a/src/mainboard/emulation/qemu-q35/romstage.c b/src/mainboard/emulation/qemu-q35/romstage.c
index 1bb51da..870dd07 100644
--- a/src/mainboard/emulation/qemu-q35/romstage.c
+++ b/src/mainboard/emulation/qemu-q35/romstage.c
@@ -42,9 +42,6 @@ void * asmlinkage romstage_main(unsigned long bist)
 	/* Halt if there was a built in self test failure */
 	report_bist_failure(bist);
 
-	//print_pci_devices();
-	//dump_pci_devices();
-
 	cbmem_was_initted = !cbmem_recovery(0);
 
 	timestamp_init(timestamp_get());



More information about the coreboot-gerrit mailing list