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

[Morgan Jang (Code Review)]

Morgan Jang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40286 )

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

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40286/4/src/drivers/ipmi/ipmi_ops.c 
File src/drivers/ipmi/ipmi_ops.c:

https://review.coreboot.org/c/coreboot/+/40286/4/src/drivers/ipmi/ipmi_ops.c... 
PS4, Line 126: unsigned char data[16];
> would it be possible to define type for this data? […]

Because I union three event types, just make sure to fill the correct event type, the sel does not need to be converted, I have verified this way, it worked.