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

soc/intel/cannonlake: Fix PMC and GPIO block values for PCH-H 

Some of the values used for GPIO_CFG and MISCCFG were not correct,
causing GPEs to not work correctly. This adjusts them according to the
values found in the original ACPI tables for the System76 Gazelle.

Unfortunately, the Intel documentation[1] mentioned below is
also incorrect. I have mentioned this to Intel already. The source
for the Intel CoffeeLake FSP also confirms these new numbers.

This was tested on a System76 Gazelle (gaze14). The EC uses GPP_K3 for
its GPE and GPP_K6 is used for the lid switch GPE. Both function
correctly after applying this change.

[1] Intel Document #572235:
    Intel ® 300 Series Chipset Families
    Platform Controller Hub
    External Design Specification (EDS) - Volume 2 of 2

Change-Id: I4ecc9552468037598ef5d4e10122d660dcbfe71d
Signed-off-by: Jeremy Soller <jeremy@system76.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33941
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
This commit is contained in:
Jeremy Soller 2019-07-01 08:14:39 -06:00 committed by Felix Held
parent c32ccb779c
commit 65f03b7c42
2 changed files with 24 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/soc/intel/cannonlake/include/soc/gpio_soc_defs_cnp_h.h
View File

@ -22,18 +22,18 @@
* communities. * communities.
*/ */
#define GPP_A 0 #define GPP_A 0x0
#define GPP_B 1 #define GPP_B 0x1
#define GPP_C 2 #define GPP_C 0x2
#define GPP_D 3 #define GPP_D 0x3
#define GPP_G 4 #define GPP_G 0x4
#define GPP_K 5 #define GPD 0x5
#define GPP_H 6 #define GPP_E 0x6
#define GPP_E 7 #define GPP_F 0x7
#define GPP_F 8 #define GPP_H 0x8
#define GPP_I 9 #define GPP_K 0x9
#define GPP_J 0xA #define GPP_I 0xA
#define GPD 0xC #define GPP_J 0xB
#define GPIO_NUM_GROUPS 12 #define GPIO_NUM_GROUPS 12
#define GPIO_MAX_NUM_PER_GROUP 24 #define GPIO_MAX_NUM_PER_GROUP 24

19
src/soc/intel/cannonlake/include/soc/pmc.h
View File

@ -117,18 +117,23 @@
#define GPE0_DW_SHIFT(x) (4*(x)) #define GPE0_DW_SHIFT(x) (4*(x))
#if CONFIG(SOC_INTEL_CANNONLAKE_PCH_H) #if CONFIG(SOC_INTEL_CANNONLAKE_PCH_H)
/*
* The values for GPIO_CFG in Intel Document #572235 are incorrect.
* These values now match what is used by the Intel CoffeeLake FSP,
* please do not modify them.
*/
#define PMC_GPP_A 0x0 #define PMC_GPP_A 0x0
#define PMC_GPP_B 0x1 #define PMC_GPP_B 0x1
#define PMC_GPP_C 0x2 #define PMC_GPP_C 0x2
#define PMC_GPP_D 0x3 #define PMC_GPP_D 0x3
#define PMC_GPP_E 0x7 #define PMC_GPP_E 0xA
#define PMC_GPP_F 0x8 #define PMC_GPP_F 0xB
#define PMC_GPP_G 0x4 #define PMC_GPP_G 0x4
#define PMC_GPP_H 0x6 #define PMC_GPP_H 0x9
#define PMC_GPP_I 0x9 #define PMC_GPP_I 0xC
#define PMC_GPP_J 0xA #define PMC_GPP_J 0xD
#define PMC_GPP_K 0x5 #define PMC_GPP_K 0x8
#define PMC_GPD 0xC #define PMC_GPD 0x7
#else #else
#define PMC_GPP_A 0x0 #define PMC_GPP_A 0x0
#define PMC_GPP_B 0x1 #define PMC_GPP_B 0x1