dmidecode-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[dmidecode] [PATCH 06/11] dmidecode: Add helper function pr_attr


From: Jean Delvare
Subject: [dmidecode] [PATCH 06/11] dmidecode: Add helper function pr_attr
Date: Tue, 24 Mar 2020 17:39:35 +0100

Print all first-level attributes through a helper function pr_attr.

Signed-off-by: Jean Delvare <address@hidden>
---
 dmidecode.c | 1361 ++++++++++++++++++++++++++++--------------------------------
 dmioem.c    |   66 +-
 dmioutput.c |   12 
 dmioutput.h |    1 
 4 files changed, 693 insertions(+), 747 deletions(-)

--- dmidecode.orig/dmidecode.c  2020-03-24 15:00:19.058165425 +0100
+++ dmidecode/dmidecode.c       2020-03-24 15:00:33.122336695 +0100
@@ -261,7 +261,7 @@ static void dmi_dump(const struct dmi_he
 }
 
 /* shift is 0 if the value is in bytes, 1 if it is in kilobytes */
-static void dmi_print_memory_size(u64 code, int shift)
+static void dmi_print_memory_size(const char *attr, u64 code, int shift)
 {
        unsigned long capacity;
        u16 split[7];
@@ -301,7 +301,7 @@ static void dmi_print_memory_size(u64 co
        else
                capacity = split[i];
 
-       printf(" %lu %s", capacity, unit[i + shift]);
+       pr_attr(attr, "%lu %s", capacity, unit[i + shift]);
 }
 
 /*
@@ -310,10 +310,19 @@ static void dmi_print_memory_size(u64 co
 
 static void dmi_bios_runtime_size(u32 code)
 {
+       const char *format;
+
        if (code & 0x000003FF)
-               printf(" %u bytes", code);
+       {
+               format = "%u bytes";
+       }
        else
-               printf(" %u kB", code >> 10);
+       {
+               format = "%u kB";
+               code >>= 10;
+       }
+
+       pr_attr("Runtime Size", format, code);
 }
 
 static void dmi_bios_rom_size(u8 code1, u16 code2)
@@ -325,10 +334,10 @@ static void dmi_bios_rom_size(u8 code1,
        if (code1 != 0xFF)
        {
                u64 s = { .l = (code1 + 1) << 6 };
-               dmi_print_memory_size(s, 1);
+               dmi_print_memory_size("ROM Size", s, 1);
        }
        else
-               printf(" %u %s", code2 & 0x3FFF, unit[code2 >> 14]);
+               pr_attr("ROM Size", "%u %s", code2 & 0x3FFF, unit[code2 >> 14]);
 }
 
 static void dmi_bios_characteristics(u64 code, const char *prefix)
@@ -426,7 +435,7 @@ static void dmi_bios_characteristics_x2(
  * 7.2 System Information (Type 1)
  */
 
-static void dmi_system_uuid(const u8 *p, u16 ver)
+static void dmi_system_uuid(const char *attr, const u8 *p, u16 ver)
 {
        int only0xFF = 1, only0x00 = 1;
        int i;
@@ -439,12 +448,18 @@ static void dmi_system_uuid(const u8 *p,
 
        if (only0xFF)
        {
-               printf("Not Present");
+               if (attr)
+                       pr_attr(attr, "Not Present");
+               else
+                       printf("Not Present\n");
                return;
        }
        if (only0x00)
        {
-               printf("Not Settable");
+               if (attr)
+                       pr_attr(attr, "Not Settable");
+               else
+                       printf("Not Settable\n");
                return;
        }
 
@@ -457,13 +472,29 @@ static void dmi_system_uuid(const u8 *p,
         * for older versions.
         */
        if (ver >= 0x0206)
-               
printf("%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                       p[3], p[2], p[1], p[0], p[5], p[4], p[7], p[6],
-                       p[8], p[9], p[10], p[11], p[12], p[13], p[14], p[15]);
-       else
-               
printf("%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x",
-                       p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7],
-                       p[8], p[9], p[10], p[11], p[12], p[13], p[14], p[15]);
+       {
+               if (attr)
+                       pr_attr(attr,
+                               
"%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x",
+                               p[3], p[2], p[1], p[0], p[5], p[4], p[7], p[6],
+                               p[8], p[9], p[10], p[11], p[12], p[13], p[14], 
p[15]);
+               else
+                       
printf("%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x\n",
+                               p[3], p[2], p[1], p[0], p[5], p[4], p[7], p[6],
+                               p[8], p[9], p[10], p[11], p[12], p[13], p[14], 
p[15]);
+       }
+       else
+       {
+               if (attr)
+                       pr_attr(attr,
+                               
"%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x",
+                               p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7],
+                               p[8], p[9], p[10], p[11], p[12], p[13], p[14], 
p[15]);
+               else
+                       
printf("%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x\n",
+                               p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7],
+                               p[8], p[9], p[10], p[11], p[12], p[13], p[14], 
p[15]);
+       }
 }
 
 static const char *dmi_system_wake_up_type(u8 code)
@@ -649,17 +680,17 @@ static const char *dmi_chassis_security_
 static void dmi_chassis_height(u8 code)
 {
        if (code == 0x00)
-               printf(" Unspecified");
+               pr_attr("Height", "Unspecified");
        else
-               printf(" %u U", code);
+               pr_attr("Height", "%u U", code);
 }
 
 static void dmi_chassis_power_cords(u8 code)
 {
        if (code == 0x00)
-               printf(" Unspecified");
+               pr_attr("Number Of Power Cords", "Unspecified");
        else
-               printf(" %u", code);
+               pr_attr("Number Of Power Cords", "%u", code);
 }
 
 static void dmi_chassis_elements(u8 count, u8 len, const u8 *p, const char 
*prefix)
@@ -1052,8 +1083,8 @@ static void dmi_processor_id(const struc
         * This might help learn about new processors supporting the
         * CPUID instruction or another form of identification.
         */
-       printf("%sID: %02X %02X %02X %02X %02X %02X %02X %02X\n",
-               prefix, p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7]);
+       pr_attr("ID", "%02X %02X %02X %02X %02X %02X %02X %02X",
+               p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7]);
 
        if (type == 0x05) /* 80386 */
        {
@@ -1061,8 +1092,9 @@ static void dmi_processor_id(const struc
                /*
                 * 80386 have a different signature.
                 */
-               printf("%sSignature: Type %u, Family %u, Major Stepping %u, 
Minor Stepping %u\n",
-                       prefix, dx >> 12, (dx >> 8) & 0xF,
+               pr_attr("Signature",
+                       "Type %u, Family %u, Major Stepping %u, Minor Stepping 
%u",
+                       dx >> 12, (dx >> 8) & 0xF,
                        (dx >> 4) & 0xF, dx & 0xF);
                return;
        }
@@ -1080,8 +1112,9 @@ static void dmi_processor_id(const struc
                        sig = 1;
                else
                {
-                       printf("%sSignature: Type %u, Family %u, Model %u, 
Stepping %u\n",
-                               prefix, (dx >> 12) & 0x3, (dx >> 8) & 0xF,
+                       pr_attr("Signature",
+                               "Type %u, Family %u, Model %u, Stepping %u",
+                               (dx >> 12) & 0x3, (dx >> 8) & 0xF,
                                (dx >> 4) & 0xF, dx & 0xF);
                        return;
                }
@@ -1097,8 +1130,9 @@ static void dmi_processor_id(const struc
                 */
                if (midr == 0)
                        return;
-               printf("%sSignature: Implementor 0x%02x, Variant 0x%x, 
Architecture %u, Part 0x%03x, Revision %u\n",
-                       prefix, midr >> 24, (midr >> 20) & 0xF,
+               pr_attr("Signature",
+                       "Implementor 0x%02x, Variant 0x%x, Architecture %u, 
Part 0x%03x, Revision %u",
+                       midr >> 24, (midr >> 20) & 0xF,
                        (midr >> 16) & 0xF, (midr >> 4) & 0xFFF, midr & 0xF);
                return;
        }
@@ -1153,15 +1187,15 @@ static void dmi_processor_id(const struc
        switch (sig)
        {
                case 1: /* Intel */
-                       printf("%sSignature: Type %u, Family %u, Model %u, 
Stepping %u\n",
-                               prefix, (eax >> 12) & 0x3,
+                       pr_attr("Signature",
+                               "Type %u, Family %u, Model %u, Stepping %u",
+                               (eax >> 12) & 0x3,
                                ((eax >> 20) & 0xFF) + ((eax >> 8) & 0x0F),
                                ((eax >> 12) & 0xF0) + ((eax >> 4) & 0x0F),
                                eax & 0xF);
                        break;
                case 2: /* AMD, publication #25481 revision 2.28 */
-                       printf("%sSignature: Family %u, Model %u, Stepping 
%u\n",
-                               prefix,
+                       pr_attr("Signature", "Family %u, Model %u, Stepping %u",
                                ((eax >> 8) & 0xF) + (((eax >> 8) & 0xF) == 0xF 
? (eax >> 20) & 0xFF : 0),
                                ((eax >> 4) & 0xF) | (((eax >> 8) & 0xF) == 0xF 
? (eax >> 12) & 0xF0 : 0),
                                eax & 0xF);
@@ -1183,7 +1217,7 @@ static void dmi_processor_id(const struc
        }
 }
 
-static void dmi_processor_voltage(u8 code)
+static void dmi_processor_voltage(const char *attr, u8 code)
 {
        /* 7.5.4 */
        static const char *voltage[] = {
@@ -1194,25 +1228,47 @@ static void dmi_processor_voltage(u8 cod
        int i;
 
        if (code & 0x80)
-               printf(" %.1f V", (float)(code & 0x7f) / 10);
+               pr_attr(attr, "%.1f V", (float)(code & 0x7f) / 10);
        else if ((code & 0x07) == 0x00)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
        {
+               char voltage_str[18];
+               int off = 0;
+
                for (i = 0; i <= 2; i++)
-                       if (code & (1 << i))
-                               printf(" %s", voltage[i]);
+               {
+                       if (code & (1 << i))
+                       {
+                               /* Insert space if not the first value */
+                               off += sprintf(voltage_str + off,
+                                              off ? " %s" :"%s",
+                                              voltage[i]);
+                       }
+               }
+               if (off)
+                       pr_attr(attr, voltage_str);
        }
 }
 
-static void dmi_processor_frequency(const u8 *p)
+static void dmi_processor_frequency(const char *attr, const u8 *p)
 {
        u16 code = WORD(p);
 
        if (code)
-               printf("%u MHz", code);
+       {
+               if (attr)
+                       pr_attr(attr, "%u MHz", code);
+               else
+                       printf("%u MHz\n", code);
+       }
        else
-               printf("Unknown");
+       {
+               if (attr)
+                       pr_attr(attr, "Unknown");
+               else
+                       printf("Unknown\n");
+       }
 }
 
 /* code is assumed to be a 3-bit value */
@@ -1303,17 +1359,18 @@ static const char *dmi_processor_upgrade
        return out_of_spec;
 }
 
-static void dmi_processor_cache(u16 code, const char *level, u16 ver)
+static void dmi_processor_cache(const char *attr, u16 code, const char *level,
+                               u16 ver)
 {
        if (code == 0xFFFF)
        {
                if (ver >= 0x0203)
-                       printf(" Not Provided");
+                       pr_attr(attr, "Not Provided");
                else
-                       printf(" No %s Cache", level);
+                       pr_attr(attr, "No %s Cache", level);
        }
        else
-               printf(" 0x%04X", code);
+               pr_attr(attr, "0x%04X", code);
 }
 
 static void dmi_processor_characteristics(u16 code, const char *prefix)
@@ -1478,46 +1535,48 @@ static void dmi_memory_module_types(u16
 static void dmi_memory_module_connections(u8 code)
 {
        if (code == 0xFF)
-               printf(" None");
+               pr_attr("Bank Connections", "None");
+       else if ((code & 0xF0) == 0xF0)
+               pr_attr("Bank Connections", "%u", code & 0x0F);
+       else if ((code & 0x0F) == 0x0F)
+               pr_attr("Bank Connections", "%u", code >> 4);
        else
-       {
-               if ((code & 0xF0) != 0xF0)
-                       printf(" %u", code >> 4);
-               if ((code & 0x0F) != 0x0F)
-                       printf(" %u", code & 0x0F);
-       }
+               pr_attr("Bank Connections", "%u %u", code >> 4, code & 0x0F);
 }
 
-static void dmi_memory_module_speed(u8 code)
+static void dmi_memory_module_speed(const char *attr, u8 code)
 {
        if (code == 0)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %u ns", code);
+               pr_attr(attr, "%u ns", code);
 }
 
-static void dmi_memory_module_size(u8 code)
+static void dmi_memory_module_size(const char *attr, u8 code)
 {
+       const char *connection;
+
        /* 7.7.2 */
+       if (code & 0x80)
+               connection = " (Double-bank Connection)";
+       else
+               connection = " (Single-bank Connection)";
+
        switch (code & 0x7F)
        {
                case 0x7D:
-                       printf(" Not Determinable");
+                       pr_attr(attr, "Not Determinable%s", connection);
                        break;
                case 0x7E:
-                       printf(" Disabled");
+                       pr_attr(attr, "Disabled%s", connection);
                        break;
                case 0x7F:
-                       printf(" Not Installed");
+                       pr_attr(attr, "Not Installed");
                        return;
                default:
-                       printf(" %u MB", 1 << (code & 0x7F));
+                       pr_attr(attr, "%u MB%s", 1 << (code & 0x7F),
+                               connection);
        }
-
-       if (code & 0x80)
-               printf(" (Double-bank Connection)");
-       else
-               printf(" (Single-bank Connection)");
 }
 
 static void dmi_memory_module_error(u8 code)
@@ -1530,9 +1589,9 @@ static void dmi_memory_module_error(u8 c
        };
 
        if (code & (1 << 2))
-               printf(" See Event Log\n");
+               pr_attr("Error Status", "See Event Log");
        else
-               printf(" %s\n", status[code & 0x03]);
+               pr_attr("Error Status", "%s", status[code & 0x03]);
 }
 
 /*
@@ -1564,7 +1623,7 @@ static const char *dmi_cache_location(u8
        return location[code];
 }
 
-static void dmi_cache_size_2(u32 code)
+static void dmi_cache_size_2(const char *attr, u32 code)
 {
        u64 size;
 
@@ -1581,12 +1640,13 @@ static void dmi_cache_size_2(u32 code)
        }
 
        /* Use a more convenient unit for large cache size */
-       dmi_print_memory_size(size, 1);
+       dmi_print_memory_size(attr, size, 1);
 }
 
-static void dmi_cache_size(u16 code)
+static void dmi_cache_size(const char *attr, u16 code)
 {
-       dmi_cache_size_2((((u32)code & 0x8000LU) << 16) | (code & 0x7FFFLU));
+       dmi_cache_size_2(attr,
+                        (((u32)code & 0x8000LU) << 16) | (code & 0x7FFFLU));
 }
 
 static void dmi_cache_types(u16 code, const char *sep)
@@ -1938,16 +1998,16 @@ static const char *dmi_slot_length(u8 co
        return out_of_spec;
 }
 
-static void dmi_slot_id(u8 code1, u8 code2, u8 type, const char *prefix)
+static void dmi_slot_id(u8 code1, u8 code2, u8 type)
 {
        /* 7.10.5 */
        switch (type)
        {
                case 0x04: /* MCA */
-                       printf("%sID: %u\n", prefix, code1);
+                       pr_attr("ID", "%u", code1);
                        break;
                case 0x05: /* EISA */
-                       printf("%sID: %u\n", prefix, code1);
+                       pr_attr("ID", "%u", code1);
                        break;
                case 0x06: /* PCI */
                case 0x0E: /* PCI */
@@ -1985,10 +2045,10 @@ static void dmi_slot_id(u8 code1, u8 cod
                case 0xBB: /* PCI Express 4 */
                case 0xBC: /* PCI Express 4 */
                case 0xBD: /* PCI Express 4 */
-                       printf("%sID: %u\n", prefix, code1);
+                       pr_attr("ID", "%u", code1);
                        break;
                case 0x07: /* PCMCIA */
-                       printf("%sID: Adapter %u, Socket %u\n", prefix, code1, 
code2);
+                       pr_attr("ID", "Adapter %u, Socket %u", code1, code2);
                        break;
        }
 }
@@ -2031,22 +2091,26 @@ static void dmi_slot_characteristics(u8
        }
 }
 
-static void dmi_slot_segment_bus_func(u16 code1, u8 code2, u8 code3, const 
char *prefix)
+static void dmi_slot_segment_bus_func(u16 code1, u8 code2, u8 code3)
 {
        /* 7.10.8 */
        if (!(code1 == 0xFFFF && code2 == 0xFF && code3 == 0xFF))
-               printf("%sBus Address: %04x:%02x:%02x.%x\n",
-                      prefix, code1, code2, code3 >> 3, code3 & 0x7);
+               pr_attr("Bus Address", "%04x:%02x:%02x.%x",
+                       code1, code2, code3 >> 3, code3 & 0x7);
 }
 
-static void dmi_slot_peers(u8 n, const u8 *data, const char *prefix)
+static void dmi_slot_peers(u8 n, const u8 *data)
 {
+       char attr[16];
        int i;
 
        for (i = 1; i <= n; i++, data += 5)
-               printf("%sPeer Device %d: %04x:%02x:%02x.%x (Width %u)\n",
-                      prefix, i, WORD(data), data[2], data[3] >> 3,
-                      data[3] & 0x07, data[4]);
+       {
+               sprintf(attr, "Peer Device %hu", i);
+               pr_attr(attr, "%04x:%02x:%02x.%x (Width %u)",
+                       WORD(data), data[2], data[3] >> 3, data[3] & 0x07,
+                       data[4]);
+       }
 }
 
 /*
@@ -2074,7 +2138,7 @@ static const char *dmi_on_board_devices_
        return out_of_spec;
 }
 
-static void dmi_on_board_devices(const struct dmi_header *h, const char 
*prefix)
+static void dmi_on_board_devices(const struct dmi_header *h)
 {
        u8 *p = h->data + 4;
        u8 count = (h->length - 0x04) / 2;
@@ -2087,12 +2151,11 @@ static void dmi_on_board_devices(const s
                else
                        pr_handle_name("On Board Device %d Information",
                                       i + 1);
-               printf("%s\tType: %s\n",
-                       prefix, dmi_on_board_devices_type(p[2 * i] & 0x7F));
-               printf("%s\tStatus: %s\n",
-                       prefix, p[2 * i] & 0x80 ? "Enabled" : "Disabled");
-               printf("%s\tDescription: %s\n",
-                       prefix, dmi_string(h, p[2 * i + 1]));
+               pr_attr("Type", "%s",
+                       dmi_on_board_devices_type(p[2 * i] & 0x7F));
+               pr_attr("Status", "%s",
+                       p[2 * i] & 0x80 ? "Enabled" : "Disabled");
+               pr_attr("Description", "%s", dmi_string(h, p[2 * i + 1]));
        }
 }
 
@@ -2100,30 +2163,36 @@ static void dmi_on_board_devices(const s
  * 7.12 OEM Strings (Type 11)
  */
 
-static void dmi_oem_strings(const struct dmi_header *h, const char *prefix)
+static void dmi_oem_strings(const struct dmi_header *h)
 {
+       char attr[11];
        u8 *p = h->data + 4;
        u8 count = p[0x00];
        int i;
 
        for (i = 1; i <= count; i++)
-               printf("%sString %d: %s\n",
-                       prefix, i, dmi_string(h, i));
+       {
+               sprintf(attr, "String %hu", i);
+               pr_attr(attr, "%s",dmi_string(h, i));
+       }
 }
 
 /*
  * 7.13 System Configuration Options (Type 12)
  */
 
-static void dmi_system_configuration_options(const struct dmi_header *h, const 
char *prefix)
+static void dmi_system_configuration_options(const struct dmi_header *h)
 {
+       char attr[11];
        u8 *p = h->data + 4;
        u8 count = p[0x00];
        int i;
 
        for (i = 1; i <= count; i++)
-               printf("%sOption %d: %s\n",
-                       prefix, i, dmi_string(h, i));
+       {
+               sprintf(attr, "Option %hu", i);
+               pr_attr(attr, "%s",dmi_string(h, i));
+       }
 }
 
 /*
@@ -2197,7 +2266,7 @@ static void dmi_event_log_status(u8 code
                "Full" /* 1 */
        };
 
-       printf(" %s, %s",
+       pr_attr("Status", "%s, %s",
                valid[(code >> 0) & 1], full[(code >> 1) & 1]);
 }
 
@@ -2209,16 +2278,17 @@ static void dmi_event_log_address(u8 met
                case 0x00:
                case 0x01:
                case 0x02:
-                       printf(" Index 0x%04X, Data 0x%04X", WORD(p), WORD(p + 
2));
+                       pr_attr("Access Address", "Index 0x%04X, Data 0x%04X",
+                               WORD(p), WORD(p + 2));
                        break;
                case 0x03:
-                       printf(" 0x%08X", DWORD(p));
+                       pr_attr("Access Address", "0x%08X", DWORD(p));
                        break;
                case 0x04:
-                       printf(" 0x%04X", WORD(p));
+                       pr_attr("Access Address", "0x%04X", WORD(p));
                        break;
                default:
-                       printf(" Unknown");
+                       pr_attr("Access Address", "Unknown");
        }
 }
 
@@ -2295,19 +2365,22 @@ static const char *dmi_event_log_descrip
        return out_of_spec;
 }
 
-static void dmi_event_log_descriptors(u8 count, u8 len, const u8 *p, const 
char *prefix)
+static void dmi_event_log_descriptors(u8 count, u8 len, const u8 *p)
 {
        /* 7.16.1 */
+       char attr[16];
        int i;
 
        for (i = 0; i < count; i++)
        {
                if (len >= 0x02)
                {
-                       printf("%sDescriptor %u: %s\n",
-                               prefix, i + 1, 
dmi_event_log_descriptor_type(p[i * len]));
-                       printf("%sData Format %u: %s\n",
-                               prefix, i + 1, 
dmi_event_log_descriptor_format(p[i * len + 1]));
+                       sprintf(attr, "Descriptor %hu", i + 1);
+                       pr_attr(attr, "%s",
+                               dmi_event_log_descriptor_type(p[i * len]));
+                       sprintf(attr, "Data Format %hu", i + 1);
+                       pr_attr(attr, "%s",
+                               dmi_event_log_descriptor_format(p[i * len + 
1]));
                }
        }
 }
@@ -2385,40 +2458,40 @@ static const char *dmi_memory_array_ec_t
 static void dmi_memory_array_error_handle(u16 code)
 {
        if (code == 0xFFFE)
-               printf(" Not Provided");
+               pr_attr("Error Information Handle", "Not Provided");
        else if (code == 0xFFFF)
-               printf(" No Error");
+               pr_attr("Error Information Handle", "No Error");
        else
-               printf(" 0x%04X", code);
+               pr_attr("Error Information Handle", "0x%04X", code);
 }
 
 /*
  * 7.18 Memory Device (Type 17)
  */
 
-static void dmi_memory_device_width(u16 code)
+static void dmi_memory_device_width(const char *attr, u16 code)
 {
        /*
         * If no memory module is present, width may be 0
         */
        if (code == 0xFFFF || code == 0)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %u bits", code);
+               pr_attr(attr, "%u bits", code);
 }
 
 static void dmi_memory_device_size(u16 code)
 {
        if (code == 0)
-               printf(" No Module Installed");
+               pr_attr("Size", "No Module Installed");
        else if (code == 0xFFFF)
-               printf(" Unknown");
+               pr_attr("Size", "Unknown");
        else
        {
                u64 s = { .l = code & 0x7FFF };
                if (!(code & 0x8000))
                        s.l <<= 10;
-               dmi_print_memory_size(s, 1);
+               dmi_print_memory_size("Size", s, 1);
        }
 }
 
@@ -2431,19 +2504,20 @@ static void dmi_memory_device_extended_s
         * as an integer without rounding
         */
        if (code & 0x3FFUL)
-               printf(" %lu MB", (unsigned long)code);
+               pr_attr("Size", "%lu MB", (unsigned long)code);
        else if (code & 0xFFC00UL)
-               printf(" %lu GB", (unsigned long)code >> 10);
+               pr_attr("Size", "%lu GB", (unsigned long)code >> 10);
        else
-               printf(" %lu TB", (unsigned long)code >> 20);
+               pr_attr("Size", "%lu TB", (unsigned long)code >> 20);
 }
 
-static void dmi_memory_voltage_value(u16 code)
+static void dmi_memory_voltage_value(const char *attr, u16 code)
 {
        if (code == 0)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(code % 100 ? " %g V" : " %.1f V", (float)code / 1000);
+               pr_attr(attr, code % 100 ? "%g V" : "%.1f V",
+                       (float)code / 1000);
 }
 
 static const char *dmi_memory_device_form_factor(u8 code)
@@ -2476,11 +2550,11 @@ static const char *dmi_memory_device_for
 static void dmi_memory_device_set(u8 code)
 {
        if (code == 0)
-               printf(" None");
+               pr_attr("Set", "None");
        else if (code == 0xFF)
-               printf(" Unknown");
+               pr_attr("Set", "Unknown");
        else
-               printf(" %u", code);
+               pr_attr("Set", "%u", code);
 }
 
 static const char *dmi_memory_device_type(u8 code)
@@ -2547,25 +2621,29 @@ static void dmi_memory_device_type_detai
                "Unbuffered (Unregistered)",
                "LRDIMM"  /* 15 */
        };
+       char list[172];         /* Update length if you touch the array above */
 
        if ((code & 0xFFFE) == 0)
-               printf(" None");
+               pr_attr("Type Detail", "None");
        else
        {
-               int i;
+               int i, off = 0;
 
+               list[0] = '\0';
                for (i = 1; i <= 15; i++)
                        if (code & (1 << i))
-                               printf(" %s", detail[i - 1]);
+                               off += sprintf(list + off, off ? " %s" : "%s",
+                                              detail[i - 1]);
+               pr_attr("Type Detail", list);
        }
 }
 
-static void dmi_memory_device_speed(u16 code)
+static void dmi_memory_device_speed(const char *attr, u16 code)
 {
        if (code == 0)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %u MT/s", code);
+               pr_attr(attr, "%u MT/s", code);
 }
 
 static void dmi_memory_technology(u8 code)
@@ -2581,9 +2659,9 @@ static void dmi_memory_technology(u8 cod
                "Intel Optane DC persistent memory" /* 0x07 */
        };
        if (code >= 0x01 && code <= 0x07)
-               printf(" %s", technology[code - 0x01]);
+               pr_attr("Memory Technology", "%s", technology[code - 0x01]);
        else
-               printf(" %s", out_of_spec);
+               pr_attr("Memory Technology", "%s", out_of_spec);
 }
 
 static void dmi_memory_operating_mode_capability(u16 code)
@@ -2596,49 +2674,54 @@ static void dmi_memory_operating_mode_ca
                "Byte-accessible persistent memory",
                "Block-accessible persistent memory" /* 5 */
        };
+       char list[99];          /* Update length if you touch the array above */
 
        if ((code & 0xFFFE) == 0)
-               printf(" None");
+               pr_attr("Memory Operating Mode Capability", "None");
        else {
-               int i;
+               int i, off = 0;
 
+               list[0] = '\0';
                for (i = 1; i <= 5; i++)
                        if (code & (1 << i))
-                               printf(" %s", mode[i - 1]);
+                               off += sprintf(list + off, off ? " %s" : "%s",
+                                              mode[i - 1]);
+               pr_attr("Memory Operating Mode Capability", list);
        }
 }
 
-static void dmi_memory_manufacturer_id(u16 code)
+static void dmi_memory_manufacturer_id(const char *attr, u16 code)
 {
        /* 7.18.8 */
        /* 7.18.10 */
        /* LSB is 7-bit Odd Parity number of continuation codes */
        if (code == 0)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" Bank %d, Hex 0x%02X", (code & 0x7F) + 1, code >> 8);
+               pr_attr(attr, "Bank %d, Hex 0x%02X",
+                       (code & 0x7F) + 1, code >> 8);
 }
 
-static void dmi_memory_product_id(u16 code)
+static void dmi_memory_product_id(const char *attr, u16 code)
 {
        /* 7.18.9 */
        /* 7.18.11 */
        if (code == 0)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" 0x%04X", code);
+               pr_attr(attr, "0x%04X", code);
 }
 
-static void dmi_memory_size(u64 code)
+static void dmi_memory_size(const char *attr, u64 code)
 {
        /* 7.18.12 */
        /* 7.18.13 */
        if (code.h == 0xFFFFFFFF && code.l == 0xFFFFFFFF)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else if (code.h == 0x0 && code.l == 0x0)
-               printf(" None");
+               pr_attr(attr, "None");
        else
-               dmi_print_memory_size(code, 0);
+               dmi_print_memory_size(attr, code, 0);
 }
 
 /*
@@ -2704,17 +2787,17 @@ static const char *dmi_memory_error_oper
 static void dmi_memory_error_syndrome(u32 code)
 {
        if (code == 0x00000000)
-               printf(" Unknown");
+               pr_attr("Vendor Syndrome", "Unknown");
        else
-               printf(" 0x%08X", code);
+               pr_attr("Vendor Syndrome", "0x%08X", code);
 }
 
-static void dmi_32bit_memory_error_address(u32 code)
+static void dmi_32bit_memory_error_address(const char *attr, u32 code)
 {
        if (code == 0x80000000)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" 0x%08X", code);
+               pr_attr(attr, "0x%08X", code);
 }
 
 /*
@@ -2724,23 +2807,23 @@ static void dmi_32bit_memory_error_addre
 static void dmi_mapped_address_size(u32 code)
 {
        if (code == 0)
-               printf(" Invalid");
+               pr_attr("Range Size", "Invalid");
        else
        {
                u64 size;
 
                size.h = 0;
                size.l = code;
-               dmi_print_memory_size(size, 1);
+               dmi_print_memory_size("Range Size", size, 1);
        }
 }
 
 static void dmi_mapped_address_extended_size(u64 start, u64 end)
 {
        if (start.h == end.h && start.l == end.l)
-               printf(" Invalid");
+               pr_attr("Range Size", "Invalid");
        else
-               dmi_print_memory_size(u64_range(start, end), 0);
+               dmi_print_memory_size("Range Size", u64_range(start, end), 0);
 }
 
 /*
@@ -2750,36 +2833,32 @@ static void dmi_mapped_address_extended_
 static void dmi_mapped_address_row_position(u8 code)
 {
        if (code == 0)
-               printf(" %s", out_of_spec);
+               pr_attr("Partition Row Position", "%s", out_of_spec);
        else if (code == 0xFF)
-               printf(" Unknown");
+               pr_attr("Partition Row Position", "Unknown");
        else
-               printf(" %u", code);
+               pr_attr("Partition Row Position", "%u", code);
 }
 
-static void dmi_mapped_address_interleave_position(u8 code, const char *prefix)
+static void dmi_mapped_address_interleave_position(u8 code)
 {
        if (code != 0)
        {
-               printf("%sInterleave Position:", prefix);
                if (code == 0xFF)
-                       printf(" Unknown");
+                       pr_attr("Interleave Position", "Unknown");
                else
-                       printf(" %u", code);
-               printf("\n");
+                       pr_attr("Interleave Position", "%u", code);
        }
 }
 
-static void dmi_mapped_address_interleaved_data_depth(u8 code, const char 
*prefix)
+static void dmi_mapped_address_interleaved_data_depth(u8 code)
 {
        if (code != 0)
        {
-               printf("%sInterleaved Data Depth:", prefix);
                if (code == 0xFF)
-                       printf(" Unknown");
+                       pr_attr("Interleaved Data Depth", "Unknown");
                else
-                       printf(" %u", code);
-               printf("\n");
+                       pr_attr("Interleaved Data Depth", "%u", code);
        }
 }
 
@@ -2859,25 +2938,25 @@ static const char *dmi_battery_chemistry
 static void dmi_battery_capacity(u16 code, u8 multiplier)
 {
        if (code == 0)
-               printf(" Unknown");
+               pr_attr("Design Capacity", "Unknown");
        else
-               printf(" %u mWh", code * multiplier);
+               pr_attr("Design Capacity", "%u mWh", code * multiplier);
 }
 
 static void dmi_battery_voltage(u16 code)
 {
        if (code == 0)
-               printf(" Unknown");
+               pr_attr("Design Voltage", "Unknown");
        else
-               printf(" %u mV", code);
+               pr_attr("Design Voltage", "%u mV", code);
 }
 
 static void dmi_battery_maximum_error(u8 code)
 {
        if (code == 0xFF)
-               printf(" Unknown");
+               pr_attr("Maximum Error", "Unknown");
        else
-               printf(" %u%%", code);
+               pr_attr("Maximum Error", "%u%%", code);
 }
 
 /*
@@ -2897,20 +2976,20 @@ static const char *dmi_system_reset_boot
        return option[code];
 }
 
-static void dmi_system_reset_count(u16 code)
+static void dmi_system_reset_count(const char *attr, u16 code)
 {
        if (code == 0xFFFF)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %u", code);
+               pr_attr(attr, "%u", code);
 }
 
-static void dmi_system_reset_timer(u16 code)
+static void dmi_system_reset_timer(const char *attr, u16 code)
 {
        if (code == 0xFFFF)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %u min", code);
+               pr_attr(attr, "%u min", code);
 }
 
 /*
@@ -2935,27 +3014,32 @@ static const char *dmi_hardware_security
 
 static void dmi_power_controls_power_on(const u8 *p)
 {
+       char time[15];
+       int off = 0;
+
        /* 7.26.1 */
        if (dmi_bcd_range(p[0], 0x01, 0x12))
-               printf(" %02X", p[0]);
+               off += sprintf(time + off, "%02X", p[0]);
        else
-               printf(" *");
+               off += sprintf(time + off, "*");
        if (dmi_bcd_range(p[1], 0x01, 0x31))
-               printf("-%02X", p[1]);
+               off += sprintf(time + off, "-%02X", p[1]);
        else
-               printf("-*");
+               off += sprintf(time + off, "-*");
        if (dmi_bcd_range(p[2], 0x00, 0x23))
-               printf(" %02X", p[2]);
+               off += sprintf(time + off, " %02X", p[2]);
        else
-               printf(" *");
+               off += sprintf(time + off, " *");
        if (dmi_bcd_range(p[3], 0x00, 0x59))
-               printf(":%02X", p[3]);
+               off += sprintf(time + off, ":%02X", p[3]);
        else
-               printf(":*");
+               off += sprintf(time + off, ":*");
        if (dmi_bcd_range(p[4], 0x00, 0x59))
-               printf(":%02X", p[4]);
+               off += sprintf(time + off, ":%02X", p[4]);
        else
-               printf(":*");
+               off += sprintf(time + off, ":*");
+
+       pr_attr("Next Scheduled Power-on", time);
 }
 
 /*
@@ -3001,28 +3085,28 @@ static const char *dmi_probe_status(u8 c
        return out_of_spec;
 }
 
-static void dmi_voltage_probe_value(u16 code)
+static void dmi_voltage_probe_value(const char *attr, u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %.3f V", (float)(i16)code / 1000);
+               pr_attr(attr, "%.3f V", (float)(i16)code / 1000);
 }
 
 static void dmi_voltage_probe_resolution(u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr("Resolution", "Unknown");
        else
-               printf(" %.1f mV", (float)code / 10);
+               pr_attr("Resolution", "%.1f mV", (float)code / 10);
 }
 
 static void dmi_probe_accuracy(u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr("Accuracy", "Unknown");
        else
-               printf(" %.2f%%", (float)code / 100);
+               pr_attr("Accuracy", "%.2f%%", (float)code / 100);
 }
 
 /*
@@ -3058,9 +3142,9 @@ static const char *dmi_cooling_device_ty
 static void dmi_cooling_device_speed(u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown Or Non-rotating");
+               pr_attr("Nominal Speed", "Unknown Or Non-rotating");
        else
-               printf(" %u rpm", code);
+               pr_attr("Nominal Speed", "%u rpm", code);
 }
 
 /*
@@ -3093,40 +3177,40 @@ static const char *dmi_temperature_probe
        return out_of_spec;
 }
 
-static void dmi_temperature_probe_value(u16 code)
+static void dmi_temperature_probe_value(const char *attr, u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %.1f deg C", (float)(i16)code / 10);
+               pr_attr(attr, "%.1f deg C", (float)(i16)code / 10);
 }
 
 static void dmi_temperature_probe_resolution(u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr("Resolution", "Unknown");
        else
-               printf(" %.3f deg C", (float)code / 1000);
+               pr_attr("Resolution", "%.3f deg C", (float)code / 1000);
 }
 
 /*
  * 7.30 Electrical Current Probe (Type 29)
  */
 
-static void dmi_current_probe_value(u16 code)
+static void dmi_current_probe_value(const char *attr, u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" %.3f A", (float)(i16)code / 1000);
+               pr_attr(attr, "%.3f A", (float)(i16)code / 1000);
 }
 
 static void dmi_current_probe_resolution(u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr("Resolution", "Unknown");
        else
-               printf(" %.1f mA", (float)code / 10);
+               pr_attr("Resolution", "%.1f mA", (float)code / 10);
 }
 
 /*
@@ -3160,12 +3244,12 @@ static const char *dmi_system_boot_statu
  * 7.34 64-bit Memory Error Information (Type 33)
  */
 
-static void dmi_64bit_memory_error_address(u64 code)
+static void dmi_64bit_memory_error_address(const char *attr, u64 code)
 {
        if (code.h == 0x80000000 && code.l == 0x00000000)
-               printf(" Unknown");
+               pr_attr(attr, "Unknown");
        else
-               printf(" 0x%08X%08X", code.h, code.l);
+               pr_attr(attr, "0x%08X%08X", code.h, code.l);
 }
 
 /*
@@ -3253,17 +3337,20 @@ static const char *dmi_memory_channel_ty
        return out_of_spec;
 }
 
-static void dmi_memory_channel_devices(u8 count, const u8 *p, const char 
*prefix)
+static void dmi_memory_channel_devices(u8 count, const u8 *p)
 {
+       char attr[18];
        int i;
 
        for (i = 1; i <= count; i++)
        {
-               printf("%sDevice %u Load: %u\n",
-                       prefix, i, p[3 * i]);
+               sprintf(attr, "Device %hu Load", i);
+               pr_attr(attr, "%u", p[3 * i]);
                if (!(opt.flags & FLAG_QUIET))
-                       printf("%sDevice %u Handle: 0x%04X\n",
-                               prefix, i, WORD(p + 3 * i + 1));
+               {
+                       sprintf(attr, "Device %hu Handle", i);
+                       pr_attr(attr, "0x%04X", WORD(p + 3 * i + 1));
+               }
        }
 }
 
@@ -3291,12 +3378,13 @@ static void dmi_ipmi_base_address(u8 typ
 {
        if (type == 0x04) /* SSIF */
        {
-               printf("0x%02X (SMBus)", (*p) >> 1);
+               pr_attr("Base Address", "0x%02X (SMBus)", (*p) >> 1);
        }
        else
        {
                u64 address = QWORD(p);
-               printf("0x%08X%08X (%s)", address.h, (address.l & ~1) | lsb,
+               pr_attr("Base Address", "0x%08X%08X (%s)",
+                       address.h, (address.l & ~1) | lsb,
                        address.l & 1 ? "I/O" : "Memory-mapped");
        }
 }
@@ -3322,9 +3410,9 @@ static const char *dmi_ipmi_register_spa
 static void dmi_power_supply_power(u16 code)
 {
        if (code == 0x8000)
-               printf(" Unknown");
+               pr_attr("Max Power Capacity", "Unknown");
        else
-               printf(" %u W", (unsigned int)code);
+               pr_attr("Max Power Capacity", "%u W", (unsigned int)code);
 }
 
 static const char *dmi_power_supply_type(u8 code)
@@ -3387,7 +3475,7 @@ static const char *dmi_power_supply_rang
  * whether it's worth the effort.
  */
 
-static void dmi_additional_info(const struct dmi_header *h, const char *prefix)
+static void dmi_additional_info(const struct dmi_header *h)
 {
        u8 *p = h->data + 4;
        u8 count = *p++;
@@ -3403,30 +3491,28 @@ static void dmi_additional_info(const st
                length = p[0x00];
                if (length < 0x05 || h->length < offset + length) break;
 
-               printf("%s\tReferenced Handle: 0x%04x\n",
-                       prefix, WORD(p + 0x01));
-               printf("%s\tReferenced Offset: 0x%02x\n",
-                       prefix, p[0x03]);
-               printf("%s\tString: %s\n",
-                       prefix, dmi_string(h, p[0x04]));
+               pr_attr("Referenced Handle", "0x%04x",
+                       WORD(p + 0x01));
+               pr_attr("Referenced Offset", "0x%02x",
+                       p[0x03]);
+               pr_attr("String", "%s",
+                       dmi_string(h, p[0x04]));
 
-               printf("%s\tValue: ", prefix);
                switch (length - 0x05)
                {
                        case 1:
-                               printf("0x%02x", p[0x05]);
+                               pr_attr("Value", "0x%02x", p[0x05]);
                                break;
                        case 2:
-                               printf("0x%04x", WORD(p + 0x05));
+                               pr_attr("Value", "0x%04x", WORD(p + 0x05));
                                break;
                        case 4:
-                               printf("0x%08x", DWORD(p + 0x05));
+                               pr_attr("Value", "0x%08x", DWORD(p + 0x05));
                                break;
                        default:
-                               printf("Unexpected size");
+                               pr_attr("Value", "Unexpected size");
                                break;
                }
-               printf("\n");
 
                p += length;
                offset += length;
@@ -3580,8 +3666,7 @@ static void dmi_parse_protocol_record(co
         * we can just pick a sufficiently recent version here.
         */
        printf("%s\tService UUID: ", prefix);
-       dmi_system_uuid(&rdata[0], 0x311);
-       printf("\n");
+       dmi_system_uuid(NULL, &rdata[0], 0x311);        /* FIXME */
 
        /*
         * DSP0270: 8.6: Redfish Over IP Host IP Assignment Type
@@ -3710,7 +3795,7 @@ static void dmi_parse_controller_structu
                return;
 
        type = data[0x4];
-       printf("%sHost Interface Type: %s\n", prefix,
+       pr_attr("Host Interface Type", "%s",
                dmi_management_controller_host_type(type));
 
        /*
@@ -3725,17 +3810,17 @@ static void dmi_parse_controller_structu
                /* DSP0270: 8.3 Table 2: Device Type */
                type = data[0x6];
 
-               printf("%sDevice Type: %s\n", prefix,
+               pr_attr("Device Type", "%s",
                        dmi_parse_device_type(type));
                if (type == 0x2 && len >= 5)
                {
                        /* USB Device Type - need at least 6 bytes */
                        u8 *usbdata = &data[0x7];
                        /* USB Device Descriptor: idVendor */
-                       printf("%sidVendor: 0x%04x\n", prefix,
+                       pr_attr("idVendor", "0x%04x",
                                WORD(&usbdata[0x0]));
                        /* USB Device Descriptor: idProduct */
-                       printf("%sidProduct: 0x%04x\n", prefix,
+                       pr_attr("idProduct", "0x%04x",
                                WORD(&usbdata[0x2]));
                        /*
                         * USB Serial number is here, but its useless, don't
@@ -3747,16 +3832,16 @@ static void dmi_parse_controller_structu
                        /* PCI Device Type - Need at least 8 bytes */
                        u8 *pcidata = &data[0x7];
                        /* PCI Device Descriptor: VendorID */
-                       printf("%sVendorID: 0x%04x\n", prefix,
+                       pr_attr("VendorID", "0x%04x",
                                WORD(&pcidata[0x0]));
                        /* PCI Device Descriptor: DeviceID */
-                       printf("%sDeviceID: 0x%04x\n", prefix,
+                       pr_attr("DeviceID", "0x%04x",
                                WORD(&pcidata[0x2]));
                        /* PCI Device Descriptor: PCI SubvendorID */
-                       printf("%sSubVendorID: 0x%04x\n", prefix,
+                       pr_attr("SubVendorID", "0x%04x",
                                WORD(&pcidata[0x4]));
                        /* PCI Device Descriptor: PCI SubdeviceID */
-                       printf("%sSubDeviceID: 0x%04x\n", prefix,
+                       pr_attr("SubDeviceID", "0x%04x",
                                WORD(&pcidata[0x6]));
                }
                else if (type == 0x4 && len >= 5)
@@ -3764,8 +3849,8 @@ static void dmi_parse_controller_structu
                        /* OEM Device Type - Need at least 4 bytes */
                        u8 *oemdata = &data[0x7];
                        /* OEM Device Descriptor: IANA */
-                       printf("%sVendor ID: 0x%02x:0x%02x:0x%02x:0x%02x\n",
-                               prefix, oemdata[0x0], oemdata[0x1],
+                       pr_attr("Vendor ID", "0x%02x:0x%02x:0x%02x:0x%02x",
+                               oemdata[0x0], oemdata[0x1],
                                oemdata[0x2], oemdata[0x3]);
                }
                /* Don't mess with unknown types for now */
@@ -3848,7 +3933,7 @@ static void dmi_tpm_vendor_id(const u8 *
        /* Terminate the string */
        vendor_id[i] = '\0';
 
-       printf(" %s", vendor_id);
+       pr_attr("Vendor ID", "%s", vendor_id);
 }
 
 static void dmi_tpm_characteristics(u64 code, const char *prefix)
@@ -3894,11 +3979,11 @@ static void dmi_decode(const struct dmi_
                case 0: /* 7.1 BIOS Information */
                        pr_handle_name("BIOS Information");
                        if (h->length < 0x12) break;
-                       printf("\tVendor: %s\n",
+                       pr_attr("Vendor", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tVersion: %s\n",
+                       pr_attr("Version", "%s",
                                dmi_string(h, data[0x05]));
-                       printf("\tRelease Date: %s\n",
+                       pr_attr("Release Date", "%s",
                                dmi_string(h, data[0x08]));
                        /*
                         * On IA-64, the BIOS base address will read 0 because
@@ -3907,15 +3992,11 @@ static void dmi_decode(const struct dmi_
                         */
                        if (WORD(data + 0x06) != 0)
                        {
-                               printf("\tAddress: 0x%04X0\n",
+                               pr_attr("Address", "0x%04X0",
                                        WORD(data + 0x06));
-                               printf("\tRuntime Size:");
                                dmi_bios_runtime_size((0x10000 - WORD(data + 
0x06)) << 4);
-                               printf("\n");
                        }
-                       printf("\tROM Size:");
                        dmi_bios_rom_size(data[0x09], h->length < 0x1A ? 16 : 
WORD(data + 0x18));
-                       printf("\n");
                        printf("\tCharacteristics:\n");
                        dmi_bios_characteristics(QWORD(data + 0x0A), "\t\t");
                        if (h->length < 0x13) break;
@@ -3924,61 +4005,59 @@ static void dmi_decode(const struct dmi_
                        dmi_bios_characteristics_x2(data[0x13], "\t\t");
                        if (h->length < 0x18) break;
                        if (data[0x14] != 0xFF && data[0x15] != 0xFF)
-                               printf("\tBIOS Revision: %u.%u\n",
+                               pr_attr("BIOS Revision", "%u.%u",
                                        data[0x14], data[0x15]);
                        if (data[0x16] != 0xFF && data[0x17] != 0xFF)
-                               printf("\tFirmware Revision: %u.%u\n",
+                               pr_attr("Firmware Revision", "%u.%u",
                                        data[0x16], data[0x17]);
                        break;
 
                case 1: /* 7.2 System Information */
                        pr_handle_name("System Information");
                        if (h->length < 0x08) break;
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tProduct Name: %s\n",
+                       pr_attr("Product Name", "%s",
                                dmi_string(h, data[0x05]));
-                       printf("\tVersion: %s\n",
+                       pr_attr("Version", "%s",
                                dmi_string(h, data[0x06]));
-                       printf("\tSerial Number: %s\n",
+                       pr_attr("Serial Number", "%s",
                                dmi_string(h, data[0x07]));
                        if (h->length < 0x19) break;
-                       printf("\tUUID: ");
-                       dmi_system_uuid(data + 0x08, ver);
-                       printf("\n");
-                       printf("\tWake-up Type: %s\n",
+                       dmi_system_uuid("UUID", data + 0x08, ver);
+                       pr_attr("Wake-up Type", "%s",
                                dmi_system_wake_up_type(data[0x18]));
                        if (h->length < 0x1B) break;
-                       printf("\tSKU Number: %s\n",
+                       pr_attr("SKU Number", "%s",
                                dmi_string(h, data[0x19]));
-                       printf("\tFamily: %s\n",
+                       pr_attr("Family", "%s",
                                dmi_string(h, data[0x1A]));
                        break;
 
                case 2: /* 7.3 Base Board Information */
                        pr_handle_name("Base Board Information");
                        if (h->length < 0x08) break;
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tProduct Name: %s\n",
+                       pr_attr("Product Name", "%s",
                                dmi_string(h, data[0x05]));
-                       printf("\tVersion: %s\n",
+                       pr_attr("Version", "%s",
                                dmi_string(h, data[0x06]));
-                       printf("\tSerial Number: %s\n",
+                       pr_attr("Serial Number", "%s",
                                dmi_string(h, data[0x07]));
                        if (h->length < 0x09) break;
-                       printf("\tAsset Tag: %s\n",
+                       pr_attr("Asset Tag", "%s",
                                dmi_string(h, data[0x08]));
                        if (h->length < 0x0A) break;
                        printf("\tFeatures:");
                        dmi_base_board_features(data[0x09], "\t\t");
                        if (h->length < 0x0E) break;
-                       printf("\tLocation In Chassis: %s\n",
+                       pr_attr("Location In Chassis", "%s",
                                dmi_string(h, data[0x0A]));
                        if (!(opt.flags & FLAG_QUIET))
-                               printf("\tChassis Handle: 0x%04X\n",
+                               pr_attr("Chassis Handle", "0x%04X",
                                        WORD(data + 0x0B));
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_base_board_type(data[0x0D]));
                        if (h->length < 0x0F) break;
                        if (h->length < 0x0F + data[0x0E] * sizeof(u16)) break;
@@ -3989,109 +4068,94 @@ static void dmi_decode(const struct dmi_
                case 3: /* 7.4 Chassis Information */
                        pr_handle_name("Chassis Information");
                        if (h->length < 0x09) break;
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_chassis_type(data[0x05]));
-                       printf("\tLock: %s\n",
+                       pr_attr("Lock", "%s",
                                dmi_chassis_lock(data[0x05] >> 7));
-                       printf("\tVersion: %s\n",
+                       pr_attr("Version", "%s",
                                dmi_string(h, data[0x06]));
-                       printf("\tSerial Number: %s\n",
+                       pr_attr("Serial Number", "%s",
                                dmi_string(h, data[0x07]));
-                       printf("\tAsset Tag: %s\n",
+                       pr_attr("Asset Tag", "%s",
                                dmi_string(h, data[0x08]));
                        if (h->length < 0x0D) break;
-                       printf("\tBoot-up State: %s\n",
+                       pr_attr("Boot-up State", "%s",
                                dmi_chassis_state(data[0x09]));
-                       printf("\tPower Supply State: %s\n",
+                       pr_attr("Power Supply State", "%s",
                                dmi_chassis_state(data[0x0A]));
-                       printf("\tThermal State: %s\n",
+                       pr_attr("Thermal State", "%s",
                                dmi_chassis_state(data[0x0B]));
-                       printf("\tSecurity Status: %s\n",
+                       pr_attr("Security Status", "%s",
                                dmi_chassis_security_status(data[0x0C]));
                        if (h->length < 0x11) break;
-                       printf("\tOEM Information: 0x%08X\n",
+                       pr_attr("OEM Information", "0x%08X",
                                DWORD(data + 0x0D));
                        if (h->length < 0x13) break;
-                       printf("\tHeight:");
                        dmi_chassis_height(data[0x11]);
-                       printf("\n");
-                       printf("\tNumber Of Power Cords:");
                        dmi_chassis_power_cords(data[0x12]);
-                       printf("\n");
                        if (h->length < 0x15) break;
                        if (h->length < 0x15 + data[0x13] * data[0x14]) break;
                        dmi_chassis_elements(data[0x13], data[0x14], data + 
0x15, "\t");
                        if (h->length < 0x16 + data[0x13] * data[0x14]) break;
-                       printf("\tSKU Number: %s\n",
+                       pr_attr("SKU Number", "%s",
                                dmi_string(h, data[0x15 + data[0x13] * 
data[0x14]]));
                        break;
 
                case 4: /* 7.5 Processor Information */
                        pr_handle_name("Processor Information");
                        if (h->length < 0x1A) break;
-                       printf("\tSocket Designation: %s\n",
+                       pr_attr("Socket Designation", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_processor_type(data[0x05]));
-                       printf("\tFamily: %s\n",
+                       pr_attr("Family", "%s",
                                dmi_processor_family(h, ver));
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x07]));
                        dmi_processor_id(h, "\t");
-                       printf("\tVersion: %s\n",
+                       pr_attr("Version", "%s",
                                dmi_string(h, data[0x10]));
-                       printf("\tVoltage:");
-                       dmi_processor_voltage(data[0x11]);
-                       printf("\n");
-                       printf("\tExternal Clock: ");
-                       dmi_processor_frequency(data + 0x12);
-                       printf("\n");
-                       printf("\tMax Speed: ");
-                       dmi_processor_frequency(data + 0x14);
-                       printf("\n");
-                       printf("\tCurrent Speed: ");
-                       dmi_processor_frequency(data + 0x16);
-                       printf("\n");
+                       dmi_processor_voltage("Voltage", data[0x11]);
+                       dmi_processor_frequency("External Clock", data + 0x12);
+                       dmi_processor_frequency("Max Speed", data + 0x14);
+                       dmi_processor_frequency("Current Speed", data + 0x16);
                        if (data[0x18] & (1 << 6))
-                               printf("\tStatus: Populated, %s\n",
+                               pr_attr("Status", "Populated, %s",
                                        dmi_processor_status(data[0x18] & 
0x07));
                        else
-                               printf("\tStatus: Unpopulated\n");
-                       printf("\tUpgrade: %s\n",
+                               pr_attr("Status", "Unpopulated");
+                       pr_attr("Upgrade", "%s",
                                dmi_processor_upgrade(data[0x19]));
                        if (h->length < 0x20) break;
                        if (!(opt.flags & FLAG_QUIET))
                        {
-                               printf("\tL1 Cache Handle:");
-                               dmi_processor_cache(WORD(data + 0x1A), "L1", 
ver);
-                               printf("\n");
-                               printf("\tL2 Cache Handle:");
-                               dmi_processor_cache(WORD(data + 0x1C), "L2", 
ver);
-                               printf("\n");
-                               printf("\tL3 Cache Handle:");
-                               dmi_processor_cache(WORD(data + 0x1E), "L3", 
ver);
-                               printf("\n");
+                               dmi_processor_cache("L1 Cache Handle",
+                                                   WORD(data + 0x1A), "L1", 
ver);
+                               dmi_processor_cache("L2 Cache Handle",
+                                                   WORD(data + 0x1C), "L2", 
ver);
+                               dmi_processor_cache("L3 Cache Handle",
+                                                   WORD(data + 0x1E), "L3", 
ver);
                        }
                        if (h->length < 0x23) break;
-                       printf("\tSerial Number: %s\n",
+                       pr_attr("Serial Number", "%s",
                                dmi_string(h, data[0x20]));
-                       printf("\tAsset Tag: %s\n",
+                       pr_attr("Asset Tag", "%s",
                                dmi_string(h, data[0x21]));
-                       printf("\tPart Number: %s\n",
+                       pr_attr("Part Number", "%s",
                                dmi_string(h, data[0x22]));
                        if (h->length < 0x28) break;
                        if (data[0x23] != 0)
-                               printf("\tCore Count: %u\n",
+                               pr_attr("Core Count", "%u",
                                        h->length >= 0x2C && data[0x23] == 0xFF 
?
                                        WORD(data + 0x2A) : data[0x23]);
                        if (data[0x24] != 0)
-                               printf("\tCore Enabled: %u\n",
+                               pr_attr("Core Enabled", "%u",
                                        h->length >= 0x2E && data[0x24] == 0xFF 
?
                                        WORD(data + 0x2C) : data[0x24]);
                        if (data[0x25] != 0)
-                               printf("\tThread Count: %u\n",
+                               pr_attr("Thread Count", "%u",
                                        h->length >= 0x30 && data[0x25] == 0xFF 
?
                                        WORD(data + 0x2E) : data[0x25]);
                        printf("\tCharacteristics:");
@@ -4101,26 +4165,24 @@ static void dmi_decode(const struct dmi_
                case 5: /* 7.6 Memory Controller Information */
                        pr_handle_name("Memory Controller Information");
                        if (h->length < 0x0F) break;
-                       printf("\tError Detecting Method: %s\n",
+                       pr_attr("Error Detecting Method", "%s",
                                dmi_memory_controller_ed_method(data[0x04]));
                        printf("\tError Correcting Capabilities:");
                        dmi_memory_controller_ec_capabilities(data[0x05], 
"\t\t");
-                       printf("\tSupported Interleave: %s\n",
+                       pr_attr("Supported Interleave", "%s",
                                dmi_memory_controller_interleave(data[0x06]));
-                       printf("\tCurrent Interleave: %s\n",
+                       pr_attr("Current Interleave", "%s",
                                dmi_memory_controller_interleave(data[0x07]));
-                       printf("\tMaximum Memory Module Size: %u MB\n",
+                       pr_attr("Maximum Memory Module Size", "%u MB",
                                1 << data[0x08]);
-                       printf("\tMaximum Total Memory Size: %u MB\n",
+                       pr_attr("Maximum Total Memory Size", "%u MB",
                                data[0x0E] * (1 << data[0x08]));
                        printf("\tSupported Speeds:");
                        dmi_memory_controller_speeds(WORD(data + 0x09), "\t\t");
                        printf("\tSupported Memory Types:");
                        dmi_memory_module_types(WORD(data + 0x0B), "\n\t\t");
                        printf("\n");
-                       printf("\tMemory Module Voltage:");
-                       dmi_processor_voltage(data[0x0D]);
-                       printf("\n");
+                       dmi_processor_voltage("Memory Module Voltage", 
data[0x0D]);
                        if (h->length < 0x0F + data[0x0E] * sizeof(u16)) break;
                        dmi_memory_controller_slots(data[0x0E], data + 0x0F, 
"\t");
                        if (h->length < 0x10 + data[0x0E] * sizeof(u16)) break;
@@ -4131,52 +4193,39 @@ static void dmi_decode(const struct dmi_
                case 6: /* 7.7 Memory Module Information */
                        pr_handle_name("Memory Module Information");
                        if (h->length < 0x0C) break;
-                       printf("\tSocket Designation: %s\n",
+                       pr_attr("Socket Designation", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tBank Connections:");
                        dmi_memory_module_connections(data[0x05]);
-                       printf("\n");
-                       printf("\tCurrent Speed:");
-                       dmi_memory_module_speed(data[0x06]);
-                       printf("\n");
+                       dmi_memory_module_speed("Current Speed", data[0x06]);
                        printf("\tType:");
                        dmi_memory_module_types(WORD(data + 0x07), " ");
                        printf("\n");
-                       printf("\tInstalled Size:");
-                       dmi_memory_module_size(data[0x09]);
-                       printf("\n");
-                       printf("\tEnabled Size:");
-                       dmi_memory_module_size(data[0x0A]);
-                       printf("\n");
-                       printf("\tError Status:");
+                       dmi_memory_module_size("Installed Size", data[0x09]);
+                       dmi_memory_module_size("Enabled Size", data[0x0A]);
                        dmi_memory_module_error(data[0x0B]);
                        break;
 
                case 7: /* 7.8 Cache Information */
                        pr_handle_name("Cache Information");
                        if (h->length < 0x0F) break;
-                       printf("\tSocket Designation: %s\n",
+                       pr_attr("Socket Designation", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tConfiguration: %s, %s, Level %u\n",
+                       pr_attr("Configuration", "%s, %s, Level %u",
                                WORD(data + 0x05) & 0x0080 ? "Enabled" : 
"Disabled",
                                WORD(data + 0x05) & 0x0008 ? "Socketed" : "Not 
Socketed",
                                (WORD(data + 0x05) & 0x0007) + 1);
-                       printf("\tOperational Mode: %s\n",
+                       pr_attr("Operational Mode", "%s",
                                dmi_cache_mode((WORD(data + 0x05) >> 8) & 
0x0003));
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_cache_location((WORD(data + 0x05) >> 5) & 
0x0003));
-                       printf("\tInstalled Size:");
                        if (h->length >= 0x1B)
-                               dmi_cache_size_2(DWORD(data + 0x17));
+                               dmi_cache_size_2("Installed Size", DWORD(data + 
0x17));
                        else
-                               dmi_cache_size(WORD(data + 0x09));
-                       printf("\n");
-                       printf("\tMaximum Size:");
+                               dmi_cache_size("Installed Size", WORD(data + 
0x09));
                        if (h->length >= 0x17)
-                               dmi_cache_size_2(DWORD(data + 0x13));
+                               dmi_cache_size_2("Maximum Size", DWORD(data + 
0x13));
                        else
-                               dmi_cache_size(WORD(data + 0x07));
-                       printf("\n");
+                               dmi_cache_size("Maximum Size", WORD(data + 
0x07));
                        printf("\tSupported SRAM Types:");
                        dmi_cache_types(WORD(data + 0x0B), "\n\t\t");
                        printf("\n");
@@ -4184,73 +4233,71 @@ static void dmi_decode(const struct dmi_
                        dmi_cache_types(WORD(data + 0x0D), " ");
                        printf("\n");
                        if (h->length < 0x13) break;
-                       printf("\tSpeed:");
-                       dmi_memory_module_speed(data[0x0F]);
-                       printf("\n");
-                       printf("\tError Correction Type: %s\n",
+                       dmi_memory_module_speed("Speed", data[0x0F]);
+                       pr_attr("Error Correction Type", "%s",
                                dmi_cache_ec_type(data[0x10]));
-                       printf("\tSystem Type: %s\n",
+                       pr_attr("System Type", "%s",
                                dmi_cache_type(data[0x11]));
-                       printf("\tAssociativity: %s\n",
+                       pr_attr("Associativity", "%s",
                                dmi_cache_associativity(data[0x12]));
                        break;
 
                case 8: /* 7.9 Port Connector Information */
                        pr_handle_name("Port Connector Information");
                        if (h->length < 0x09) break;
-                       printf("\tInternal Reference Designator: %s\n",
+                       pr_attr("Internal Reference Designator", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tInternal Connector Type: %s\n",
+                       pr_attr("Internal Connector Type", "%s",
                                dmi_port_connector_type(data[0x05]));
-                       printf("\tExternal Reference Designator: %s\n",
+                       pr_attr("External Reference Designator", "%s",
                                dmi_string(h, data[0x06]));
-                       printf("\tExternal Connector Type: %s\n",
+                       pr_attr("External Connector Type", "%s",
                                dmi_port_connector_type(data[0x07]));
-                       printf("\tPort Type: %s\n",
+                       pr_attr("Port Type", "%s",
                                dmi_port_type(data[0x08]));
                        break;
 
                case 9: /* 7.10 System Slots */
                        pr_handle_name("System Slot Information");
                        if (h->length < 0x0C) break;
-                       printf("\tDesignation: %s\n",
+                       pr_attr("Designation", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tType: %s%s\n",
+                       pr_attr("Type", "%s%s",
                                dmi_slot_bus_width(data[0x06]),
                                dmi_slot_type(data[0x05]));
-                       printf("\tCurrent Usage: %s\n",
+                       pr_attr("Current Usage", "%s",
                                dmi_slot_current_usage(data[0x07]));
-                       printf("\tLength: %s\n",
+                       pr_attr("Length", "%s",
                                dmi_slot_length(data[0x08]));
-                       dmi_slot_id(data[0x09], data[0x0A], data[0x05], "\t");
+                       dmi_slot_id(data[0x09], data[0x0A], data[0x05]);
                        printf("\tCharacteristics:");
                        if (h->length < 0x0D)
                                dmi_slot_characteristics(data[0x0B], 0x00, 
"\t\t");
                        else
                                dmi_slot_characteristics(data[0x0B], 
data[0x0C], "\t\t");
                        if (h->length < 0x11) break;
-                       dmi_slot_segment_bus_func(WORD(data + 0x0D), 
data[0x0F], data[0x10], "\t");
+                       dmi_slot_segment_bus_func(WORD(data + 0x0D), 
data[0x0F], data[0x10]);
                        if (h->length < 0x13) break;
-                       printf("\tData Bus Width: %u\n", data[0x11]);
-                       printf("\tPeer Devices: %u\n", data[0x12]);
+                       pr_attr("Data Bus Width", "%u", data[0x11]);
+                       pr_attr("Peer Devices", "%u", data[0x12]);
                        if (h->length - 0x13 >= data[0x12] * 5)
-                               dmi_slot_peers(data[0x12], data + 0x13, "\t");
+                               dmi_slot_peers(data[0x12], data + 0x13);
                        break;
 
                case 10: /* 7.11 On Board Devices Information */
-                       dmi_on_board_devices(h, "");
+                       dmi_on_board_devices(h);
                        break;
 
                case 11: /* 7.12 OEM Strings */
                        pr_handle_name("OEM Strings");
                        if (h->length < 0x05) break;
-                       dmi_oem_strings(h, "\t");
+                       dmi_oem_strings(h);
                        break;
 
                case 12: /* 7.13 System Configuration Options */
                        pr_handle_name("System Configuration Options");
                        if (h->length < 0x05) break;
-                       dmi_system_configuration_options(h, "\t");
+                       dmi_system_configuration_options(h);
                        break;
 
                case 13: /* 7.14 BIOS Language Information */
@@ -4258,19 +4305,19 @@ static void dmi_decode(const struct dmi_
                        if (h->length < 0x16) break;
                        if (ver >= 0x0201)
                        {
-                               printf("\tLanguage Description Format: %s\n",
+                               pr_attr("Language Description Format", "%s",
                                        dmi_bios_language_format(data[0x05]));
                        }
                        printf("\tInstallable Languages: %u\n", data[0x04]);
                        dmi_bios_languages(h, "\t\t");
-                       printf("\tCurrently Installed Language: %s\n",
+                       pr_attr("Currently Installed Language", "%s",
                                dmi_string(h, data[0x15]));
                        break;
 
                case 14: /* 7.15 Group Associations */
                        pr_handle_name("Group Associations");
                        if (h->length < 0x05) break;
-                       printf("\tName: %s\n",
+                       pr_attr("Name", "%s",
                                dmi_string(h, data[0x04]));
                        printf("\tItems: %u\n",
                                (h->length - 0x05) / 3);
@@ -4280,51 +4327,47 @@ static void dmi_decode(const struct dmi_
                case 15: /* 7.16 System Event Log */
                        pr_handle_name("System Event Log");
                        if (h->length < 0x14) break;
-                       printf("\tArea Length: %u bytes\n",
+                       pr_attr("Area Length", "%u bytes",
                                WORD(data + 0x04));
-                       printf("\tHeader Start Offset: 0x%04X\n",
+                       pr_attr("Header Start Offset", "0x%04X",
                                WORD(data + 0x06));
                        if (WORD(data + 0x08) - WORD(data + 0x06))
-                               printf("\tHeader Length: %u byte%s\n",
+                               pr_attr("Header Length", "%u byte%s",
                                        WORD(data + 0x08) - WORD(data + 0x06),
                                        WORD(data + 0x08) - WORD(data + 0x06) > 
1 ? "s" : "");
-                       printf("\tData Start Offset: 0x%04X\n",
+                       pr_attr("Data Start Offset", "0x%04X",
                                WORD(data + 0x08));
-                       printf("\tAccess Method: %s\n",
+                       pr_attr("Access Method", "%s",
                                dmi_event_log_method(data[0x0A]));
-                       printf("\tAccess Address:");
                        dmi_event_log_address(data[0x0A], data + 0x10);
-                       printf("\n");
-                       printf("\tStatus:");
                        dmi_event_log_status(data[0x0B]);
-                       printf("\n");
-                       printf("\tChange Token: 0x%08X\n",
+                       pr_attr("Change Token", "0x%08X",
                                DWORD(data + 0x0C));
                        if (h->length < 0x17) break;
-                       printf("\tHeader Format: %s\n",
+                       pr_attr("Header Format", "%s",
                                dmi_event_log_header_type(data[0x14]));
-                       printf("\tSupported Log Type Descriptors: %u\n",
+                       pr_attr("Supported Log Type Descriptors", "%u",
                                data[0x15]);
                        if (h->length < 0x17 + data[0x15] * data[0x16]) break;
-                       dmi_event_log_descriptors(data[0x15], data[0x16], data 
+ 0x17, "\t");
+                       dmi_event_log_descriptors(data[0x15], data[0x16], data 
+ 0x17);
                        break;
 
                case 16: /* 7.17 Physical Memory Array */
                        pr_handle_name("Physical Memory Array");
                        if (h->length < 0x0F) break;
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_memory_array_location(data[0x04]));
-                       printf("\tUse: %s\n",
+                       pr_attr("Use", "%s",
                                dmi_memory_array_use(data[0x05]));
-                       printf("\tError Correction Type: %s\n",
+                       pr_attr("Error Correction Type", "%s",
                                dmi_memory_array_ec_type(data[0x06]));
-                       printf("\tMaximum Capacity:");
                        if (DWORD(data + 0x07) == 0x80000000)
                        {
                                if (h->length < 0x17)
-                                       printf(" Unknown");
+                                       pr_attr("Maximum Capacity", "Unknown");
                                else
-                                       dmi_print_memory_size(QWORD(data + 
0x0F), 0);
+                                       dmi_print_memory_size("Maximum 
Capacity",
+                                                             QWORD(data + 
0x0F), 0);
                        }
                        else
                        {
@@ -4332,16 +4375,12 @@ static void dmi_decode(const struct dmi_
 
                                capacity.h = 0;
                                capacity.l = DWORD(data + 0x07);
-                               dmi_print_memory_size(capacity, 1);
+                               dmi_print_memory_size("Maximum Capacity",
+                                                     capacity, 1);
                        }
-                       printf("\n");
                        if (!(opt.flags & FLAG_QUIET))
-                       {
-                               printf("\tError Information Handle:");
                                dmi_memory_array_error_handle(WORD(data + 
0x0B));
-                               printf("\n");
-                       }
-                       printf("\tNumber Of Devices: %u\n",
+                       pr_attr("Number Of Devices", "%u",
                                WORD(data + 0x0D));
                        break;
 
@@ -4350,132 +4389,91 @@ static void dmi_decode(const struct dmi_
                        if (h->length < 0x15) break;
                        if (!(opt.flags & FLAG_QUIET))
                        {
-                               printf("\tArray Handle: 0x%04X\n",
+                               pr_attr("Array Handle", "0x%04X",
                                        WORD(data + 0x04));
-                               printf("\tError Information Handle:");
                                dmi_memory_array_error_handle(WORD(data + 
0x06));
-                               printf("\n");
                        }
-                       printf("\tTotal Width:");
-                       dmi_memory_device_width(WORD(data + 0x08));
-                       printf("\n");
-                       printf("\tData Width:");
-                       dmi_memory_device_width(WORD(data + 0x0A));
-                       printf("\n");
-                       printf("\tSize:");
+                       dmi_memory_device_width("Total Width", WORD(data + 
0x08));
+                       dmi_memory_device_width("Data Width", WORD(data + 
0x0A));
                        if (h->length >= 0x20 && WORD(data + 0x0C) == 0x7FFF)
                                dmi_memory_device_extended_size(DWORD(data + 
0x1C));
                        else
                                dmi_memory_device_size(WORD(data + 0x0C));
-                       printf("\n");
-                       printf("\tForm Factor: %s\n",
+                       pr_attr("Form Factor", "%s",
                                dmi_memory_device_form_factor(data[0x0E]));
-                       printf("\tSet:");
                        dmi_memory_device_set(data[0x0F]);
-                       printf("\n");
-                       printf("\tLocator: %s\n",
+                       pr_attr("Locator", "%s",
                                dmi_string(h, data[0x10]));
-                       printf("\tBank Locator: %s\n",
+                       pr_attr("Bank Locator", "%s",
                                dmi_string(h, data[0x11]));
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_memory_device_type(data[0x12]));
-                       printf("\tType Detail:");
                        dmi_memory_device_type_detail(WORD(data + 0x13));
-                       printf("\n");
                        if (h->length < 0x17) break;
-                       printf("\tSpeed:");
-                       dmi_memory_device_speed(WORD(data + 0x15));
-                       printf("\n");
+                       dmi_memory_device_speed("Speed", WORD(data + 0x15));
                        if (h->length < 0x1B) break;
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x17]));
-                       printf("\tSerial Number: %s\n",
+                       pr_attr("Serial Number", "%s",
                                dmi_string(h, data[0x18]));
-                       printf("\tAsset Tag: %s\n",
+                       pr_attr("Asset Tag", "%s",
                                dmi_string(h, data[0x19]));
-                       printf("\tPart Number: %s\n",
+                       pr_attr("Part Number", "%s",
                                dmi_string(h, data[0x1A]));
                        if (h->length < 0x1C) break;
-                       printf("\tRank: ");
                        if ((data[0x1B] & 0x0F) == 0)
-                               printf("Unknown");
+                               pr_attr("Rank", "Unknown");
                        else
-                               printf("%u", data[0x1B] & 0x0F);
-                       printf("\n");
+                               pr_attr("Rank", "%u", data[0x1B] & 0x0F);
                        if (h->length < 0x22) break;
-                       printf("\tConfigured Memory Speed:");
-                       dmi_memory_device_speed(WORD(data + 0x20));
-                       printf("\n");
+                       dmi_memory_device_speed("Configured Memory Speed",
+                                               WORD(data + 0x20));
                        if (h->length < 0x28) break;
-                       printf("\tMinimum Voltage:");
-                       dmi_memory_voltage_value(WORD(data + 0x22));
-                       printf("\n");
-                       printf("\tMaximum Voltage:");
-                       dmi_memory_voltage_value(WORD(data + 0x24));
-                       printf("\n");
-                       printf("\tConfigured Voltage:");
-                       dmi_memory_voltage_value(WORD(data + 0x26));
-                       printf("\n");
+                       dmi_memory_voltage_value("Minimum Voltage",
+                                                WORD(data + 0x22));
+                       dmi_memory_voltage_value("Maximum Voltage",
+                                                WORD(data + 0x24));
+                       dmi_memory_voltage_value("Configured Voltage",
+                                                WORD(data + 0x26));
                        if (h->length < 0x34) break;
-                       printf("\tMemory Technology:");
                        dmi_memory_technology(data[0x28]);
-                       printf("\n");
-                       printf("\tMemory Operating Mode Capability:");
                        dmi_memory_operating_mode_capability(WORD(data + 0x29));
-                       printf("\n");
-                       printf("\tFirmware Version: %s\n",
+                       pr_attr("Firmware Version", "%s",
                                dmi_string(h, data[0x2B]));
-                       printf("\tModule Manufacturer ID:");
-                       dmi_memory_manufacturer_id(WORD(data + 0x2C));
-                       printf("\n");
-                       printf("\tModule Product ID:");
-                       dmi_memory_product_id(WORD(data + 0x2E));
-                       printf("\n");
-                       printf("\tMemory Subsystem Controller Manufacturer 
ID:");
-                       dmi_memory_manufacturer_id(WORD(data + 0x30));
-                       printf("\n");
-                       printf("\tMemory Subsystem Controller Product ID:");
-                       dmi_memory_product_id(WORD(data + 0x32));
-                       printf("\n");
+                       dmi_memory_manufacturer_id("Module Manufacturer ID",
+                                                  WORD(data + 0x2C));
+                       dmi_memory_product_id("Module Product ID",
+                                             WORD(data + 0x2E));
+                       dmi_memory_manufacturer_id("Memory Subsystem Controller 
Manufacturer ID",
+                                                  WORD(data + 0x30));
+                       dmi_memory_product_id("Memory Subsystem Controller 
Product ID",
+                                             WORD(data + 0x32));
                        if (h->length < 0x3C) break;
-                       printf("\tNon-Volatile Size:");
-                       dmi_memory_size(QWORD(data + 0x34));
-                       printf("\n");
+                       dmi_memory_size("Non-Volatile Size", QWORD(data + 
0x34));
                        if (h->length < 0x44) break;
-                       printf("\tVolatile Size:");
-                       dmi_memory_size(QWORD(data + 0x3C));
-                       printf("\n");
+                       dmi_memory_size("Volatile Size", QWORD(data + 0x3C));
                        if (h->length < 0x4C) break;
-                       printf("\tCache Size:");
-                       dmi_memory_size(QWORD(data + 0x44));
-                       printf("\n");
+                       dmi_memory_size("Cache Size", QWORD(data + 0x44));
                        if (h->length < 0x54) break;
-                       printf("\tLogical Size:");
-                       dmi_memory_size(QWORD(data + 0x4C));
-                       printf("\n");
+                       dmi_memory_size("Logical Size", QWORD(data + 0x4C));
                        break;
 
                case 18: /* 7.19 32-bit Memory Error Information */
                        pr_handle_name("32-bit Memory Error Information");
                        if (h->length < 0x17) break;
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_memory_error_type(data[0x04]));
-                       printf("\tGranularity: %s\n",
+                       pr_attr("Granularity", "%s",
                                dmi_memory_error_granularity(data[0x05]));
-                       printf("\tOperation: %s\n",
+                       pr_attr("Operation", "%s",
                                dmi_memory_error_operation(data[0x06]));
-                       printf("\tVendor Syndrome:");
                        dmi_memory_error_syndrome(DWORD(data + 0x07));
-                       printf("\n");
-                       printf("\tMemory Array Address:");
-                       dmi_32bit_memory_error_address(DWORD(data + 0x0B));
-                       printf("\n");
-                       printf("\tDevice Address:");
-                       dmi_32bit_memory_error_address(DWORD(data + 0x0F));
-                       printf("\n");
-                       printf("\tResolution:");
-                       dmi_32bit_memory_error_address(DWORD(data + 0x13));
-                       printf("\n");
+                       dmi_32bit_memory_error_address("Memory Array Address",
+                                                      DWORD(data + 0x0B));
+                       dmi_32bit_memory_error_address("Device Address",
+                                                      DWORD(data + 0x0F));
+                       dmi_32bit_memory_error_address("Resolution",
+                                                      DWORD(data + 0x13));
                        break;
 
                case 19: /* 7.20 Memory Array Mapped Address */
@@ -4488,29 +4486,26 @@ static void dmi_decode(const struct dmi_
                                start = QWORD(data + 0x0F);
                                end = QWORD(data + 0x17);
 
-                               printf("\tStarting Address: 0x%08X%08Xk\n",
+                               pr_attr("Starting Address", "0x%08X%08Xk",
                                        start.h, start.l);
-                               printf("\tEnding Address: 0x%08X%08Xk\n",
+                               pr_attr("Ending Address", "0x%08X%08Xk",
                                        end.h, end.l);
-                               printf("\tRange Size:");
                                dmi_mapped_address_extended_size(start, end);
                        }
                        else
                        {
-                               printf("\tStarting Address: 0x%08X%03X\n",
+                               pr_attr("Starting Address", "0x%08X%03X",
                                        DWORD(data + 0x04) >> 2,
                                        (DWORD(data + 0x04) & 0x3) << 10);
-                               printf("\tEnding Address: 0x%08X%03X\n",
+                               pr_attr("Ending Address", "0x%08X%03X",
                                        DWORD(data + 0x08) >> 2,
                                        ((DWORD(data + 0x08) & 0x3) << 10) + 
0x3FF);
-                               printf("\tRange Size:");
                                dmi_mapped_address_size(DWORD(data + 0x08) - 
DWORD(data + 0x04) + 1);
                        }
-                       printf("\n");
                        if (!(opt.flags & FLAG_QUIET))
-                               printf("\tPhysical Array Handle: 0x%04X\n",
+                               pr_attr("Physical Array Handle", "0x%04X",
                                        WORD(data + 0x0C));
-                       printf("\tPartition Width: %u\n",
+                       pr_attr("Partition Width", "%u",
                                data[0x0E]);
                        break;
 
@@ -4524,344 +4519,288 @@ static void dmi_decode(const struct dmi_
                                start = QWORD(data + 0x13);
                                end = QWORD(data + 0x1B);
 
-                               printf("\tStarting Address: 0x%08X%08Xk\n",
+                               pr_attr("Starting Address", "0x%08X%08Xk",
                                        start.h, start.l);
-                               printf("\tEnding Address: 0x%08X%08Xk\n",
+                               pr_attr("Ending Address", "0x%08X%08Xk",
                                        end.h, end.l);
-                               printf("\tRange Size:");
                                dmi_mapped_address_extended_size(start, end);
                        }
                        else
                        {
-                               printf("\tStarting Address: 0x%08X%03X\n",
+                               pr_attr("Starting Address", "0x%08X%03X",
                                        DWORD(data + 0x04) >> 2,
                                        (DWORD(data + 0x04) & 0x3) << 10);
-                               printf("\tEnding Address: 0x%08X%03X\n",
+                               pr_attr("Ending Address", "0x%08X%03X",
                                        DWORD(data + 0x08) >> 2,
                                        ((DWORD(data + 0x08) & 0x3) << 10) + 
0x3FF);
-                               printf("\tRange Size:");
                                dmi_mapped_address_size(DWORD(data + 0x08) - 
DWORD(data + 0x04) + 1);
                        }
-                       printf("\n");
                        if (!(opt.flags & FLAG_QUIET))
                        {
-                               printf("\tPhysical Device Handle: 0x%04X\n",
+                               pr_attr("Physical Device Handle", "0x%04X",
                                        WORD(data + 0x0C));
-                               printf("\tMemory Array Mapped Address Handle: 
0x%04X\n",
+                               pr_attr("Memory Array Mapped Address Handle", 
"0x%04X",
                                        WORD(data + 0x0E));
                        }
-                       printf("\tPartition Row Position:");
                        dmi_mapped_address_row_position(data[0x10]);
-                       printf("\n");
-                       dmi_mapped_address_interleave_position(data[0x11], 
"\t");
-                       dmi_mapped_address_interleaved_data_depth(data[0x12], 
"\t");
+                       dmi_mapped_address_interleave_position(data[0x11]);
+                       dmi_mapped_address_interleaved_data_depth(data[0x12]);
                        break;
 
                case 21: /* 7.22 Built-in Pointing Device */
                        pr_handle_name("Built-in Pointing Device");
                        if (h->length < 0x07) break;
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_pointing_device_type(data[0x04]));
-                       printf("\tInterface: %s\n",
+                       pr_attr("Interface", "%s",
                                dmi_pointing_device_interface(data[0x05]));
-                       printf("\tButtons: %u\n",
+                       pr_attr("Buttons", "%u",
                                data[0x06]);
                        break;
 
                case 22: /* 7.23 Portable Battery */
                        pr_handle_name("Portable Battery");
                        if (h->length < 0x10) break;
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x05]));
                        if (data[0x06] || h->length < 0x1A)
-                               printf("\tManufacture Date: %s\n",
+                               pr_attr("Manufacture Date", "%s",
                                        dmi_string(h, data[0x06]));
                        if (data[0x07] || h->length < 0x1A)
-                               printf("\tSerial Number: %s\n",
+                               pr_attr("Serial Number", "%s",
                                        dmi_string(h, data[0x07]));
-                       printf("\tName: %s\n",
+                       pr_attr("Name", "%s",
                                dmi_string(h, data[0x08]));
                        if (data[0x09] != 0x02 || h->length < 0x1A)
-                               printf("\tChemistry: %s\n",
+                               pr_attr("Chemistry", "%s",
                                        dmi_battery_chemistry(data[0x09]));
-                       printf("\tDesign Capacity:");
                        if (h->length < 0x16)
                                dmi_battery_capacity(WORD(data + 0x0A), 1);
                        else
                                dmi_battery_capacity(WORD(data + 0x0A), 
data[0x15]);
-                       printf("\n");
-                       printf("\tDesign Voltage:");
                        dmi_battery_voltage(WORD(data + 0x0C));
-                       printf("\n");
-                       printf("\tSBDS Version: %s\n",
+                       pr_attr("SBDS Version", "%s",
                                dmi_string(h, data[0x0E]));
-                       printf("\tMaximum Error:");
                        dmi_battery_maximum_error(data[0x0F]);
-                       printf("\n");
                        if (h->length < 0x1A) break;
                        if (data[0x07] == 0)
-                               printf("\tSBDS Serial Number: %04X\n",
+                               pr_attr("SBDS Serial Number", "%04X",
                                        WORD(data + 0x10));
                        if (data[0x06] == 0)
-                               printf("\tSBDS Manufacture Date: 
%u-%02u-%02u\n",
+                               pr_attr("SBDS Manufacture Date", "%u-%02u-%02u",
                                        1980 + (WORD(data + 0x12) >> 9),
                                        (WORD(data + 0x12) >> 5) & 0x0F,
                                        WORD(data + 0x12) & 0x1F);
                        if (data[0x09] == 0x02)
-                               printf("\tSBDS Chemistry: %s\n",
+                               pr_attr("SBDS Chemistry", "%s",
                                        dmi_string(h, data[0x14]));
-                       printf("\tOEM-specific Information: 0x%08X\n",
+                       pr_attr("OEM-specific Information", "0x%08X",
                                DWORD(data + 0x16));
                        break;
 
                case 23: /* 7.24 System Reset */
                        pr_handle_name("System Reset");
                        if (h->length < 0x0D) break;
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                data[0x04] & (1 << 0) ? "Enabled" : "Disabled");
-                       printf("\tWatchdog Timer: %s\n",
+                       pr_attr("Watchdog Timer", "%s",
                                data[0x04] & (1 << 5) ? "Present" : "Not 
Present");
                        if (!(data[0x04] & (1 << 5)))
                                break;
-                       printf("\tBoot Option: %s\n",
+                       pr_attr("Boot Option", "%s",
                                dmi_system_reset_boot_option((data[0x04] >> 1) 
& 0x3));
-                       printf("\tBoot Option On Limit: %s\n",
+                       pr_attr("Boot Option On Limit", "%s",
                                dmi_system_reset_boot_option((data[0x04] >> 3) 
& 0x3));
-                       printf("\tReset Count:");
-                       dmi_system_reset_count(WORD(data + 0x05));
-                       printf("\n");
-                       printf("\tReset Limit:");
-                       dmi_system_reset_count(WORD(data + 0x07));
-                       printf("\n");
-                       printf("\tTimer Interval:");
-                       dmi_system_reset_timer(WORD(data + 0x09));
-                       printf("\n");
-                       printf("\tTimeout:");
-                       dmi_system_reset_timer(WORD(data + 0x0B));
-                       printf("\n");
+                       dmi_system_reset_count("Reset Count", WORD(data + 
0x05));
+                       dmi_system_reset_count("Reset Limit", WORD(data + 
0x07));
+                       dmi_system_reset_timer("Timer Interval", WORD(data + 
0x09));
+                       dmi_system_reset_timer("Timeout", WORD(data + 0x0B));
                        break;
 
                case 24: /* 7.25 Hardware Security */
                        pr_handle_name("Hardware Security");
                        if (h->length < 0x05) break;
-                       printf("\tPower-On Password Status: %s\n",
+                       pr_attr("Power-On Password Status", "%s",
                                dmi_hardware_security_status(data[0x04] >> 6));
-                       printf("\tKeyboard Password Status: %s\n",
+                       pr_attr("Keyboard Password Status", "%s",
                                dmi_hardware_security_status((data[0x04] >> 4) 
& 0x3));
-                       printf("\tAdministrator Password Status: %s\n",
+                       pr_attr("Administrator Password Status", "%s",
                                dmi_hardware_security_status((data[0x04] >> 2) 
& 0x3));
-                       printf("\tFront Panel Reset Status: %s\n",
+                       pr_attr("Front Panel Reset Status", "%s",
                                dmi_hardware_security_status(data[0x04] & 0x3));
                        break;
 
                case 25: /* 7.26 System Power Controls */
                        pr_handle_name("System Power Controls");
                        if (h->length < 0x09) break;
-                       printf("\tNext Scheduled Power-on:");
                        dmi_power_controls_power_on(data + 0x04);
-                       printf("\n");
                        break;
 
                case 26: /* 7.27 Voltage Probe */
                        pr_handle_name("Voltage Probe");
                        if (h->length < 0x14) break;
-                       printf("\tDescription: %s\n",
+                       pr_attr("Description", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_voltage_probe_location(data[0x05] & 0x1f));
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                dmi_probe_status(data[0x05] >> 5));
-                       printf("\tMaximum Value:");
-                       dmi_voltage_probe_value(WORD(data + 0x06));
-                       printf("\n");
-                       printf("\tMinimum Value:");
-                       dmi_voltage_probe_value(WORD(data + 0x08));
-                       printf("\n");
-                       printf("\tResolution:");
+                       dmi_voltage_probe_value("Maximum Value", WORD(data + 
0x06));
+                       dmi_voltage_probe_value("Minimum Value", WORD(data + 
0x08));
                        dmi_voltage_probe_resolution(WORD(data + 0x0A));
-                       printf("\n");
-                       printf("\tTolerance:");
-                       dmi_voltage_probe_value(WORD(data + 0x0C));
-                       printf("\n");
-                       printf("\tAccuracy:");
+                       dmi_voltage_probe_value("Tolerance", WORD(data + 0x0C));
                        dmi_probe_accuracy(WORD(data + 0x0E));
-                       printf("\n");
-                       printf("\tOEM-specific Information: 0x%08X\n",
+                       pr_attr("OEM-specific Information", "0x%08X",
                                DWORD(data + 0x10));
                        if (h->length < 0x16) break;
-                       printf("\tNominal Value:");
-                       dmi_voltage_probe_value(WORD(data + 0x14));
-                       printf("\n");
+                       dmi_voltage_probe_value("Nominal Value", WORD(data + 
0x14));
                        break;
 
                case 27: /* 7.28 Cooling Device */
                        pr_handle_name("Cooling Device");
                        if (h->length < 0x0C) break;
                        if (!(opt.flags & FLAG_QUIET) && WORD(data + 0x04) != 
0xFFFF)
-                               printf("\tTemperature Probe Handle: 0x%04X\n",
+                               pr_attr("Temperature Probe Handle", "0x%04X",
                                        WORD(data + 0x04));
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_cooling_device_type(data[0x06] & 0x1f));
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                dmi_probe_status(data[0x06] >> 5));
                        if (data[0x07] != 0x00)
-                               printf("\tCooling Unit Group: %u\n",
+                               pr_attr("Cooling Unit Group", "%u",
                                        data[0x07]);
-                       printf("\tOEM-specific Information: 0x%08X\n",
+                       pr_attr("OEM-specific Information", "0x%08X",
                                DWORD(data + 0x08));
                        if (h->length < 0x0E) break;
-                       printf("\tNominal Speed:");
                        dmi_cooling_device_speed(WORD(data + 0x0C));
-                       printf("\n");
                        if (h->length < 0x0F) break;
-                       printf("\tDescription: %s\n", dmi_string(h, 
data[0x0E]));
+                       pr_attr("Description", "%s", dmi_string(h, data[0x0E]));
                        break;
 
                case 28: /* 7.29 Temperature Probe */
                        pr_handle_name("Temperature Probe");
                        if (h->length < 0x14) break;
-                       printf("\tDescription: %s\n",
+                       pr_attr("Description", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_temperature_probe_location(data[0x05] & 
0x1F));
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                dmi_probe_status(data[0x05] >> 5));
-                       printf("\tMaximum Value:");
-                       dmi_temperature_probe_value(WORD(data + 0x06));
-                       printf("\n");
-                       printf("\tMinimum Value:");
-                       dmi_temperature_probe_value(WORD(data + 0x08));
-                       printf("\n");
-                       printf("\tResolution:");
+                       dmi_temperature_probe_value("Maximum Value",
+                                                   WORD(data + 0x06));
+                       dmi_temperature_probe_value("Minimum Value",
+                                                   WORD(data + 0x08));
                        dmi_temperature_probe_resolution(WORD(data + 0x0A));
-                       printf("\n");
-                       printf("\tTolerance:");
-                       dmi_temperature_probe_value(WORD(data + 0x0C));
-                       printf("\n");
-                       printf("\tAccuracy:");
+                       dmi_temperature_probe_value("Tolerance",
+                                                   WORD(data + 0x0C));
                        dmi_probe_accuracy(WORD(data + 0x0E));
-                       printf("\n");
-                       printf("\tOEM-specific Information: 0x%08X\n",
+                       pr_attr("OEM-specific Information", "0x%08X",
                                DWORD(data + 0x10));
                        if (h->length < 0x16) break;
-                       printf("\tNominal Value:");
-                       dmi_temperature_probe_value(WORD(data + 0x14));
-                       printf("\n");
+                       dmi_temperature_probe_value("Nominal Value",
+                                                   WORD(data + 0x14));
                        break;
 
                case 29: /* 7.30 Electrical Current Probe */
                        pr_handle_name("Electrical Current Probe");
                        if (h->length < 0x14) break;
-                       printf("\tDescription: %s\n",
+                       pr_attr("Description", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_voltage_probe_location(data[5] & 0x1F));
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                dmi_probe_status(data[0x05] >> 5));
-                       printf("\tMaximum Value:");
-                       dmi_current_probe_value(WORD(data + 0x06));
-                       printf("\n");
-                       printf("\tMinimum Value:");
-                       dmi_current_probe_value(WORD(data + 0x08));
-                       printf("\n");
-                       printf("\tResolution:");
+                       dmi_current_probe_value("Maximum Value",
+                                               WORD(data + 0x06));
+                       dmi_current_probe_value("Minimum Value",
+                                               WORD(data + 0x08));
                        dmi_current_probe_resolution(WORD(data + 0x0A));
-                       printf("\n");
-                       printf("\tTolerance:");
-                       dmi_current_probe_value(WORD(data + 0x0C));
-                       printf("\n");
-                       printf("\tAccuracy:");
+                       dmi_current_probe_value("Tolerance",
+                                               WORD(data + 0x0C));
                        dmi_probe_accuracy(WORD(data + 0x0E));
-                       printf("\n");
-                       printf("\tOEM-specific Information: 0x%08X\n",
+                       pr_attr("OEM-specific Information", "0x%08X",
                                DWORD(data + 0x10));
                        if (h->length < 0x16) break;
-                       printf("\tNominal Value:");
-                       dmi_current_probe_value(WORD(data + 0x14));
-                       printf("\n");
+                       dmi_current_probe_value("Nominal Value",
+                                               WORD(data + 0x14));
                        break;
 
                case 30: /* 7.31 Out-of-band Remote Access */
                        pr_handle_name("Out-of-band Remote Access");
                        if (h->length < 0x06) break;
-                       printf("\tManufacturer Name: %s\n",
+                       pr_attr("Manufacturer Name", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tInbound Connection: %s\n",
+                       pr_attr("Inbound Connection", "%s",
                                data[0x05] & (1 << 0) ? "Enabled" : "Disabled");
-                       printf("\tOutbound Connection: %s\n",
+                       pr_attr("Outbound Connection", "%s",
                                data[0x05] & (1 << 1) ? "Enabled" : "Disabled");
                        break;
 
                case 31: /* 7.32 Boot Integrity Services Entry Point */
                        pr_handle_name("Boot Integrity Services Entry Point");
                        if (h->length < 0x1C) break;
-                       printf("\tChecksum: %s\n",
+                       pr_attr("Checksum", "%s",
                                checksum(data, h->length) ? "OK" : "Invalid");
-                       printf("\t16-bit Entry Point Address: %04X:%04X\n",
+                       pr_attr("16-bit Entry Point Address", "%04X:%04X",
                                DWORD(data + 0x08) >> 16,
                                DWORD(data + 0x08) & 0xFFFF);
-                       printf("\t32-bit Entry Point Address: 0x%08X\n",
+                       pr_attr("32-bit Entry Point Address", "0x%08X",
                                DWORD(data + 0x0C));
                        break;
 
                case 32: /* 7.33 System Boot Information */
                        pr_handle_name("System Boot Information");
                        if (h->length < 0x0B) break;
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                dmi_system_boot_status(data[0x0A]));
                        break;
 
                case 33: /* 7.34 64-bit Memory Error Information */
                        pr_handle_name("64-bit Memory Error Information");
                        if (h->length < 0x1F) break;
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_memory_error_type(data[0x04]));
-                       printf("\tGranularity: %s\n",
+                       pr_attr("Granularity", "%s",
                                dmi_memory_error_granularity(data[0x05]));
-                       printf("\tOperation: %s\n",
+                       pr_attr("Operation", "%s",
                                dmi_memory_error_operation(data[0x06]));
-                       printf("\tVendor Syndrome:");
                        dmi_memory_error_syndrome(DWORD(data + 0x07));
-                       printf("\n");
-                       printf("\tMemory Array Address:");
-                       dmi_64bit_memory_error_address(QWORD(data + 0x0B));
-                       printf("\n");
-                       printf("\tDevice Address:");
-                       dmi_64bit_memory_error_address(QWORD(data + 0x13));
-                       printf("\n");
-                       printf("\tResolution:");
-                       dmi_32bit_memory_error_address(DWORD(data + 0x1B));
-                       printf("\n");
+                       dmi_64bit_memory_error_address("Memory Array Address",
+                                                      QWORD(data + 0x0B));
+                       dmi_64bit_memory_error_address("Device Address",
+                                                      QWORD(data + 0x13));
+                       dmi_32bit_memory_error_address("Resolution",
+                                                      DWORD(data + 0x1B));
                        break;
 
                case 34: /* 7.35 Management Device */
                        pr_handle_name("Management Device");
                        if (h->length < 0x0B) break;
-                       printf("\tDescription: %s\n",
+                       pr_attr("Description", "%s",
                                dmi_string(h, data[0x04]));
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_management_device_type(data[0x05]));
-                       printf("\tAddress: 0x%08X\n",
+                       pr_attr("Address", "0x%08X",
                                DWORD(data + 0x06));
-                       printf("\tAddress Type: %s\n",
+                       pr_attr("Address Type", "%s",
                                dmi_management_device_address_type(data[0x0A]));
                        break;
 
                case 35: /* 7.36 Management Device Component */
                        pr_handle_name("Management Device Component");
                        if (h->length < 0x0B) break;
-                       printf("\tDescription: %s\n",
+                       pr_attr("Description", "%s",
                                dmi_string(h, data[0x04]));
                        if (!(opt.flags & FLAG_QUIET))
                        {
-                               printf("\tManagement Device Handle: 0x%04X\n",
+                               pr_attr("Management Device Handle", "0x%04X",
                                        WORD(data + 0x05));
-                               printf("\tComponent Handle: 0x%04X\n",
+                               pr_attr("Component Handle", "0x%04X",
                                        WORD(data + 0x07));
                                if (WORD(data + 0x09) != 0xFFFF)
-                                       printf("\tThreshold Handle: 0x%04X\n",
+                                       pr_attr("Threshold Handle", "0x%04X",
                                                WORD(data + 0x09));
                        }
                        break;
@@ -4870,36 +4809,36 @@ static void dmi_decode(const struct dmi_
                        pr_handle_name("Management Device Threshold Data");
                        if (h->length < 0x10) break;
                        if (WORD(data + 0x04) != 0x8000)
-                               printf("\tLower Non-critical Threshold: %d\n",
+                               pr_attr("Lower Non-critical Threshold", "%d",
                                        (i16)WORD(data + 0x04));
                        if (WORD(data + 0x06) != 0x8000)
-                               printf("\tUpper Non-critical Threshold: %d\n",
+                               pr_attr("Upper Non-critical Threshold", "%d",
                                        (i16)WORD(data + 0x06));
                        if (WORD(data + 0x08) != 0x8000)
-                               printf("\tLower Critical Threshold: %d\n",
+                               pr_attr("Lower Critical Threshold", "%d",
                                        (i16)WORD(data + 0x08));
                        if (WORD(data + 0x0A) != 0x8000)
-                               printf("\tUpper Critical Threshold: %d\n",
+                               pr_attr("Upper Critical Threshold", "%d",
                                        (i16)WORD(data + 0x0A));
                        if (WORD(data + 0x0C) != 0x8000)
-                               printf("\tLower Non-recoverable Threshold: 
%d\n",
+                               pr_attr("Lower Non-recoverable Threshold", "%d",
                                        (i16)WORD(data + 0x0C));
                        if (WORD(data + 0x0E) != 0x8000)
-                               printf("\tUpper Non-recoverable Threshold: 
%d\n",
+                               pr_attr("Upper Non-recoverable Threshold", "%d",
                                        (i16)WORD(data + 0x0E));
                        break;
 
                case 37: /* 7.38 Memory Channel */
                        pr_handle_name("Memory Channel");
                        if (h->length < 0x07) break;
-                       printf("\tType: %s\n",
+                       pr_attr("Type", "%s",
                                dmi_memory_channel_type(data[0x04]));
-                       printf("\tMaximal Load: %u\n",
+                       pr_attr("Maximal Load", "%u",
                                data[0x05]);
-                       printf("\tDevices: %u\n",
+                       pr_attr("Devices", "%u",
                                data[0x06]);
                        if (h->length < 0x07 + 3 * data[0x06]) break;
-                       dmi_memory_channel_devices(data[0x06], data + 0x07, 
"\t");
+                       dmi_memory_channel_devices(data[0x06], data + 0x07);
                        break;
 
                case 38: /* 7.39 IPMI Device Information */
@@ -4909,37 +4848,35 @@ static void dmi_decode(const struct dmi_
                         */
                        pr_handle_name("IPMI Device Information");
                        if (h->length < 0x10) break;
-                       printf("\tInterface Type: %s\n",
+                       pr_attr("Interface Type", "%s",
                                dmi_ipmi_interface_type(data[0x04]));
-                       printf("\tSpecification Version: %u.%u\n",
+                       pr_attr("Specification Version", "%u.%u",
                                data[0x05] >> 4, data[0x05] & 0x0F);
-                       printf("\tI2C Slave Address: 0x%02x\n",
+                       pr_attr("I2C Slave Address", "0x%02x",
                                data[0x06] >> 1);
                        if (data[0x07] != 0xFF)
-                               printf("\tNV Storage Device Address: %u\n",
+                               pr_attr("NV Storage Device Address", "%u",
                                        data[0x07]);
                        else
-                               printf("\tNV Storage Device: Not Present\n");
-                       printf("\tBase Address: ");
+                               pr_attr("NV Storage Device", "Not Present");
                        dmi_ipmi_base_address(data[0x04], data + 0x08,
                                h->length < 0x11 ? 0 : (data[0x10] >> 4) & 1);
-                       printf("\n");
                        if (h->length < 0x12) break;
                        if (data[0x04] != 0x04)
                        {
-                               printf("\tRegister Spacing: %s\n",
+                               pr_attr("Register Spacing", "%s",
                                        dmi_ipmi_register_spacing(data[0x10] >> 
6));
                                if (data[0x10] & (1 << 3))
                                {
-                                       printf("\tInterrupt Polarity: %s\n",
+                                       pr_attr("Interrupt Polarity", "%s",
                                                data[0x10] & (1 << 1) ? "Active 
High" : "Active Low");
-                                       printf("\tInterrupt Trigger Mode: %s\n",
+                                       pr_attr("Interrupt Trigger Mode", "%s",
                                                data[0x10] & (1 << 0) ? "Level" 
: "Edge");
                                }
                        }
                        if (data[0x11] != 0x00)
                        {
-                               printf("\tInterrupt Number: %u\n",
+                               pr_attr("Interrupt Number", "%u",
                                        data[0x11]);
                        }
                        break;
@@ -4948,51 +4885,47 @@ static void dmi_decode(const struct dmi_
                        pr_handle_name("System Power Supply");
                        if (h->length < 0x10) break;
                        if (data[0x04] != 0x00)
-                               printf("\tPower Unit Group: %u\n",
+                               pr_attr("Power Unit Group", "%u",
                                        data[0x04]);
-                       printf("\tLocation: %s\n",
+                       pr_attr("Location", "%s",
                                dmi_string(h, data[0x05]));
-                       printf("\tName: %s\n",
+                       pr_attr("Name", "%s",
                                dmi_string(h, data[0x06]));
-                       printf("\tManufacturer: %s\n",
+                       pr_attr("Manufacturer", "%s",
                                dmi_string(h, data[0x07]));
-                       printf("\tSerial Number: %s\n",
+                       pr_attr("Serial Number", "%s",
                                dmi_string(h, data[0x08]));
-                       printf("\tAsset Tag: %s\n",
+                       pr_attr("Asset Tag", "%s",
                                dmi_string(h, data[0x09]));
-                       printf("\tModel Part Number: %s\n",
+                       pr_attr("Model Part Number", "%s",
                                dmi_string(h, data[0x0A]));
-                       printf("\tRevision: %s\n",
+                       pr_attr("Revision", "%s",
                                dmi_string(h, data[0x0B]));
-                       printf("\tMax Power Capacity:");
                        dmi_power_supply_power(WORD(data + 0x0C));
-                       printf("\n");
-                       printf("\tStatus:");
                        if (WORD(data + 0x0E) & (1 << 1))
-                               printf(" Present, %s",
+                               pr_attr("Status", "Present, %s",
                                        dmi_power_supply_status((WORD(data + 
0x0E) >> 7) & 0x07));
                        else
-                               printf(" Not Present");
-                       printf("\n");
-                       printf("\tType: %s\n",
+                               pr_attr("Status", "Not Present");
+                       pr_attr("Type", "%s",
                                dmi_power_supply_type((WORD(data + 0x0E) >> 10) 
& 0x0F));
-                       printf("\tInput Voltage Range Switching: %s\n",
+                       pr_attr("Input Voltage Range Switching", "%s",
                                dmi_power_supply_range_switching((WORD(data + 
0x0E) >> 3) & 0x0F));
-                       printf("\tPlugged: %s\n",
+                       pr_attr("Plugged", "%s",
                                WORD(data + 0x0E) & (1 << 2) ? "No" : "Yes");
-                       printf("\tHot Replaceable: %s\n",
+                       pr_attr("Hot Replaceable", "%s",
                                WORD(data + 0x0E) & (1 << 0) ? "Yes" : "No");
                        if (h->length < 0x16) break;
                        if (!(opt.flags & FLAG_QUIET))
                        {
                                if (WORD(data + 0x10) != 0xFFFF)
-                                       printf("\tInput Voltage Probe Handle: 
0x%04X\n",
+                                       pr_attr("Input Voltage Probe Handle", 
"0x%04X",
                                                WORD(data + 0x10));
                                if (WORD(data + 0x12) != 0xFFFF)
-                                       printf("\tCooling Device Handle: 
0x%04X\n",
+                                       pr_attr("Cooling Device Handle", 
"0x%04X",
                                                WORD(data + 0x12));
                                if (WORD(data + 0x14) != 0xFFFF)
-                                       printf("\tInput Current Probe Handle: 
0x%04X\n",
+                                       pr_attr("Input Current Probe Handle", 
"0x%04X",
                                                WORD(data + 0x14));
                        }
                        break;
@@ -5001,19 +4934,19 @@ static void dmi_decode(const struct dmi_
                        if (h->length < 0x0B) break;
                        if (opt.flags & FLAG_QUIET)
                                return;
-                       dmi_additional_info(h, "");
+                       dmi_additional_info(h);
                        break;
 
                case 41: /* 7.42 Onboard Device Extended Information */
                        pr_handle_name("Onboard Device");
                        if (h->length < 0x0B) break;
-                       printf("\tReference Designation: %s\n", dmi_string(h, 
data[0x04]));
-                       printf("\tType: %s\n",
+                       pr_attr("Reference Designation", "%s", dmi_string(h, 
data[0x04]));
+                       pr_attr("Type", "%s",
                                dmi_on_board_devices_type(data[0x05] & 0x7F));
-                       printf("\tStatus: %s\n",
+                       pr_attr("Status", "%s",
                                data[0x05] & 0x80 ? "Enabled" : "Disabled");
-                       printf("\tType Instance: %u\n", data[0x06]);
-                       dmi_slot_segment_bus_func(WORD(data + 0x07), 
data[0x09], data[0x0A], "\t");
+                       pr_attr("Type Instance", "%u", data[0x06]);
+                       dmi_slot_segment_bus_func(WORD(data + 0x07), 
data[0x09], data[0x0A]);
                        break;
 
                case 42: /* 7.43 Management Controller Host Interface */
@@ -5021,7 +4954,7 @@ static void dmi_decode(const struct dmi_
                        if (ver < 0x0302)
                        {
                                if (h->length < 0x05) break;
-                               printf("\tInterface Type: %s\n",
+                               pr_attr("Interface Type", "%s",
                                        
dmi_management_controller_host_type(data[0x04]));
                                /*
                                 * There you have a type-dependent, 
variable-length
@@ -5032,7 +4965,7 @@ static void dmi_decode(const struct dmi_
                                if (h->length < 0x09) break;
                                if (data[0x04] == 0xF0)         /* OEM */
                                {
-                                       printf("\tVendor ID: 
0x%02X%02X%02X%02X\n",
+                                       pr_attr("Vendor ID", 
"0x%02X%02X%02X%02X",
                                                data[0x05], data[0x06], 
data[0x07],
                                                data[0x08]);
                                }
@@ -5044,10 +4977,8 @@ static void dmi_decode(const struct dmi_
                case 43: /* 7.44 TPM Device */
                        pr_handle_name("TPM Device");
                        if (h->length < 0x1B) break;
-                       printf("\tVendor ID:");
                        dmi_tpm_vendor_id(data + 0x04);
-                       printf("\n");
-                       printf("\tSpecification Version: %d.%d\n", data[0x08], 
data[0x09]);
+                       pr_attr("Specification Version", "%d.%d", data[0x08], 
data[0x09]);
                        switch (data[0x08])
                        {
                                case 0x01:
@@ -5056,11 +4987,11 @@ static void dmi_decode(const struct dmi_
                                         * redundant with the above, and uncoded
                                         * in a silly way.
                                         */
-                                       printf("\tFirmware Revision: %u.%u\n",
+                                       pr_attr("Firmware Revision", "%u.%u",
                                                data[0x0C], data[0x0D]);
                                        break;
                                case 0x02:
-                                       printf("\tFirmware Revision: %u.%u\n",
+                                       pr_attr("Firmware Revision", "%u.%u",
                                                DWORD(data + 0x0A) >> 16,
                                                DWORD(data + 0x0A) & 0xFFFF);
                                        /*
@@ -5070,11 +5001,11 @@ static void dmi_decode(const struct dmi_
                                         */
                                        break;
                        }
-                       printf("\tDescription: %s\n", dmi_string(h, 
data[0x12]));
+                       pr_attr("Description", "%s", dmi_string(h, data[0x12]));
                        printf("\tCharacteristics:\n");
                        dmi_tpm_characteristics(QWORD(data + 0x13), "\t\t");
                        if (h->length < 0x1F) break;
-                       printf("\tOEM-specific Information: 0x%08X\n",
+                       pr_attr("OEM-specific Information", "0x%08X",
                                DWORD(data + 0x1B));
                        break;
 
@@ -5141,8 +5072,7 @@ static void dmi_table_string(const struc
                                printf("%u.%u\n", data[key - 1], data[key]);
                        break;
                case 0x108:
-                       dmi_system_uuid(data + offset, ver);
-                       printf("\n");
+                       dmi_system_uuid(NULL, data + offset, ver);
                        break;
                case 0x305:
                        printf("%s\n", dmi_chassis_type(data[offset]));
@@ -5151,8 +5081,7 @@ static void dmi_table_string(const struc
                        printf("%s\n", dmi_processor_family(h, ver));
                        break;
                case 0x416:
-                       dmi_processor_frequency(data + offset);
-                       printf("\n");
+                       dmi_processor_frequency(NULL, data + offset);
                        break;
                default:
                        printf("%s\n", dmi_string(h, data[offset]));
--- dmidecode.orig/dmioutput.c  2020-03-24 15:00:19.058165425 +0100
+++ dmidecode/dmioutput.c       2020-03-24 15:00:33.122336695 +0100
@@ -59,3 +59,15 @@ void pr_handle_name(const char *format,
        va_end(args);
        printf("\n");
 }
+
+void pr_attr(const char *name, const char *format, ...)
+{
+       va_list args;
+
+       printf("\t%s: ", name);
+
+       va_start(args, format);
+       vprintf(format, args);
+       va_end(args);
+       printf("\n");
+}
--- dmidecode.orig/dmioutput.h  2020-03-24 15:00:19.058165425 +0100
+++ dmidecode/dmioutput.h       2020-03-24 15:00:33.122336695 +0100
@@ -25,3 +25,4 @@ void pr_comment(const char *format, ...)
 void pr_info(const char *format, ...);
 void pr_handle(const struct dmi_header *h);
 void pr_handle_name(const char *format, ...);
+void pr_attr(const char *name, const char *format, ...);
--- dmidecode.orig/dmioem.c     2020-03-24 15:00:19.058165425 +0100
+++ dmidecode/dmioem.c  2020-03-24 15:00:33.122336695 +0100
@@ -96,16 +96,16 @@ static int dmi_decode_acer(const struct
                        pr_handle_name("Acer Hotkey Function");
                        if (h->length < 0x0F) break;
                        cap = WORD(data + 0x04);
-                       printf("\tFunction bitmap for Communication Button: 
0x%04hx\n", cap);
+                       pr_attr("Function bitmap for Communication Button", 
"0x%04hx", cap);
                        printf("\t\tWiFi: %s\n", cap & 0x0001 ? "Yes" : "No");
                        printf("\t\t3G: %s\n", cap & 0x0040 ? "Yes" : "No");
                        printf("\t\tWiMAX: %s\n", cap & 0x0080 ? "Yes" : "No");
                        printf("\t\tBluetooth: %s\n", cap & 0x0800 ? "Yes" : 
"No");
-                       printf("\tFunction bitmap for Application Button: 
0x%04hx\n", WORD(data + 0x06));
-                       printf("\tFunction bitmap for Media Button: 0x%04hx\n", 
WORD(data + 0x08));
-                       printf("\tFunction bitmap for Display Button: 
0x%04hx\n", WORD(data + 0x0A));
-                       printf("\tFunction bitmap for Others Button: 
0x%04hx\n", WORD(data + 0x0C));
-                       printf("\tCommunication Function Key Number: %d\n", 
data[0x0E]);
+                       pr_attr("Function bitmap for Application Button", 
"0x%04hx", WORD(data + 0x06));
+                       pr_attr("Function bitmap for Media Button", "0x%04hx", 
WORD(data + 0x08));
+                       pr_attr("Function bitmap for Display Button", 
"0x%04hx", WORD(data + 0x0A));
+                       pr_attr("Function bitmap for Others Button", "0x%04hx", 
WORD(data + 0x0C));
+                       pr_attr("Communication Function Key Number", "%d", 
data[0x0E]);
                        break;
 
                default:
@@ -128,19 +128,21 @@ static void dmi_print_hp_net_iface_rec(u
         * 640K ought to be enough for anybody(said no one, ever).
         * */
        static u8 nic_ctr;
+       char attr[8];
 
        if (id == 0xFF)
                id = ++nic_ctr;
 
+       sprintf(attr, "NIC %hu", id);
        if (dev == 0x00 && bus == 0x00)
-               printf("\tNIC %d: Disabled\n", id);
+               pr_attr(attr, "Disabled");
        else if (dev == 0xFF && bus == 0xFF)
-               printf("\tNIC %d: Not Installed\n", id);
+               pr_attr(attr, "Not Installed");
        else
        {
-               printf("\tNIC %d: PCI device %02x:%02x.%x, "
-                       "MAC address %02X:%02X:%02X:%02X:%02X:%02X\n",
-                       id, bus, dev >> 3, dev & 7,
+               pr_attr(attr, "PCI device %02x:%02x.%x, "
+                       "MAC address %02X:%02X:%02X:%02X:%02X:%02X",
+                       bus, dev >> 3, dev & 7,
                        mac[0], mac[1], mac[2], mac[3], mac[4], mac[5]);
        }
 }
@@ -160,13 +162,13 @@ static int dmi_decode_hp(const struct dm
                         */
                        pr_handle_name("%s ProLiant System/Rack Locator", 
company);
                        if (h->length < 0x0B) break;
-                       printf("\tRack Name: %s\n", dmi_string(h, data[0x04]));
-                       printf("\tEnclosure Name: %s\n", dmi_string(h, 
data[0x05]));
-                       printf("\tEnclosure Model: %s\n", dmi_string(h, 
data[0x06]));
-                       printf("\tEnclosure Serial: %s\n", dmi_string(h, 
data[0x0A]));
-                       printf("\tEnclosure Bays: %d\n", data[0x08]);
-                       printf("\tServer Bay: %s\n", dmi_string(h, data[0x07]));
-                       printf("\tBays Filled: %d\n", data[0x09]);
+                       pr_attr("Rack Name", "%s", dmi_string(h, data[0x04]));
+                       pr_attr("Enclosure Name", "%s", dmi_string(h, 
data[0x05]));
+                       pr_attr("Enclosure Model", "%s", dmi_string(h, 
data[0x06]));
+                       pr_attr("Enclosure Serial", "%s", dmi_string(h, 
data[0x0A]));
+                       pr_attr("Enclosure Bays", "%d", data[0x08]);
+                       pr_attr("Server Bay", "%s", dmi_string(h, data[0x07]));
+                       pr_attr("Bays Filled", "%d", data[0x09]);
                        break;
 
                case 209:
@@ -243,20 +245,22 @@ static int dmi_decode_hp(const struct dm
                         */
                        pr_handle_name("%s 64-bit CRU Information", company);
                        if (h->length < 0x18) break;
-                       printf("\tSignature: 0x%08x", DWORD(data + 0x04));
                        if (is_printable(data + 0x04, 4))
-                               printf(" (%c%c%c%c)", data[0x04], data[0x05],
+                               pr_attr("Signature", "0x%08x (%c%c%c%c)",
+                                       DWORD(data + 0x04),
+                                       data[0x04], data[0x05],
                                        data[0x06], data[0x07]);
-                       printf("\n");
+                       else
+                               pr_attr("Signature", "0x%08x", DWORD(data + 
0x04));
                        if (DWORD(data + 0x04) == 0x55524324)
                        {
                                u64 paddr = QWORD(data + 0x08);
                                paddr.l += DWORD(data + 0x14);
                                if (paddr.l < DWORD(data + 0x14))
                                        paddr.h++;
-                               printf("\tPhysical Address: 0x%08x%08x\n",
+                               pr_attr("Physical Address", "0x%08x%08x",
                                        paddr.h, paddr.l);
-                               printf("\tLength: 0x%08x\n", DWORD(data + 
0x10));
+                               pr_attr("Length", "0x%08x", DWORD(data + 0x10));
                        }
                        break;
 
@@ -268,12 +272,12 @@ static int dmi_decode_hp(const struct dm
                         */
                        pr_handle_name("%s ProLiant Information", company);
                        if (h->length < 0x08) break;
-                       printf("\tPower Features: 0x%08x\n", DWORD(data + 
0x04));
+                       pr_attr("Power Features", "0x%08x", DWORD(data + 0x04));
                        if (h->length < 0x0C) break;
-                       printf("\tOmega Features: 0x%08x\n", DWORD(data + 
0x08));
+                       pr_attr("Omega Features", "0x%08x", DWORD(data + 0x08));
                        if (h->length < 0x14) break;
                        feat = DWORD(data + 0x10);
-                       printf("\tMisc. Features: 0x%08x\n", feat);
+                       pr_attr("Misc. Features", "0x%08x", feat);
                        printf("\t\tiCRU: %s\n", feat & 0x0001 ? "Yes" : "No");
                        printf("\t\tUEFI: %s\n", feat & 0x1400 ? "Yes" : "No");
                        break;
@@ -320,8 +324,8 @@ static int dmi_decode_ibm_lenovo(const s
                                return 0;
 
                        pr_handle_name("ThinkVantage Technologies");
-                       printf("\tVersion: %u\n", data[0x04]);
-                       printf("\tDiagnostics: %s\n",
+                       pr_attr("Version", "%u", data[0x04]);
+                       pr_attr("Diagnostics", "%s",
                                data[0x14] & 0x80 ? "Available" : "No");
                        break;
 
@@ -359,7 +363,7 @@ static int dmi_decode_ibm_lenovo(const s
                                return 0;
 
                        pr_handle_name("ThinkPad Device Presence Detection");
-                       printf("\tFingerprint Reader: %s\n",
+                       pr_attr("Fingerprint Reader", "%s",
                                data[0x09] & 0x01 ? "Present" : "No");
                        break;
 
@@ -392,8 +396,8 @@ static int dmi_decode_ibm_lenovo(const s
                                return 0;
 
                        pr_handle_name("ThinkPad Embedded Controller Program");
-                       printf("\tVersion ID: %s\n", dmi_string(h, 1));
-                       printf("\tRelease Date: %s\n", dmi_string(h, 2));
+                       pr_attr("Version ID", "%s", dmi_string(h, 1));
+                       pr_attr("Release Date", "%s", dmi_string(h, 2));
                        break;
 
                default:

-- 
Jean Delvare
SUSE L3 Support



reply via email to

[Prev in Thread] Current Thread [Next in Thread]