diff options
| author | Shamile Khan <shamile.khan@intel.com> | 2019-01-10 18:46:02 -0800 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-01-28 13:37:18 +0000 |
| commit | 6afeef829f1a47aa8817275ae1bac2fb46bea18a (patch) | |
| tree | 157ec3cd95593bd6d0c77700f30a982ee4ebaff9 | |
| parent | 6dda1e05565785e6fa41b33957c1096d47ef7c77 (diff) | |
mb/google/octopus: Fix termination for unused dual voltage pins
These pins should not have pull downs configured in standby state as that
can cause contention on the termination circuitry and lead to incorrect
behavior as per Doc# 572688 Gemini Lake Processor GPIOTermination
Configuration.
BUG=b:79982669
TEST=Checked that code compiles with changes.
Change-Id: If3cadc000ec6fc56019ee3f57e556dc819d5e0a5
Signed-off-by: Shamile Khan <shamile.khan@intel.com>
Reviewed-on: https://review.coreboot.org/c/30823
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
| -rw-r--r-- | src/mainboard/google/octopus/variants/casta/gpio.c | 6 | ||||
| -rw-r--r-- | src/mainboard/google/octopus/variants/fleex/gpio.c | 4 |
2 files changed, 5 insertions, 5 deletions
diff --git a/src/mainboard/google/octopus/variants/casta/gpio.c b/src/mainboard/google/octopus/variants/casta/gpio.c index 1746e4ff6b..fcb7ab5d56 100644 --- a/src/mainboard/google/octopus/variants/casta/gpio.c +++ b/src/mainboard/google/octopus/variants/casta/gpio.c @@ -27,11 +27,11 @@ static const struct pad_config default_override_table[] = { PAD_NC(GPIO_67, UP_20K), /* EN_PP3300_DX_LTE_SOC -- unused */ - PAD_NC(GPIO_105, DN_20K), /* TOUCHSCREEN_RST -- unused */ + PAD_NC(GPIO_105, UP_20K), /* TOUCHSCREEN_RST -- unused */ PAD_NC(GPIO_108, NONE), /* PMU_SUSCLK -- unused */ - PAD_NC(GPIO_114, DN_20K), /* I2C7 Touchscreen -- unused */ - PAD_NC(GPIO_115, DN_20K), /* I2C7 Touchscreen -- unused */ + PAD_NC(GPIO_114, UP_20K), /* I2C7 Touchscreen -- unused */ + PAD_NC(GPIO_115, UP_20K), /* I2C7 Touchscreen -- unused */ PAD_NC(GPIO_117, UP_20K), /* PCIE_WAKE1_B - No LTE*/ PAD_NC(GPIO_119, UP_20K), /* PCIE_WAKE3_B - only use CNVI */ diff --git a/src/mainboard/google/octopus/variants/fleex/gpio.c b/src/mainboard/google/octopus/variants/fleex/gpio.c index b1f377b32b..6bb78aa373 100644 --- a/src/mainboard/google/octopus/variants/fleex/gpio.c +++ b/src/mainboard/google/octopus/variants/fleex/gpio.c @@ -28,8 +28,8 @@ static const struct pad_config default_override_table[] = { PAD_NC(GPIO_67, UP_20K), PAD_NC(GPIO_117, UP_20K), - PAD_NC(GPIO_138, DN_20K), - PAD_NC(GPIO_139, DN_20K), + PAD_NC(GPIO_138, UP_20K), + PAD_NC(GPIO_139, UP_20K), PAD_NC(GPIO_140, UP_20K), PAD_NC(GPIO_143, UP_20K), PAD_NC(GPIO_144, UP_20K), |