From 7d14af815492d7d886244086ad24dde3d9805381 Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Tue, 7 Feb 2017 11:33:56 -0600 Subject: soc/intel/apollolake: dump CSE status Dump the CSE status registers for potential debugging purposes. Explicitly call out manufacturing mode of the part since it's important shipping devices ensure manufacturing mode is locked down. Intel is planning on writing a common driver so a complete status -> string dumps was not done because (surprise surprise) not all the fields are equal with previous implementations. BUG=chrome-os-partner:62177 BRANCH=reef TEST=Booted and noted dump of CSE status registers. Change-Id: I71d15722bb193877f1569c1d3e7f441302f5bd14 Signed-off-by: Aaron Durbin Reviewed-on: https://review.coreboot.org/18303 Reviewed-by: Duncan Laurie Tested-by: build bot (Jenkins) Reviewed-by: Furquan Shaikh --- src/soc/intel/apollolake/Makefile.inc | 1 + src/soc/intel/apollolake/cse.c | 57 +++++++++++++++++++++++++++++++++++ 2 files changed, 58 insertions(+) create mode 100644 src/soc/intel/apollolake/cse.c (limited to 'src/soc/intel') diff --git a/src/soc/intel/apollolake/Makefile.inc b/src/soc/intel/apollolake/Makefile.inc index 5a65f43485..df93f0bd4d 100644 --- a/src/soc/intel/apollolake/Makefile.inc +++ b/src/soc/intel/apollolake/Makefile.inc @@ -57,6 +57,7 @@ smm-y += uart_early.c ramstage-$(CONFIG_HAVE_ACPI_TABLES) += acpi.c ramstage-y += cpu.c ramstage-y += chip.c +ramstage-y += cse.c ramstage-y += elog.c ramstage-y += flash_ctrlr.c ramstage-y += dsp.c diff --git a/src/soc/intel/apollolake/cse.c b/src/soc/intel/apollolake/cse.c new file mode 100644 index 0000000000..b8ab1f0a1a --- /dev/null +++ b/src/soc/intel/apollolake/cse.c @@ -0,0 +1,57 @@ +/* + * This file is part of the coreboot project. + * + * Copyright 2017 Google, Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include +#include +#include +#include +#include + +#define PCI_ME_HFSTS1 0x40 +#define PCI_ME_HFSTS2 0x48 +#define PCI_ME_HFSTS3 0x60 +#define PCI_ME_HFSTS4 0x64 +#define PCI_ME_HFSTS5 0x68 +#define PCI_ME_HFSTS6 0x6c + +static uint32_t dump_status(int index, int reg_addr) +{ + uint32_t reg = pci_read_config32(CSE_DEV, reg_addr); + + printk(BIOS_DEBUG, "CSE FWSTS%d: 0x%08x\n", index, reg); + + return reg; +} + +static void dump_cse_state(void *unused) +{ + uint32_t fwsts1; + + fwsts1 = dump_status(1, PCI_ME_HFSTS1); + dump_status(2, PCI_ME_HFSTS2); + dump_status(3, PCI_ME_HFSTS3); + dump_status(4, PCI_ME_HFSTS4); + dump_status(5, PCI_ME_HFSTS5); + dump_status(6, PCI_ME_HFSTS6); + + /* Minimal decoding is done here in order to call out most important + pieces. Manufacturing mode needs to be locked down prior to shipping + the product so it's called out explicitly. */ + printk(BIOS_DEBUG, "ME: Manufacturing Mode : %s\n", + (fwsts1 & (1 << 0x4)) ? "YES" : "NO"); +} + +BOOT_STATE_INIT_ENTRY(BS_OS_RESUME, BS_ON_ENTRY, dump_cse_state, NULL); +BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_LOAD, BS_ON_EXIT, dump_cse_state, NULL); -- cgit v1.2.3