[coreboot-gerrit] Change in coreboot[master]: soc/intel/apollolake: Measure bootblock from IFWI

[Arthur Heymans (Code Review)]

Attention is currently required from: Nico Huber, Werner Zeh, Andrey Petrov.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64493 )

Change subject: soc/intel/apollolake: Measure bootblock from IFWI
......................................................................

Patch Set 5:

(1 comment)

File src/soc/intel/apollolake/include/soc/iomap.h:

https://review.coreboot.org/c/coreboot/+/64493/comment/fcb8f408_4cecfdd3 
PS5, Line 34: #define SHARED_SRAM_BASE		0xffff8000
            : #define SHARED_SRAM_SIZE		(32 * KiB)
> Does it make sense to extend the region device for this case? We do not access the contents in this use case. Or shall I move the defines over to bootblock_measure.c to stay consistent?

I think it makes sense. If someone wants to measure the microcode updates or print out the FIT table or elements of it... I'm not sure if that will happen but there is no harm in using the full size here.