[coreboot-gerrit] Patch set updated for coreboot: 3845df2 jetway/nf81-t56n-lf: Use hexdump() for dumping ACPI tables
Edward O'Callaghan (eocallaghan@alterapraxis.com)
gerrit at coreboot.org
Mon Apr 14 06:40:24 CEST 2014
Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5466
-gerrit
commit 3845df24cb0de09f1808c738883bbd728cdcbc92
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date: Sun Apr 6 20:18:27 2014 +1000
jetway/nf81-t56n-lf: Use hexdump() for dumping ACPI tables
Use hexdump() instead of a local implementation for dumping ACPI_TABLES.
Change-Id: I20354a4f9dff4105de5af696bb9da4a4f6cca788
Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
src/mainboard/jetway/nf81-t56n-lf/acpi_tables.c | 40 ++++++++-----------------
1 file changed, 12 insertions(+), 28 deletions(-)
diff --git a/src/mainboard/jetway/nf81-t56n-lf/acpi_tables.c b/src/mainboard/jetway/nf81-t56n-lf/acpi_tables.c
index 90f556b..2096ab3 100644
--- a/src/mainboard/jetway/nf81-t56n-lf/acpi_tables.c
+++ b/src/mainboard/jetway/nf81-t56n-lf/acpi_tables.c
@@ -29,23 +29,7 @@
#include <string.h>
#define DUMP_ACPI_TABLES 0
-
-#if DUMP_ACPI_TABLES == 1
-
-static void dump_mem(u32 start, u32 end)
-{
-
- u32 i;
- print_debug("dump_mem:");
- for (i = start; i < end; i++) {
- if ((i & 0xf) == 0) {
- printk(BIOS_DEBUG, "\n%08x:", i);
- }
- printk(BIOS_DEBUG, " %02x", (u8)*((u8 *)i));
- }
- print_debug("\n");
-}
-#endif
+#include <lib.h> /* used for hexdump for DUMP_ACPI_TABLES */
extern const unsigned char AmlCode[];
@@ -285,35 +269,35 @@ unsigned long write_acpi_tables(unsigned long start)
#if DUMP_ACPI_TABLES == 1
printk(BIOS_DEBUG, "rsdp\n");
- dump_mem(rsdp, ((void *)rsdp) + sizeof(acpi_rsdp_t));
+ hexdump(rsdp, sizeof(acpi_rsdp_t));
printk(BIOS_DEBUG, "rsdt\n");
- dump_mem(rsdt, ((void *)rsdt) + sizeof(acpi_rsdt_t));
+ hexdump(rsdt, sizeof(acpi_rsdt_t));
printk(BIOS_DEBUG, "madt\n");
- dump_mem(madt, ((void *)madt) + madt->header.length);
+ hexdump(madt, madt->header.length);
printk(BIOS_DEBUG, "srat\n");
- dump_mem(srat, ((void *)srat) + srat->header.length);
+ hexdump(srat, srat->header.length);
printk(BIOS_DEBUG, "slit\n");
- dump_mem(slit, ((void *)slit) + slit->header.length);
+ hexdump(slit, slit->header.length);
printk(BIOS_DEBUG, "alib\n");
- dump_mem(ssdt, ((void *)alib) + alib->length);
+ hexdump(ssdt, alib->length);
printk(BIOS_DEBUG, "ssdt\n");
- dump_mem(ssdt, ((void *)ssdt) + ssdt->length);
+ hexdump(ssdt, ssdt->length);
printk(BIOS_DEBUG, "ssdt2\n");
- dump_mem(ssdt2, ((void *)ssdt2) + ssdt2->length);
+ hexdump(ssdt2, ssdt2->length);
printk(BIOS_DEBUG, "fadt\n");
- dump_mem(fadt, ((void *)fadt) + fadt->header.length);
+ hexdump(fadt, fadt->header.length);
printk(BIOS_DEBUG, "hest\n");
- dump_mem(hest, ((void *)hest) + hest->header.length);
-#endif
+ hexdump(hest, hest->header.length);
+#endif /* DUMP_ACPI_TABLES */
printk(BIOS_INFO, "ACPI: done.\n");
return current;
More information about the coreboot-gerrit
mailing list