[coreboot-gerrit] Change in coreboot[master]: drivers/ipmi: Implement "IPMI add SEL" function

[Morgan Jang (Code Review)]

Morgan Jang has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/40286 )

Change subject: drivers/ipmi: Implement "IPMI add SEL" function
......................................................................

drivers/ipmi: Implement "IPMI add SEL" function

Implemented for functions need to add SEL, the information of SEL can be clear when adding SEL.

TEST=Check if SEL is add in BMC

Change-Id: I38f3acb958d12c196d33d34fd5cfa0b784f403b7
Signed-off-by: Morgan Jang 
---
M src/drivers/ipmi/ipmi_kcs.h
M src/drivers/ipmi/ipmi_ops.c
M src/drivers/ipmi/ipmi_ops.h
3 files changed, 82 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/40286/1

diff --git a/src/drivers/ipmi/ipmi_kcs.h b/src/drivers/ipmi/ipmi_kcs.h
index d511edf..44f668d 100644
--- a/src/drivers/ipmi/ipmi_kcs.h
+++ b/src/drivers/ipmi/ipmi_kcs.h
@@ -21,6 +21,7 @@
 #define IPMI_NETFN_FIRMWARE 0x08
 #define IPMI_NETFN_STORAGE 0x0a
 #define   IPMI_READ_FRU_DATA 0x11
+#define   IPMI_ADD_SEL_ENTRY 0x44
 #define IPMI_NETFN_TRANSPORT 0x0c

#define IPMI_CMD_ACPI_POWERON 0x06
diff --git a/src/drivers/ipmi/ipmi_ops.c b/src/drivers/ipmi/ipmi_ops.c
index 25bf077..6d6e562 100644
--- a/src/drivers/ipmi/ipmi_ops.c
+++ b/src/drivers/ipmi/ipmi_ops.c
@@ -118,3 +118,44 @@
    memcpy(uuid, rsp.data, 16);
    return CB_SUCCESS;
 }
+
+enum cb_err ipmi_add_sel(const int port, struct sel_event_record *sel)
+{
+	int ret;
+	struct ipmi_add_sel_rsp rsp;
+	unsigned char data[16];
+
+	if (sel == NULL) {
+		printk(BIOS_ERR, "%s failed, null pointer parameter\n",
+			 __func__);
+		return CB_ERR;
+	}
+
+	// system event record
+	if (sel->record_type == 0x2) {
+		memcpy(data, sel, 16);
+	}
+
+	// OEM timestamped
+	if (sel->record_type >= 0xC0 && sel->record_type <= 0xDF) {
+		memcpy(data, sel, 3);
+		memcpy(data + 3, &sel->sel_type.oem_ts_type, 13);
+	}
+
+	// OEM non-timestamped
+	if (sel->record_type >= 0xE0) {
+		memcpy(data, sel, 3);
+		memcpy(data + 3, &sel->sel_type.oem_nots_type, 13);
+	}
+
+	ret = ipmi_kcs_message(port, IPMI_NETFN_STORAGE, 0x0,
+			IPMI_ADD_SEL_ENTRY, data,
+			16, (unsigned char *) &rsp, sizeof(rsp));
+
+	if (ret < sizeof(struct ipmi_rsp) || rsp.resp.completion_code) {
+		printk(BIOS_ERR, "IPMI: %s command failed (ret=%d resp=0x%x)\n",
+			 __func__, ret, rsp.resp.completion_code);
+		return CB_ERR;
+	}
+	return CB_SUCCESS;
+}
diff --git a/src/drivers/ipmi/ipmi_ops.h b/src/drivers/ipmi/ipmi_ops.h
index f889595..5774c58 100644
--- a/src/drivers/ipmi/ipmi_ops.h
+++ b/src/drivers/ipmi/ipmi_ops.h
@@ -50,6 +50,42 @@
    uint8_t data[CONFIG_IPMI_FRU_SINGLE_RW_SZ];
 } __packed;

+struct standard_spec_sel_rec{
+	uint32_t timestamp;
+	uint16_t gen_id;
+	uint8_t	evm_rev;
+	uint8_t	sensor_type;
+	uint8_t	sensor_num;
+	uint8_t	event_dir_type;
+	uint8_t	event_data[3];
+};
+
+struct oem_ts_spec_sel_rec{
+	uint32_t timestamp;
+	uint8_t manf_id[3];
+	uint8_t	oem_defined[6];
+};
+
+struct oem_nots_spec_sel_rec{
+	uint8_t oem_defined[13];
+};
+
+/* SEL Event Record */
+struct sel_event_record {
+	uint16_t record_id;
+	uint8_t	record_type;
+	union{
+		struct standard_spec_sel_rec standard_type;
+		struct oem_ts_spec_sel_rec oem_ts_type;
+		struct oem_nots_spec_sel_rec oem_nots_type;
+	} sel_type;
+} __packed;
+
+struct ipmi_add_sel_rsp {
+	struct ipmi_rsp resp;
+	uint16_t record_id;
+} __packed;
+
 /* Platform Management FRU Information Storage Definition Spec. */
 #define PRODUCT_MAN_TYPE_LEN_OFFSET 3
 #define BOARD_MAN_TYPE_LEN_OFFSET 6
@@ -123,4 +159,8 @@
 /* Read a particular FRU inventory area into fru_info_str. */
 void read_fru_one_area(const int port, uint8_t id, uint16_t offset,
    	struct fru_info_str *fru_info_str, enum fru_area fru_area);
+
+/* Add a SEL record entry, returns CB_SUCCESS on success and CB_ERR
+ * if an error occurred */
+enum cb_err ipmi_add_sel(const int port, struct sel_event_record *sel);
 #endif