Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32588 )
Change subject: intel/fsp1_1: Drop `boot_mode` from `pei_data` ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/32588/1/src/drivers/intel/fsp1_1/romstage.c File src/drivers/intel/fsp1_1/romstage.c:
https://review.coreboot.org/#/c/32588/1/src/drivers/intel/fsp1_1/romstage.c@... PS1, Line 144: raminit(params, s3wake); Any reason for not using params->power_state->prev_sleep_state in raminit()?