Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42277 )
Change subject: mb/ocp/deltalake: Update SMBIOS type 2 Location In Chassis from BMC ......................................................................
Patch Set 8:
(2 comments)
https://review.coreboot.org/c/coreboot/+/42277/8/src/mainboard/ocp/deltalake... File src/mainboard/ocp/deltalake/ramstage.c:
https://review.coreboot.org/c/coreboot/+/42277/8/src/mainboard/ocp/deltalake... PS8, Line 22: return ""; If the result is not CB_SUCCESS, either the BMC communication is not working or there is bug in BMC, let's print an ERROR message.
https://review.coreboot.org/c/coreboot/+/42277/8/src/mainboard/ocp/deltalake... PS8, Line 24: /* Sanity check, slot_id can only be 1~4 since there are 4 slots in YV3 */ Let's set up an enum in ipmi.h for slot1 to slot4, that would be better than hard coding here.