Jeremy Soller uploaded patch set #5 to this change.

View Change

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

Soem 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>
---
M src/soc/intel/cannonlake/include/soc/gpio_soc_defs_cnp_h.h
M src/soc/intel/cannonlake/include/soc/pmc.h
2 files changed, 19 insertions(+), 19 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/33941/5

To view, visit change 33941. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4ecc9552468037598ef5d4e10122d660dcbfe71d
Gerrit-Change-Number: 33941
Gerrit-PatchSet: 5
Gerrit-Owner: Jeremy Soller <jackpot51@gmail.com>
Gerrit-Reviewer: Jeremy Soller <jackpot51@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: newpatchset