Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37329 )
Change subject: sb/amd/cimx/sb800: add C bootblock southbridge initialization ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37329/2/src/southbridge/amd/cimx/sb... File src/southbridge/amd/cimx/sb800/bootblock.c:
https://review.coreboot.org/c/coreboot/+/37329/2/src/southbridge/amd/cimx/sb... PS2, Line 129: bootblock_southbridge_init();
We need to check what sb_poweron_init() does and if it is required in bootblock already.
It does lots of things: PMIOA programming, ALINK configuration, SPI fast speed configuration, legacy_free option handling for LPC decoding, some PCI devices configuration (SATA, LPC, P2P).
Maybe not necessarily in bootblock, but most likely before AGESA.