summaryrefslogtreecommitdiff
path: root/src/soc/amd
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2021-07-14 17:12:49 +0200
committerFelix Held <felix-coreboot@felixheld.de>2021-07-15 17:03:54 +0000
commit5ce2751d6d9d1d160e532ce3a44274c395c82217 (patch)
treeb8184540b39d6b43cf8dd999d2c41b94137283b9 /src/soc/amd
parente84c3f189876630acfc2e09310300de8048e4fc6 (diff)
soc/amd/common/block/cpu/mca: move function prototypes to local header
Since those functions are implemented and used only inside the common MCA(X) code, there's no need to have them in the header file that gets included in the SoC-specific code. Change-Id: Ia84e149d67ac7d80de595379c73a6cf08730719d Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/56309 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/amd')
-rw-r--r--src/soc/amd/common/block/cpu/mca/mca.c1
-rw-r--r--src/soc/amd/common/block/cpu/mca/mca_common.c1
-rw-r--r--src/soc/amd/common/block/cpu/mca/mca_common_defs.h2
-rw-r--r--src/soc/amd/common/block/cpu/mca/mcax.c1
-rw-r--r--src/soc/amd/common/block/include/amdblocks/mca.h2
5 files changed, 5 insertions, 2 deletions
diff --git a/src/soc/amd/common/block/cpu/mca/mca.c b/src/soc/amd/common/block/cpu/mca/mca.c
index c7d3e6f711..2d91894491 100644
--- a/src/soc/amd/common/block/cpu/mca/mca.c
+++ b/src/soc/amd/common/block/cpu/mca/mca.c
@@ -7,6 +7,7 @@
#include <cpu/x86/msr.h>
#include <console/console.h>
#include <types.h>
+#include "mca_common_defs.h"
static void mca_print_error(unsigned int bank)
{
diff --git a/src/soc/amd/common/block/cpu/mca/mca_common.c b/src/soc/amd/common/block/cpu/mca/mca_common.c
index 0a375089f2..f029966dc7 100644
--- a/src/soc/amd/common/block/cpu/mca/mca_common.c
+++ b/src/soc/amd/common/block/cpu/mca/mca_common.c
@@ -2,6 +2,7 @@
#include <amdblocks/mca.h>
#include <cpu/x86/msr.h>
+#include "mca_common_defs.h"
void check_mca(void)
{
diff --git a/src/soc/amd/common/block/cpu/mca/mca_common_defs.h b/src/soc/amd/common/block/cpu/mca/mca_common_defs.h
index 41d73d8fe2..8142371664 100644
--- a/src/soc/amd/common/block/cpu/mca/mca_common_defs.h
+++ b/src/soc/amd/common/block/cpu/mca/mca_common_defs.h
@@ -6,6 +6,8 @@
#include <amdblocks/mca.h>
#include <cper.h>
+void mca_check_all_banks(void);
+void build_bert_mca_error(struct mca_bank_status *mci);
enum cper_x86_check_type error_to_chktype(struct mca_bank_status *mci);
void fill_generic_section(cper_proc_generic_error_section_t *sec, struct mca_bank_status *mci);
diff --git a/src/soc/amd/common/block/cpu/mca/mcax.c b/src/soc/amd/common/block/cpu/mca/mcax.c
index cb01e1d6f6..fecac0a11f 100644
--- a/src/soc/amd/common/block/cpu/mca/mcax.c
+++ b/src/soc/amd/common/block/cpu/mca/mcax.c
@@ -6,6 +6,7 @@
#include <cpu/x86/msr.h>
#include <console/console.h>
#include <types.h>
+#include "mca_common_defs.h"
static void mca_print_error(unsigned int bank)
{
diff --git a/src/soc/amd/common/block/include/amdblocks/mca.h b/src/soc/amd/common/block/include/amdblocks/mca.h
index 70a83ab7f7..c84f23215a 100644
--- a/src/soc/amd/common/block/include/amdblocks/mca.h
+++ b/src/soc/amd/common/block/include/amdblocks/mca.h
@@ -12,8 +12,6 @@ struct mca_bank_status {
};
void check_mca(void);
-void mca_check_all_banks(void);
-void build_bert_mca_error(struct mca_bank_status *mci);
bool mca_has_expected_bank_count(void);
bool mca_is_valid_bank(unsigned int bank);
const char *mca_get_bank_name(unsigned int bank);