Attention is currently required from: Paul Menzel, Stefan Reinauer.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80235?usp=email )
Change subject: util/ifdtool: Add a new switch -E to protect GPR0 ......................................................................
Patch Set 6:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/80235/comment/2bae1a07_d40819e0 : PS5, Line 7: Add new cmdline to enable GPR0 protection
Maybe: […]
Acknowledged
https://review.coreboot.org/c/coreboot/+/80235/comment/bb865e4e_31aafaa7 : PS5, Line 9: IFD Tool
I have not seen that spelling – only ifdtool.
Acknowledged
https://review.coreboot.org/c/coreboot/+/80235/comment/79269e9c_7cf74c0a : PS5, Line 11: Additionally, perform some code refactoring while adding support
It’d be great, if you added a blank line between paragraphs.
Acknowledged
https://review.coreboot.org/c/coreboot/+/80235/comment/2cd2bbb8_58165990 : PS5, Line 13:
Can you point to a document, where more information about GPR0 protection can be found?
Acknowledged