Subrata; the motivation is that AMD Picasso will use some of the FSP2.x infrastructure for the UEFI abstractions/API it provides. Not optimal, but apparently the most straight-forward choice given the layout of AMD vendorcode.

For this change, the motivation is to be able to build the file such that _car_xxx symbol references will hit garbage collection. Implementation of amd/picasso runs with romstage already in DRAM.

My understanding is Picasso needs FSP-M loader but without the actual raminit call here. Marshall or Martin should comment on that further. The attempt is to reorganise code here such that it is not necessary to initialize memranges at all.

View Change

To view, visit change 35289. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifb1e33d67722db086d9f9feff0e1d289f40d4ef5
Gerrit-Change-Number: 35289
Gerrit-PatchSet: 2
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Wed, 11 Sep 2019 06:49:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment