Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35998 )
Change subject: mb/lenovo/{x200,t400}: Add VBOOT support ......................................................................
Patch Set 15:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35998/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35998/1//COMMIT_MSG@11 PS1, Line 11: spend
Seems worth to be added to documentation […]
Ack
https://review.coreboot.org/c/coreboot/+/35998/14/src/mainboard/lenovo/t400/... File src/mainboard/lenovo/t400/vboot-rwa.fmd:
https://review.coreboot.org/c/coreboot/+/35998/14/src/mainboard/lenovo/t400/... PS14, Line 6: }
Same gaps here, intentional?
Done
https://review.coreboot.org/c/coreboot/+/35998/12/src/mainboard/lenovo/x200/... File src/mainboard/lenovo/x200/vboot-rwa.fmd:
https://review.coreboot.org/c/coreboot/+/35998/12/src/mainboard/lenovo/x200/... PS12, Line 26:
empty line?
just noise, why does Gerrit show these?