Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38711 )
Change subject: util/ifdtool: Support modification of single Flash Descriptor ......................................................................
Patch Set 2: Code-Review+1
(3 comments)
https://review.coreboot.org/c/coreboot/+/38711/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38711/2//COMMIT_MSG@9 PS2, Line 9: module drop `module`?
https://review.coreboot.org/c/coreboot/+/38711/2//COMMIT_MSG@10 PS2, Line 10: instead of raising a Segmentation Fault. Please separate paragraphs with an empty line.
https://review.coreboot.org/c/coreboot/+/38711/2/util/ifdtool/ifdtool.c File util/ifdtool/ifdtool.c:
https://review.coreboot.org/c/coreboot/+/38711/2/util/ifdtool/ifdtool.c@1342 PS2, Line 1342: module descriptor?