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

[Nico Huber (Code Review)]

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

Nico Huber 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/002ef4cd_8064c06f 
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.
> never mind. Can you try the following patch: […]

It's reading empty flash space:

FLREG10:    0xffffffff

etc.

The patch should work in this case, because the space is empty and all 1's.
But it doesn't solve the actual problem that ifdtool assumes 16 regions
for Skylake. Which seems to come from the v1/v2 idea (Intel never mentioned
any versioning, IIRC it was just made up years ago as a workaround).