Bill XIE has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39906 )
Change subject: drivers/pc80/rtc: Change the logic to use cmos.default to keep PCRs ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39906/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39906/1//COMMIT_MSG@7 PS1, Line 7: Always load cmos.default to satisfy measured boot
If this is only a requirement of measured boot, why does this change make the loading of cmos. […]
Title changed. Only the logic about how to use the file is changed. When and whether to use remains the same.
https://review.coreboot.org/c/coreboot/+/39906/1//COMMIT_MSG@10 PS1, Line 10: break
it doesn't break measured boot, just change the calculated PCRs
Done
https://review.coreboot.org/c/coreboot/+/39906/1//COMMIT_MSG@13 PS1, Line 13: unconditional
that seems to be a good solution. […]
Where had I better to write it down?