[coreboot-gerrit] Change in coreboot[master]: AGESA: Fix discovery of IO APICs number

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34589 )

Change subject: AGESA: Fix discovery of IO APICs number
......................................................................

Patch Set 5: Code-Review-2

(1 comment)

Not urgent for POSTCAR_STAGE=y fix, postponed.

https://review.coreboot.org/c/coreboot/+/34589/5/src/drivers/amd/agesa/state... 
File src/drivers/amd/agesa/state_machine.c:

https://review.coreboot.org/c/coreboot/+/34589/5/src/drivers/amd/agesa/state... 
PS5, Line 305: 	amd_create_struct(&aip, AMD_INIT_EARLY, NULL, 0);
We don't link AMD_INIT_EARLY to ramstage at all in vendorcode.
vendorcode/amd/agesa/common/agesa-entry-cfg.h

It is there in the PI blob though, but we will not open up the can of worms of having multiple callsites with AMD_CREATE_STRUCT and same *params.