Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33740 )
Change subject: Documentation: Update soc/intel/apollolake ......................................................................
Patch Set 3:
This is my `default.fmd`:
FLASH 16M { SI_DESC@0 0x1000 IFWI@0x1000 0xefe000 { LBP1@0 0x77f000 { IFWI_CORE@0 0x2ff000 } LBP2@0x77f000 0x77f000 { CB_OBB@0x80000 0x200000 { # <-- @0x800000 absolute FMAP@0 0x1000 RW_MRC_CACHE@0x10000 0x10000 COREBOOT(CBFS)@0x100000 0x100000 } NOT_MEMORY_MAPPED@0x73f000 0x40000 } } DEVICE_EXPANSION@0xeff000 0x100000 UNUSED_HOLE@0xfff000 0x1000 }
"IFWI_CORE" can be used as CONFIG_IFWI_FMAP_NAME. And "CB_OBB", well, I would name it COREBOOT, but that name is already taken for CBFS :-/
I didn't push it yet, because I'm still stuck on the flashrom side. But this works fine with the current coreboot build process while it doesn't need the weirdness described in this document (i.e. redefining terms like what IFWI covers). Any name that can also be found in Intel documents should match what it says there.