[coreboot-gerrit] Patch set updated for coreboot: 0bbff3a mainboard/supermicro/h8dme: Drop unused code

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Tue Jun 3 11:21:35 CEST 2014


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5828

-gerrit

commit 0bbff3a78e691c7ef43208175aaf48c09d870367
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sat May 24 01:45:41 2014 +1000

    mainboard/supermicro/h8dme: Drop unused code
    
    Clang complains about a unused debug function, so remove dead code.
    We have copy of dump_smbus_registers() in amdk8/debug.c.
    
    Change-Id: I5a9a785fcd90761789aaa029f712f1feddd2afd1
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/mainboard/supermicro/h8dme/romstage.c | 27 ---------------------------
 1 file changed, 27 deletions(-)

diff --git a/src/mainboard/supermicro/h8dme/romstage.c b/src/mainboard/supermicro/h8dme/romstage.c
index 47f5806..237e043 100644
--- a/src/mainboard/supermicro/h8dme/romstage.c
+++ b/src/mainboard/supermicro/h8dme/romstage.c
@@ -45,33 +45,6 @@
 
 static void memreset(int controllers, const struct mem_controller *ctrl) { }
 
-static inline void dump_smbus_registers(void)
-{
-	u32 device;
-
-	print_debug("\n");
-	for (device = 1; device < 0x80; device++) {
-		int j;
-		if (smbus_read_byte(device, 0) < 0)
-			continue;
-		printk(BIOS_DEBUG, "smbus: %02x", device);
-		for (j = 0; j < 256; j++) {
-			int status;
-			unsigned char byte;
-			status = smbus_read_byte(device, j);
-			if (status < 0) {
-				break;
-			}
-			if ((j & 0xf) == 0) {
-				printk(BIOS_DEBUG, "\n%02x: ", j);
-			}
-			byte = status & 0xff;
-			printk(BIOS_DEBUG, "%02x ", byte);
-		}
-		print_debug("\n");
-	}
-}
-
 static inline void activate_spd_rom(const struct mem_controller *ctrl)
 {
 #if 0



More information about the coreboot-gerrit mailing list