Maxim Polyakov uploaded patch set #28 to this change.

View Change

mb/ocp/tiogapass: use common GPIO config [WIP]

According to changes in the soc/xeon_sp code [1,2], server motherboards
with Lewisburg PCH can use the common pad configuration from soc/intel/
common, using macros PAD_CFG_ instead of the FSP-style GPIO definitions.

This patch adds the GPIO configuration, which has the format required by
the driver from common/gpio. The data for this was taken from the
inteltool register dump with AMI firmware and converted to macros using
intelp2m (pch-pads-parser) [3,4].

The following pads in the AMI firmware used SMI and SCI input кoute, but
since currently SMM GPIO event handlers are absent in the code for
xeon-sp, these pads will be configured as input PAD_CFG_GPI:

PAD_CFG_GPI_SCI(GPP_C14, NONE, PLTRST, LEVEL, NONE),
PAD_CFG_GPI_SMI(GPP_C22, NONE, PLTRST, LEVEL, NONE),
PAD_CFG_GPI_SMI(GPP_C23, NONE, DEEP, LEVEL, INVERT),
PAD_CFG_GPI_SMI(GPP_D0, NONE, PLTRST, LEVEL, INVERT),
PAD_CFG_GPI_SMI_DRIVER(GPP_E0, NONE, DEEP, LEVEL, NONE),
PAD_CFG_GPI_SMI_DRIVER(GPP_E1, NONE, DEEP, LEVEL, NONE).

The above configuration should be restored back when the appropriate SMM
handlers are added to the code.

[1] https://review.coreboot.org/c/coreboot/+/39425
[2] https://review.coreboot.org/c/coreboot/+/39428
[3] https://review.coreboot.org/c/coreboot/+/35643
[4] https://github.com/maxpoliak/pch-pads-parser

Change-Id: I818d040fa33f3e7b94b73c9bbbafca5df424616d
Signed-off-by: Maxim Polyakov <max.senia.poliak@gmail.com>
---
M src/mainboard/ocp/tiogapass/Makefile.inc
M src/mainboard/ocp/tiogapass/bootblock.c
A src/mainboard/ocp/tiogapass/gpio.h
M src/mainboard/ocp/tiogapass/ramstage.c
M src/mainboard/ocp/tiogapass/romstage.c
5 files changed, 619 insertions(+), 12 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/39427/28

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I818d040fa33f3e7b94b73c9bbbafca5df424616d
Gerrit-Change-Number: 39427
Gerrit-PatchSet: 28
Gerrit-Owner: Maxim Polyakov <max.senia.poliak@gmail.com>
Gerrit-Reviewer: Andrey Petrov <anpetrov@fb.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: David Hendricks <david.hendricks@gmail.com>
Gerrit-CC: Patrick Rudolph <siro@das-labor.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newpatchset