[coreboot-gerrit] [XS] Change in coreboot[main]: commonlib/bsd/cbmem_id.h: Add region for SIL_CONTEXT

[Paul Menzel (Code Review)]

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.