Patch set updated for coreboot: soc/intel/apollolake: Add function to translate gpio_t into ACPI pin

[Duncan Laurie]

Duncan Laurie ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15513

-gerrit

commit c3e2fbef77e90f6a459e638f21ed9bba8f14b4ad
Author: Duncan Laurie 
Date:   Wed Jun 29 10:47:48 2016 -0700

soc/intel/apollolake: Add function to translate gpio_t into ACPI pin

There are four GPIO communities in this SOC and they are implemented
    as separate ACPI devices.  This means the pin number that is used in
    an ACPI GPIO declaration needs to be relative to the community that
    the pin resides in.  Also select GENERIC_GPIO_LIB in the SOC Kconfig
    so this function actually gets used.

This was tested on the reef mainboard by verifying the output of the
    SSDT for the Maxim 98357A codec that the assigned GPIO_76 is listed
    as pin 0x24 which is the value relative to the Northwest community.

Change-Id: Iad2ab8eccf4c91185a075ffce8d41c81f06c1113
    Signed-off-by: Duncan Laurie 
---
 src/soc/intel/apollolake/Kconfig |  1 +
 src/soc/intel/apollolake/gpio.c  | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/src/soc/intel/apollolake/Kconfig b/src/soc/intel/apollolake/Kconfig
index 749c15b..8fd2967 100644
--- a/src/soc/intel/apollolake/Kconfig
+++ b/src/soc/intel/apollolake/Kconfig
@@ -21,6 +21,7 @@ config CPU_SPECIFIC_OPTIONS
    select C_ENVIRONMENT_BOOTBLOCK
    select COLLECT_TIMESTAMPS
    select COMMON_FADT
+	select GENERIC_GPIO_LIB
    select HAVE_INTEL_FIRMWARE
    select HAVE_SMI_HANDLER
    select MMCONF_SUPPORT
diff --git a/src/soc/intel/apollolake/gpio.c b/src/soc/intel/apollolake/gpio.c
index d0ef648..ac0d83b 100644
--- a/src/soc/intel/apollolake/gpio.c
+++ b/src/soc/intel/apollolake/gpio.c
@@ -132,3 +132,21 @@ const char *gpio_acpi_path(gpio_t gpio_num)

return NULL;
 }
+
+uint16_t gpio_acpi_pin(gpio_t gpio_num)
+{
+	const struct pad_community *comm = gpio_get_community(gpio_num);
+
+	switch (comm->port) {
+	case GPIO_NORTH:
+		return PAD_N(gpio_num);
+	case GPIO_NORTHWEST:
+		return PAD_NW(gpio_num);
+	case GPIO_WEST:
+		return PAD_W(gpio_num);
+	case GPIO_SOUTHWEST:
+		return PAD_SW(gpio_num);
+	}
+
+	return gpio_num;
+}