Attention is currently required from: Furquan Shaikh, Ricardo Quesada. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56986 )
Change subject: elog: define RW_ELOG constant ......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56986/comment/95a747d1_77ce84a1 PS1, Line 7: elog: define RW_ELOG constant Please be more specific by adding “region name”(?) somewhere. Maybe:
Define constant for RW region name
https://review.coreboot.org/c/coreboot/+/56986/comment/e394e9bc_7b55d5ec PS1, Line 9: This CL indroduces the ELOG_RW_REGION_NAME. : This constant replaced the hardcoded "RW_ELOG" value. : This constant will be used also by elogtool (see CL in the commit : chain). Please do not break lines after each sentence. (If so, please separate paragraphs by a blank line.)