Attention is currently required from: Arthur Heymans. Hello Arthur Heymans,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/63062
to review the following change.
Change subject: sec/intel/txt: Use variable ......................................................................
sec/intel/txt: Use variable
Use the variable intended for this use. This fixes building with clang.
Change-Id: I4ee61fb9533b90ddb1a1592d5d9945761739ddb6 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/security/intel/txt/logging.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/63062/1
diff --git a/src/security/intel/txt/logging.c b/src/security/intel/txt/logging.c index 46c594c..e3fcb5e 100644 --- a/src/security/intel/txt/logging.c +++ b/src/security/intel/txt/logging.c @@ -70,7 +70,7 @@ /* Errors by BIOS ACM or FIT */ if ((txt_error & ACMERROR_TXT_VALID) && (acm_status & ACMERROR_TXT_VALID)) { - intel_txt_log_acm_error(read32((void *)TXT_BIOSACM_ERRORCODE)); + intel_txt_log_acm_error(bios_acm_error); log_txt_error("FIT MICROCODE"); } /* Errors by SINIT */