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

[Andrey Petrov (Code Review)]

Andrey Petrov 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 4:

(1 comment)

thanks !

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?
this way we can avoid all the memcpy and hardcoded lengths, and just can initialize data structure in one go, i.e

stuct blah data = { .a = x, .b = y, c = z } and so on.