diff mbox

[V3,3/3] hdata/vpd: Parse the OpenPOWER OPFR record

Message ID 148877821298.31016.7192938974179849226.stgit@thinktux.in.ibm.com
State Superseded
Headers show

Commit Message

Ananth N Mavinakayanahalli March 6, 2017, 5:30 a.m. UTC 
Parse the OpenPOWER FRU VPD (OPFR) record on OpenPOWER instead
of the VINI records.

Signed-off-by: Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com>
---
 hdata/vpd.c |   45 ++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 44 insertions(+), 1 deletion(-)

Comments

Vasant Hegde March 13, 2017, 11:56 a.m. UTC | #1 
On 03/06/2017 11:00 AM, Ananth N Mavinakayanahalli wrote:
> Parse the OpenPOWER FRU VPD (OPFR) record on OpenPOWER instead
> of the VINI records.
>
> Signed-off-by: Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com>

Reviewed-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>

-Vasant
diff mbox

Patch

diff --git a/hdata/vpd.c b/hdata/vpd.c
index 462a7f0..639fa33 100644
--- a/hdata/vpd.c
+++ b/hdata/vpd.c
@@ -226,6 +226,45 @@  static const struct card_info *card_info_lookup(char *ccin)
 	return NULL;
 }
 
+/*
+ * For OpenPOWER, we only decipher OPFR records. While OP HDAT have VINI
+ * records too, populating the fields in there is optional. Also, there
+ * is an overlap in the fields contained therein.
+ */
+static void vpd_opfr_parse(struct dt_node *node,
+		const void *fruvpd, unsigned int fruvpd_sz)
+{
+	const void *kw;
+	uint8_t sz;
+
+	/* Vendor Name */
+	kw = vpd_find(fruvpd, fruvpd_sz, "OPFR", "VN", &sz);
+	if (kw)
+		dt_add_property_nstr(node, "vendor", kw, sz);
+
+	/* FRU Description */
+	kw = vpd_find(fruvpd, fruvpd_sz, "OPFR", "DR", &sz);
+	if (kw)
+		dt_add_property_nstr(node, "description", kw, sz);
+
+	/* Part number */
+	kw = vpd_find(fruvpd, fruvpd_sz, "OPFR", "VP", &sz);
+	if (kw)
+		dt_add_property_nstr(node, "part-number", kw, sz);
+
+	/* Serial number */
+	kw = vpd_find(fruvpd, fruvpd_sz, "OPFR", "VS", &sz);
+	if (kw)
+		dt_add_property_nstr(node, "serial-number", kw, sz);
+
+	/* Build date in BCD */
+	kw = vpd_find(fruvpd, fruvpd_sz, "OPFR", "MB", &sz);
+	if (kw)
+		dt_add_property_nstr(node, "build-date", kw, sz);
+
+	return;
+}
+
 static void vpd_vini_parse(struct dt_node *node,
 			   const void *fruvpd, unsigned int fruvpd_sz)
 {
@@ -446,7 +485,11 @@  struct dt_node *dt_add_vpd_node(const struct HDIF_common_hdr *hdr,
 	if (vpd_valid(fruvpd, fruvpd_sz)
 	    && !dt_find_property(node, "ibm,vpd")) {
 		dt_add_property(node, "ibm,vpd", fruvpd, fruvpd_sz);
-		vpd_vini_parse(node, fruvpd, fruvpd_sz);
+
+		if (op_platform)
+			vpd_opfr_parse(node, fruvpd, fruvpd_sz);
+		else
+			vpd_vini_parse(node, fruvpd, fruvpd_sz);
 	}
 
 	return node;