From 002e5e057cb8fa6177e38094a795a1cd407db873 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Mon, 20 Jul 2020 23:10:21 +0200 Subject: [PATCH] security/intel/stm: Add missing Jenkins does not build `config.stm` because the file name lacks the mainboard name. So, the code was not being build-tested, and it does not build because several files lacked the definition for `bool`. Add the missing #include directives. Renaming the config file so that Jenkins build-tests it is done in a follow-up. Change-Id: Idf012b7ace0648027ef6e901d821ca6682cee198 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/43622 Reviewed-by: Paul Menzel Reviewed-by: HAOUAS Elyes Reviewed-by: Eugene Myers Tested-by: build bot (Jenkins) --- src/security/intel/stm/SmmStm.c | 1 + src/security/intel/stm/SmmStm.h | 1 + src/security/intel/stm/StmPlatformSmm.c | 1 + 3 files changed, 3 insertions(+) diff --git a/src/security/intel/stm/SmmStm.c b/src/security/intel/stm/SmmStm.c index 87873f5a69..7256401bc9 100644 --- a/src/security/intel/stm/SmmStm.c +++ b/src/security/intel/stm/SmmStm.c @@ -6,6 +6,7 @@ #include #include #include +#include #include #define TXT_EVTYPE_BASE 0x400 diff --git a/src/security/intel/stm/SmmStm.h b/src/security/intel/stm/SmmStm.h index 0447a9edc2..b395493a06 100644 --- a/src/security/intel/stm/SmmStm.h +++ b/src/security/intel/stm/SmmStm.h @@ -4,6 +4,7 @@ #define _SMM_STM_H_ #include "StmApi.h" +#include /* * Load STM image. diff --git a/src/security/intel/stm/StmPlatformSmm.c b/src/security/intel/stm/StmPlatformSmm.c index 1d21bf7af2..9eccb32faa 100644 --- a/src/security/intel/stm/StmPlatformSmm.c +++ b/src/security/intel/stm/StmPlatformSmm.c @@ -9,6 +9,7 @@ #include #include +#include #include #include