[coreboot-gerrit] [S] Change in coreboot[master]: arch/x86 & commonlib: Add macros for postcodes used in x86/tables

[Martin L Roth (Code Review)]

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69200 )

Change subject: arch/x86 & commonlib: Add macros for postcodes used in x86/tables
......................................................................

arch/x86 & commonlib: Add macros for postcodes used in x86/tables

The 0x9a, 0x9b, and 0x9c postcodes are not used anywhere else in the
coreboot tree other than in arch/x86/tables.c.  Add macros to
standardize these postcodes.

Signed-off-by: Martin Roth 
Change-Id: I16be65ffa3f0b253fe4a9bb7bfb97597a760ad3f
---
M src/arch/x86/tables.c
M src/commonlib/include/commonlib/console/post_codes.h
2 files changed, 38 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/69200/1

diff --git a/src/arch/x86/tables.c b/src/arch/x86/tables.c
index 5940c64..b4b97b4 100644
--- a/src/arch/x86/tables.c
+++ b/src/arch/x86/tables.c
@@ -17,7 +17,7 @@
    unsigned long high_table_pointer;

#define MAX_PIRQ_TABLE_SIZE (4 * 1024)
-	post_code(0x9a);
+	post_code(POST_X86_WRITE_PIRQ_TABLE);

/* This table must be between 0x0f0000 and 0x100000 */
    rom_table_end = write_pirq_routing_table(rom_table_end);
@@ -49,7 +49,7 @@
    unsigned long high_table_pointer;

#define MAX_MP_TABLE_SIZE (4 * 1024)
-	post_code(0x9b);
+	post_code(POST_X86_WRITE_MPTABLE);

/* The smp table must be in 0-1K, 639K-640K, or 960K-1M */
    rom_table_end = write_smp_table(rom_table_end);
@@ -78,7 +78,7 @@
    unsigned long high_table_pointer;
    const size_t max_acpi_size = CONFIG_MAX_ACPI_TABLE_SIZE_KB * KiB;

-	post_code(0x9c);
+	post_code(POST_X86_WRITE_ACPITABLE);

/* Write ACPI tables to F segment and high tables area */

diff --git a/src/commonlib/include/commonlib/console/post_codes.h b/src/commonlib/include/commonlib/console/post_codes.h
index 8b154ca..8ab069b 100644
--- a/src/commonlib/include/commonlib/console/post_codes.h
+++ b/src/commonlib/include/commonlib/console/post_codes.h
@@ -287,6 +287,27 @@
 #define POST_FSP_SILICON_EXIT			0x99

/**
+ * \brief Entry to write_pirq_table
+ *
+ * coreboot entered write_pirq_table
+ */
+#define POST_X86_WRITE_PIRQ_TABLE		0x9a
+
+/**
+ * \brief Entry to write_mptable
+ *
+ * coreboot entered write_mptable
+ */
+#define POST_X86_WRITE_MPTABLE			0x9b
+
+/**
+ * \brief Entry to write_acpi_table
+ *
+ * coreboot entered write_acpi_table
+ */
+#define POST_X86_WRITE_ACPITABLE		0x9c
+
+/**
  * \brief Before calling FSP Multiphase SiliconInit
  *
  * Going to call into FSP binary for Multiple phase SI Init