[coreboot-gerrit] Change in coreboot[master]: drivers/ipmi: Implement the function for logging system events into BMC

[Andrey Petrov (Code Review)]

Andrey Petrov has submitted this change. ( https://review.coreboot.org/c/coreboot/+/40286 )

Change subject: drivers/ipmi: Implement the function for logging system events into BMC
......................................................................

drivers/ipmi: Implement the function for logging system events into BMC

Implemented for functions that need to log system events into BMC,
the information of system events can be specific.

TEST=Use ipmitool and execute "ipmitool sel list" command to check
     if SEL is added into BMC.

Change-Id: I38f3acb958d12c196d33d34fd5cfa0b784f403b7
Signed-off-by: Morgan Jang 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40286
Tested-by: build bot (Jenkins) 
Reviewed-by: Andrey Petrov 
Reviewed-by: Paul Menzel 
---
M src/drivers/ipmi/ipmi_kcs.h
M src/drivers/ipmi/ipmi_ops.c
M src/drivers/ipmi/ipmi_ops.h
3 files changed, 63 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Andrey Petrov: Looks good to me, approved

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..2a52ba0 100644
--- a/src/drivers/ipmi/ipmi_ops.c
+++ b/src/drivers/ipmi/ipmi_ops.c
@@ -118,3 +118,25 @@
    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;
+
+	if (sel == NULL) {
+		printk(BIOS_ERR, "%s failed, system evnt log is not present.\n", __func__);
+		return CB_ERR;
+	}
+
+	ret = ipmi_kcs_message(port, IPMI_NETFN_STORAGE, 0x0,
+			IPMI_ADD_SEL_ENTRY, (const unsigned char *) sel,
+			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..e6e6b77 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