[coreboot-gerrit] Change in coreboot[master]: nb/amd/pi: support for boards without BINARYPI_LEGACY_WRAPPER

[Michał Żygowski (Code Review)]

Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32421 )

Change subject: nb/amd/pi: support for boards without BINARYPI_LEGACY_WRAPPER
......................................................................

Patch Set 2:
> Patch Set 1:
>
> AmdGetValue() was invalid in the first place, it is not part of AGESA specification and was already removed during amd/stoneyridge reviews from that PI build.
>
> That cpu_bus_scan() uses is just wrong and probably does not even do what it is supposed to.

Any suggestions on how to rewrite the cpu_bus_scan? What would be the best approach?