Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/29662 )
Change subject: soc/intel/braswell: Add C_ENVIRONMENT_BOOTBLOCK support ......................................................................
Patch Set 5:
(2 comments)
https://review.coreboot.org/#/c/29662/5/src/soc/intel/braswell/bootblock/cac... File src/soc/intel/braswell/bootblock/cache_as_ram_cbootblock.S:
https://review.coreboot.org/#/c/29662/5/src/soc/intel/braswell/bootblock/cac... PS5, Line 21: #include <device/pci_def.h>
that looks like a copy of drivers/intel/fsp1_1/cache_as_ram.inc. […]
Specific support (provide timestamp, bootblock_pre_c_entry label) for C bootblock is required.
https://review.coreboot.org/#/c/29662/2/src/soc/intel/braswell/romstage/car_... File src/soc/intel/braswell/romstage/car_stage.S:
https://review.coreboot.org/#/c/29662/2/src/soc/intel/braswell/romstage/car_... PS2, Line 23:
What I meant is 2. Replace all occurence of romstage_c_entry on braswell with car_stage_c_entry. […]
Replacing with car_stage_c_entry() results into a build error on the Intel Quark platforms which have this function in SoC and uses FSP1.1