Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34423 )
Change subject: soc/amd/picasso: Add FSP support for including AGESA ......................................................................
Patch Set 27: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/34423/27/src/soc/amd/picasso/romsta... File src/soc/amd/picasso/romstage.c:
https://review.coreboot.org/c/coreboot/+/34423/27/src/soc/amd/picasso/romsta... PS27, Line 34: /* Disable WB from to region 4GB - TOM2 */ I'm curious why we are doing this from a policy standpoint.