Andrey Petrov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34857 )
Change subject: mb/ocp/monolake: Add IPMI CMOS clear support ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34857/4/src/mainboard/ocp/monolake/... File src/mainboard/ocp/monolake/ipmi.h:
https://review.coreboot.org/c/coreboot/+/34857/4/src/mainboard/ocp/monolake/... PS4, Line 18: please add #include so this file could be sourced independently. I think you just need "types.h"
https://review.coreboot.org/c/coreboot/+/34857/4/src/mainboard/ocp/monolake/... File src/mainboard/ocp/monolake/ipmi.c:
https://review.coreboot.org/c/coreboot/+/34857/4/src/mainboard/ocp/monolake/... PS4, Line 32: ret = ipmi_kcs_message(BMC_KCS_BASE, IPMI_NETFN_OEM, 0x0, this is confusing. We are asking for bios boot order and use it to tell if clear request is pending. This looks like a bit of overloaded message to me. Could you perhaps add a comment and explain what is happening?