[coreboot-gerrit] New patch to review for coreboot: device: Add device path display support

Leroy P Leahy (leroy.p.leahy@intel.com) gerrit at coreboot.org
Mon Feb 15 22:19:32 CET 2016


Leroy P Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13715

-gerrit

commit 2abc45148ca02356bc16ca295e8697eeb5ea678e
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Sat Feb 13 06:10:04 2016 -0800

    device: Add device path display support
    
    Add an optional routine to translate the device path types into a string
    for display.
    
    TEST=Build and run on Galileo
    
    Change-Id: Iea5d0a2430d9a8546105324e2beda0955210dca9
    Signed-off-by: Lee Leahy <leroy.p.leahy at intel.com>
---
 src/device/device_util.c  | 12 ++++++++++++
 src/include/device/path.h | 15 +++++++++++++++
 2 files changed, 27 insertions(+)

diff --git a/src/device/device_util.c b/src/device/device_util.c
index ac18538..837dfb5 100644
--- a/src/device/device_util.c
+++ b/src/device/device_util.c
@@ -921,3 +921,15 @@ int dev_count_cpu(void)
 
 	return count;
 }
+
+/* Get device path name */
+const char *dev_path_name(enum device_path_type type)
+{
+	static const char *const type_names[] = DEVICE_PATH_NAMES;
+	const char *type_name = "Unknown";
+
+	/* Translate the type value into a string */
+	if (type < ARRAY_SIZE(type_names))
+		type_name = type_names[type];
+	return type_name;
+}
diff --git a/src/include/device/path.h b/src/include/device/path.h
index 00fb727..f61bfdd 100644
--- a/src/include/device/path.h
+++ b/src/include/device/path.h
@@ -15,6 +15,20 @@ enum device_path_type {
 	DEVICE_PATH_IOAPIC,
 };
 
+#define DEVICE_PATH_NAMES {			\
+		"DEVICE_PATH_NONE",		\
+		"DEVICE_PATH_ROOT",		\
+		"DEVICE_PATH_PCI",		\
+		"DEVICE_PATH_PNP",		\
+		"DEVICE_PATH_I2C",		\
+		"DEVICE_PATH_APIC",		\
+		"DEVICE_PATH_DOMAIN",		\
+		"DEVICE_PATH_CPU_CLUSTER",	\
+		"DEVICE_PATH_CPU",		\
+		"DEVICE_PATH_CPU_BUS",		\
+		"DEVICE_PATH_IOAPIC"		\
+}
+
 struct domain_path
 {
 	unsigned domain;
@@ -86,5 +100,6 @@ struct device_path {
 #define BUS_PATH_MAX (DEVICE_PATH_MAX+10)
 
 extern int path_eq(struct device_path *path1, struct device_path *path2);
+extern const char *dev_path_name(enum device_path_type type);
 
 #endif /* DEVICE_PATH_H */



More information about the coreboot-gerrit mailing list