[coreboot-gerrit] [M] Change in coreboot[master]: Revert "util/ifdtool: Add option to create FMAP template"

[Maximilian Brune (Code Review)]

Attention is currently required from: Felix Singer, Martin L Roth, Patrick Rudolph, Subrata Banik, Matt DeVillier, Stefan Reinauer, Kapil Porwal, Angel Pons, Lean Sheng Tan.

Maximilian Brune has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74692 )

Change subject: Revert "util/ifdtool: Add option to create FMAP template"
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74692/comment/9ef515a4_203c25c7 
PS2, Line 11: Above commit inserts multiple IFD regions not present into the default
            : FMAP template, some of which overlap, resulting in far less space
            : available for BIOS region / CBFS.
> it's a 16MB chip, having one region at 0x07fff000, let alone 5, is nonsensical.

Then either the IFD is wrong or we parse the IFD regions incorrectly for skylake.