diff options
author | Felix Held <felix-coreboot@felixheld.de> | 2024-10-22 23:06:34 +0200 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2024-10-28 14:24:32 +0000 |
commit | 555551c98856c7a43ae9ab78bba11a2d5058fd81 (patch) | |
tree | 2ce43b05d4761ab104d51201c7141d193268f8db /src/soc/amd/genoa_poc | |
parent | 33314a09466d6ca98c2fae34a5b4e593a5eb0c04 (diff) |
soc/amd/common/block/psp: fix logic bug in psp_get_* functions
The functions psp_get_ftpm_capabilties, psp_get_hsti_state, and
psp_get_psp_capabilities which were added in commit 5e7ab1a23346
("soc/amd/common/psp: add helper functions to retrieve capability bits")
have a bug in the 'cmd_status' error handling logic. In case of an
error, 'cmd_status' is non-zero, while the check somehow expected the
opposite. Fix the bug by returning an error if 'cmd_status' is non-zero.
Change-Id: Iafcd185ec4a8a4c0e463b0ac5bac3ef78a0af305
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84836
Reviewed-by: Ana Carolina Cabral
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
Diffstat (limited to 'src/soc/amd/genoa_poc')
0 files changed, 0 insertions, 0 deletions