Attention is currently required from: Reka Norman.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80235?usp=email )
Change subject: util/ifdtool: Add new cmdline to enable GPR0 protection ......................................................................
Patch Set 3:
(2 comments)
File util/ifdtool/ifdtool.c:
https://review.coreboot.org/c/coreboot/+/80235/comment/1ca2f461_5a562faf : PS3, Line 1639: for (size_t index = 1; index <= fpt->count; index++) { : struct cse_fpt_sub_part *fpt_sub_part = (struct cse_fpt_sub_part *)(fpt + index);
This is kind of a hack which only works because `struct cse_fpt` and `struct cse_fpt_sub_part` happe […]
Acknowledged
https://review.coreboot.org/c/coreboot/+/80235/comment/6284a6bc_d6c38c46 : PS3, Line 1641: &fpt_sub_part->signature[0]
nit: just `fpt_sub_part->signature` should work?
Acknowledged