[coreboot-gerrit] Change in coreboot[master]: util/ifdtool: Add Cooper Lake platform support under IFDv2

[Arthur Heymans (Code Review)]

Attention is currently required from: Johnny Lin, Subrata Banik.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55745 )

Change subject: util/ifdtool: Add Cooper Lake platform support under IFDv2
......................................................................

Patch Set 3:

(3 comments)

Patchset:

PS3:
> yes +2 for patchset 1 and if you can split the patch into 2 CL ?

Why? Some things would be wrong for CPX in patchset 1 if things are split up.

File util/ifdtool/ifdtool.c:

https://review.coreboot.org/c/coreboot/+/55745/comment/25dd7aba_39a86564 
PS3, Line 796:  %s Region Write Access:            %s\n",
             : 		       region_names[8].pretty,
Not needed.

https://review.coreboot.org/c/coreboot/+/55745/comment/5f8d0389_006d64d2 
PS3, Line 812:  %s Region Read Access:             %s\n",
             : 		       region_names[8].pretty,
also not needed