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>

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)
 {

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)
 {

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);
 

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)
 {

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);