Attention is currently required from: Felix Held, Nick Kochlowski.
Paul Menzel has posted comments on this change by Nick Kochlowski. ( https://review.coreboot.org/c/coreboot/+/84804?usp=email )
Change subject: commonlib/bsd/cbmem_id.h: Add region for SIL_CONTEXT ......................................................................
Patch Set 3:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/84804/comment/2ccfc20e_4861ce65?usp... : PS3, Line 10: global SIL_CONTEXT structure used in openSIL API : calls Please elaborate. Can you already point to some specification document?
https://review.coreboot.org/c/coreboot/+/84804/comment/da4ad712_3c56a9cb?usp... : PS3, Line 9: Add an ID to identify and reserve a region in CBMEM for : the global SIL_CONTEXT structure used in openSIL API : calls. Please re-flow for 72 characters per line. (One line less is going to be used.)
https://review.coreboot.org/c/coreboot/+/84804/comment/bc95d03e_f2cb6016?usp... : PS3, Line 12: Were you able to test it already? Please add a TEST= line.