[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch: Drop use of SPD cache for puff-based variants

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47899 )

Change subject: mb/google/hatch: Drop use of SPD cache for puff-based variants
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47899/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/47899/1//COMMIT_MSG@19 
PS1, Line 19: Test: build/boot WYVERN variant, check boot times via cbmem:
            : w/SPD caching: ~722 ms
            : w/FSP reading: ~627 ms
> > Hi Matt, Do you see the different between cold reboot and warm reboot? […]

and on a Librem 14 (CML) development unit (w/2 sodimms):

with SPD cache
cold: 634,xxx (occasionally 734,xxx)
warm: 643,xxx

without SPD cache
cold: 639,xxx
warm: 637,xxx

<shrugs>
maybe I'll just continue my patch to add the SPD cache as an option for the default FMAP