erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

util/amdfwtool: Fix a bug clearing two bits of soft fuse value

Browse source 
The PSP soft fuse is a 64bit value which does not use address mode
bits. Those address mode bits are also part of the soft fuse value,
thus must not be hardcoded to 0.

Change-Id: I5a3e078800653d15baf1939fdce11a60031b9978
Signed-off-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/90789
Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Michał Żygowski 2026-01-18 17:53:45 +01:00 • committed by Matt DeVillier
commit 6c7f734f7b
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
util/amdfwtool/amdfwtool.c
View file

@ -1088,7 +1088,7 @@ static void integrate_psp_firmwares(context *ctx,
pspdir->entries[count].rsvd = 0;
pspdir->entries[count].size = 0xFFFFFFFF;
pspdir->entries[count].addr = fw_table[i].other;
pspdir->entries[count].address_mode = AMD_ADDR_PHYSICAL;
pspdir->entries[count].address_mode = fw_table[i].other >> 62;
count++;
} else if (fw_table[i].type == AMD_FW_PSP_NVRAM ||
fw_table[i].type == AMD_RPMC_NVRAM) {