GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/cannonlake: Configure GPIOs again after FSP-S is done 

FSP-S is currently configuring GPIOs that it should not. This results
in issues where mainboard devices don't behave as expected e.g. host
unable to receive TPM interrupts as the pad for the interrupt is
re-configured as something else.

Until FSP-S is fixed, this change adds a workaround by reconfiguring
GPIOs after FSP-S is run.

All mainboards need to call cnl_configure_pads instead of
gpio_configure_pads so that SoC code can maintain a reference to the
GPIO table and use that to re-configure GPIOs after FSP-S is run.

BUG=b:123721147
BRANCH=None
TEST=Verified that there are no TPM IRQ timeouts in boot log on hatch.

Change-Id: I7787aa8f185f633627bcedc7f23504bf4a5250b4
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/31250
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
This commit is contained in:
Furquan Shaikh 2019-02-05 13:59:47 -08:00 committed by Patrick Georgi
parent 6527b1acc7
commit 86d2afb86b
2 changed files with 35 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/soc/intel/cannonlake/chip.c
View File

@ -140,6 +140,33 @@ const char *soc_acpi_name(const struct device *dev)
} }
#endif #endif
/*
* TODO(furquan): Get rid of this workaround once FSP is fixed. Currently, FSP-S
* configures GPIOs when it should not and this results in coreboot GPIO
* configuration being overwritten. Until FSP is fixed, maintain the reference
* of GPIO config table from mainboard and use that to re-configure GPIOs after
* FSP-S is done.
*/
void cnl_configure_pads(const struct pad_config *cfg, size_t num_pads)
{
static const struct pad_config *g_cfg;
static size_t g_num_pads;
/*
* If cfg and num_pads are passed in from mainboard, maintain a
* reference to the GPIO table.
*/
if ((cfg == NULL) || (num_pads == 0)) {
cfg = g_cfg;
num_pads = g_num_pads;
} else {
g_cfg = cfg;
g_num_pads = num_pads;
}
gpio_configure_pads(cfg, num_pads);
}
void soc_init_pre_device(void *chip_info) void soc_init_pre_device(void *chip_info)
{ {
/* Snapshot the current GPIO IRQ polarities. FSP is setting a /* Snapshot the current GPIO IRQ polarities. FSP is setting a
@ -154,6 +181,9 @@ void soc_init_pre_device(void *chip_info)
/* Restore GPIO IRQ polarities back to previous settings. */ /* Restore GPIO IRQ polarities back to previous settings. */
itss_restore_irq_polarities(GPIO_IRQ_START, GPIO_IRQ_END); itss_restore_irq_polarities(GPIO_IRQ_START, GPIO_IRQ_END);
/* TODO(furquan): Get rid of this workaround once FSP is fixed. */
cnl_configure_pads(NULL, 0);
} }
static void pci_domain_set_resources(struct device *dev) static void pci_domain_set_resources(struct device *dev)

6
src/soc/intel/cannonlake/include/soc/gpio.h
View File

@ -16,7 +16,6 @@
#ifndef _SOC_CANNONLAKE_GPIO_H_ #ifndef _SOC_CANNONLAKE_GPIO_H_
#define _SOC_CANNONLAKE_GPIO_H_ #define _SOC_CANNONLAKE_GPIO_H_
#if IS_ENABLED(CONFIG_SOC_INTEL_CANNONLAKE_PCH_H) #if IS_ENABLED(CONFIG_SOC_INTEL_CANNONLAKE_PCH_H)
#include <soc/gpio_defs_cnp_h.h> #include <soc/gpio_defs_cnp_h.h>
#define CROS_GPIO_DEVICE_NAME "INT3450:00" #define CROS_GPIO_DEVICE_NAME "INT3450:00"
@ -26,4 +25,9 @@
#endif #endif
#include <intelblocks/gpio.h> #include <intelblocks/gpio.h>
#ifndef __ACPI__
struct pad_config;
void cnl_configure_pads(const struct pad_config *cfg, size_t num_pads);
#endif
#endif #endif