Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32762 )
Change subject: soc/intel/broadwell: Don't use a pointer for pei_data ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/#/c/32762/1/src/mainboard/intel/wtm2/romstage.c File src/mainboard/intel/wtm2/romstage.c:
https://review.coreboot.org/#/c/32762/1/src/mainboard/intel/wtm2/romstage.c@... PS1, Line 31: mainboard_fill_spd_data(&rp->pei_data);
huh? copy-pasta
very much :p
https://review.coreboot.org/#/c/32762/1/src/soc/intel/broadwell/include/soc/... File src/soc/intel/broadwell/include/soc/romstage.h:
https://review.coreboot.org/#/c/32762/1/src/soc/intel/broadwell/include/soc/... PS1, Line 21: #include <soc/pm.h>
hmm, why?
forgot to remove that. A first version also attempted the same for chipset_power_state.