diff options
| author | Grzegorz Bernacki <bernacki@google.com> | 2023-04-25 15:30:14 +0000 |
|---|---|---|
| committer | Martin L Roth <gaumless@gmail.com> | 2023-06-02 20:24:25 +0000 |
| commit | d34dbe5888fb11f2a6bb9bd0e4624d2e2f9f350a (patch) | |
| tree | 0344259bb6b5016d95acbdb61721278cd29b2391 /src/soc/amd/picasso | |
| parent | 9d6008ea5b5acf142f73f3d485c2c196d8a88a59 (diff) | |
soc/amd/common/block/cpu: Refactor ucode allocation
Move microcode load/unload to pre_mp_init and post_mp_init callbacks.
It allows to make sure that ucode is freed only if all APs updated
microcode.
BUG=b:278264488
TEST=Build and run with additional debug prints added
to confirm that data are correctly unmapped
Change-Id: I200d24df6157cc6d06bade34809faefea9f0090a
Signed-off-by: Grzegorz Bernacki <bernacki@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74777
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Diffstat (limited to 'src/soc/amd/picasso')
| -rw-r--r-- | src/soc/amd/picasso/cpu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/amd/picasso/cpu.c b/src/soc/amd/picasso/cpu.c index c5fc8a8bf9..c0b918e5c0 100644 --- a/src/soc/amd/picasso/cpu.c +++ b/src/soc/amd/picasso/cpu.c @@ -40,7 +40,7 @@ static void model_17_init(struct device *dev) check_mca(); set_cstate_io_addr(); - amd_update_microcode_from_cbfs(); + amd_apply_microcode_patch(); } static struct device_operations cpu_dev_ops = { |