summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorDinesh Gehlot <digehlot@google.com>2023-02-24 05:09:04 +0000
committerLean Sheng Tan <sheng.tan@9elements.com>2023-02-24 12:08:33 +0000
commit930fded5b7f50333179c71e0d1fcf1857862dfe0 (patch)
treef109fdce1255d939d70a4f095bd4f5a61fbf04c7 /src
parent9a5b743e56432ff13cecc814f1127f382779432d (diff)
soc/intel/adl: Select CSE defined ME spec version for alderlake
Alderlake based SoCs uses Intel's Management Engine (ME), version 16. This patch selects ME 16 specification defined at common code and removes alderlake SoC specific ME code and data structures. BUG=b:260309647 Test=Build verified for brya. Signed-off-by: Dinesh Gehlot <digehlot@google.com> Change-Id: Ib94e4662c735b1c31c8dfca1cfa881e6fa4070fa Reviewed-on: https://review.coreboot.org/c/coreboot/+/73244 Reviewed-by: Subrata Banik <subratabanik@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/intel/alderlake/Kconfig1
-rw-r--r--src/soc/intel/alderlake/Makefile.inc1
-rw-r--r--src/soc/intel/alderlake/include/soc/me.h81
-rw-r--r--src/soc/intel/alderlake/me.c98
4 files changed, 1 insertions, 180 deletions
diff --git a/src/soc/intel/alderlake/Kconfig b/src/soc/intel/alderlake/Kconfig
index afa2a752df..27176ec9c9 100644
--- a/src/soc/intel/alderlake/Kconfig
+++ b/src/soc/intel/alderlake/Kconfig
@@ -105,6 +105,7 @@ config CPU_SPECIFIC_OPTIONS
select SOC_INTEL_COMMON_BLOCK_HECI1_DISABLE_USING_PMC_IPC
select SOC_INTEL_COMMON_BLOCK_IPU if !SOC_INTEL_ALDERLAKE_PCH_S
select SOC_INTEL_COMMON_BLOCK_IRQ
+ select SOC_INTEL_COMMON_BLOCK_ME_SPEC_16
select SOC_INTEL_COMMON_BLOCK_MEMINIT
select SOC_INTEL_COMMON_BLOCK_PCIE_RTD3
select SOC_INTEL_COMMON_BLOCK_PMC_EPOC
diff --git a/src/soc/intel/alderlake/Makefile.inc b/src/soc/intel/alderlake/Makefile.inc
index fa52efbc6f..943b158564 100644
--- a/src/soc/intel/alderlake/Makefile.inc
+++ b/src/soc/intel/alderlake/Makefile.inc
@@ -33,7 +33,6 @@ ramstage-y += fsp_params.c
ramstage-y += graphics.c
ramstage-y += hsphy.c
ramstage-y += lockdown.c
-ramstage-y += me.c
ramstage-y += p2sb.c
ramstage-y += pcie_rp.c
ramstage-y += pmc.c
diff --git a/src/soc/intel/alderlake/include/soc/me.h b/src/soc/intel/alderlake/include/soc/me.h
deleted file mode 100644
index f029991fc3..0000000000
--- a/src/soc/intel/alderlake/include/soc/me.h
+++ /dev/null
@@ -1,81 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-#ifndef _ALDERLAKE_ME_H_
-#define _ALDERLAKE_ME_H_
-
-/* ME Host Firmware Status register 1 */
-union me_hfsts1 {
- uint32_t data;
- struct {
- uint32_t working_state : 4;
- uint32_t mfg_mode : 1;
- uint32_t fpt_bad : 1;
- uint32_t operation_state : 3;
- uint32_t fw_init_complete : 1;
- uint32_t ft_bup_ld_flr : 1;
- uint32_t update_in_progress : 1;
- uint32_t error_code : 4;
- uint32_t operation_mode : 4;
- uint32_t reserved_0 : 4;
- uint32_t boot_options_present : 1;
- uint32_t invoke_enhance_dbg_mode: 1;
- uint32_t reserved_1 : 5;
- uint32_t d0i3_support_valid : 1;
- } __packed fields;
-};
-
-/* Host Firmware Status Register 2 */
-union me_hfsts2 {
- uint32_t data;
- struct {
- uint32_t reserved_0 : 4;
- uint32_t cpu_replaced : 1;
- uint32_t reserved_1 : 3;
- uint32_t cpu_replaced_valid : 1;
- uint32_t low_power_state : 1;
- uint32_t reserved_2 : 22;
- } __packed fields;
-};
-
-/* ME Host Firmware Status Register 3 */
-union me_hfsts3 {
- uint32_t data;
- struct {
- uint32_t reserved_0 : 4;
- uint32_t fw_sku : 3;
- uint32_t reserved_1 : 25;
- } __packed fields;
-};
-
-
-/* Host Firmware Status Register 4 */
-union me_hfsts4 {
- uint32_t data;
- struct {
- uint32_t rsvd0;
- } __packed fields;
-};
-
-/* Host Firmware Status Register 5 */
-union me_hfsts5 {
- uint32_t data;
- struct {
- uint32_t rsvd0;
- } __packed fields;
-};
-
-/* Host Firmware Status Register 6 */
-union me_hfsts6 {
- u32 data;
- struct {
- uint32_t reserved_0 : 1;
- uint32_t cpu_debug_disable : 1;
- uint32_t reserved_1 : 19;
- uint32_t manuf_lock : 1;
- uint32_t reserved_2 : 8;
- uint32_t fpf_soc_lock : 1;
- uint32_t txt_support : 1;
- } __packed fields;
-};
-
-#endif /* _ALDERLAKE_ME_H_ */
diff --git a/src/soc/intel/alderlake/me.c b/src/soc/intel/alderlake/me.c
deleted file mode 100644
index 6239eb24da..0000000000
--- a/src/soc/intel/alderlake/me.c
+++ /dev/null
@@ -1,98 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-#include <bootstate.h>
-#include <intelblocks/cse.h>
-#include <console/console.h>
-#include <soc/me.h>
-#include <stdint.h>
-
-static bool is_manuf_mode(union me_hfsts1 hfsts1, union me_hfsts6 hfsts6)
-{
- /*
- * ME manufacturing mode is disabled if the descriptor is locked, fuses
- * are programmed and manufacturing variables are locked.
- */
- return !((hfsts1.fields.mfg_mode == 0) &&
- (hfsts6.fields.fpf_soc_lock == 1) &&
- (hfsts6.fields.manuf_lock == 1));
-}
-
-static void dump_me_status(void *unused)
-{
- union me_hfsts1 hfsts1;
- union me_hfsts2 hfsts2;
- union me_hfsts3 hfsts3;
- union me_hfsts4 hfsts4;
- union me_hfsts5 hfsts5;
- union me_hfsts6 hfsts6;
- bool manuf_mode;
-
- if (!is_cse_enabled())
- return;
-
- hfsts1.data = me_read_config32(PCI_ME_HFSTS1);
- hfsts2.data = me_read_config32(PCI_ME_HFSTS2);
- hfsts3.data = me_read_config32(PCI_ME_HFSTS3);
- hfsts4.data = me_read_config32(PCI_ME_HFSTS4);
- hfsts5.data = me_read_config32(PCI_ME_HFSTS5);
- hfsts6.data = me_read_config32(PCI_ME_HFSTS6);
-
- printk(BIOS_DEBUG, "ME: HFSTS1 : 0x%08X\n", hfsts1.data);
- printk(BIOS_DEBUG, "ME: HFSTS2 : 0x%08X\n", hfsts2.data);
- printk(BIOS_DEBUG, "ME: HFSTS3 : 0x%08X\n", hfsts3.data);
- printk(BIOS_DEBUG, "ME: HFSTS4 : 0x%08X\n", hfsts4.data);
- printk(BIOS_DEBUG, "ME: HFSTS5 : 0x%08X\n", hfsts5.data);
- printk(BIOS_DEBUG, "ME: HFSTS6 : 0x%08X\n", hfsts6.data);
-
- manuf_mode = is_manuf_mode(hfsts1, hfsts6);
- printk(BIOS_DEBUG, "ME: Manufacturing Mode : %s\n",
- manuf_mode ? "YES" : "NO");
- /*
- * The SPI Protection Mode bit reflects SPI descriptor
- * locked(0) or unlocked(1).
- */
- printk(BIOS_DEBUG, "ME: SPI Protection Mode Enabled : %s\n",
- hfsts1.fields.mfg_mode ? "NO" : "YES");
- printk(BIOS_DEBUG, "ME: FPFs Committed : %s\n",
- hfsts6.fields.fpf_soc_lock ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: Manufacturing Vars Locked : %s\n",
- hfsts6.fields.manuf_lock ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: FW Partition Table : %s\n",
- hfsts1.fields.fpt_bad ? "BAD" : "OK");
- printk(BIOS_DEBUG, "ME: Bringup Loader Failure : %s\n",
- hfsts1.fields.ft_bup_ld_flr ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: Firmware Init Complete : %s\n",
- hfsts1.fields.fw_init_complete ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: Boot Options Present : %s\n",
- hfsts1.fields.boot_options_present ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: Update In Progress : %s\n",
- hfsts1.fields.update_in_progress ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: D0i3 Support : %s\n",
- hfsts1.fields.d0i3_support_valid ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: Low Power State Enabled : %s\n",
- hfsts2.fields.low_power_state ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: CPU Replaced : %s\n",
- hfsts2.fields.cpu_replaced ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: CPU Replacement Valid : %s\n",
- hfsts2.fields.cpu_replaced_valid ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: Current Working State : %u\n",
- hfsts1.fields.working_state);
- printk(BIOS_DEBUG, "ME: Current Operation State : %u\n",
- hfsts1.fields.operation_state);
- printk(BIOS_DEBUG, "ME: Current Operation Mode : %u\n",
- hfsts1.fields.operation_mode);
- printk(BIOS_DEBUG, "ME: Error Code : %u\n",
- hfsts1.fields.error_code);
- printk(BIOS_DEBUG, "ME: Enhanced Debug Mode : %s\n",
- hfsts1.fields.invoke_enhance_dbg_mode ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: CPU Debug Disabled : %s\n",
- hfsts6.fields.cpu_debug_disable ? "YES" : "NO");
- printk(BIOS_DEBUG, "ME: TXT Support : %s\n",
- hfsts6.fields.txt_support ? "YES" : "NO");
-
- if (CONFIG(SOC_INTEL_CSE_LITE_SKU))
- cse_log_ro_write_protection_info(manuf_mode);
-}
-
-BOOT_STATE_INIT_ENTRY(BS_DEV_ENABLE, BS_ON_EXIT, print_me_fw_version, NULL);
-BOOT_STATE_INIT_ENTRY(BS_OS_RESUME_CHECK, BS_ON_EXIT, dump_me_status, NULL);