summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2019-01-31 21:51:54 +0100
committerPatrick Georgi <pgeorgi@google.com>2019-02-06 11:08:52 +0000
commit1bde3124b487ae76e8d5433eba0ba8c58c67a7ea (patch)
tree8a8c799fdfa22df03f7c0c37c9e371f8c8f6346c
parente51c95179943b5130e6b7d2141d1c6258f9e0898 (diff)
downloadcoreboot-1bde3124b487ae76e8d5433eba0ba8c58c67a7ea.tar.xz
mb/lenovo/x200/gpio.c: Unclutter the code
Some settings don't make sense like specifying input/output on native ports or high/low on input ports. Change-Id: Ib37837b9cdb8bb05e2523e0c43cc71fe4fbf243b Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/31187 Reviewed-by: Patrick Georgi <pgeorgi@google.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/mainboard/lenovo/x200/gpio.c189
1 files changed, 4 insertions, 185 deletions
diff --git a/src/mainboard/lenovo/x200/gpio.c b/src/mainboard/lenovo/x200/gpio.c
index 5906fc311f..2da409738d 100644
--- a/src/mainboard/lenovo/x200/gpio.c
+++ b/src/mainboard/lenovo/x200/gpio.c
@@ -14,7 +14,6 @@
#include <southbridge/intel/common/gpio.h>
const struct pch_gpio_set1 pch_gpio_set1_mode = {
- .gpio0 = GPIO_MODE_NATIVE,
.gpio1 = GPIO_MODE_GPIO,
.gpio2 = GPIO_MODE_GPIO,
.gpio3 = GPIO_MODE_GPIO,
@@ -24,32 +23,19 @@ const struct pch_gpio_set1 pch_gpio_set1_mode = {
.gpio7 = GPIO_MODE_GPIO,
.gpio8 = GPIO_MODE_GPIO,
.gpio9 = GPIO_MODE_GPIO,
- .gpio10 = GPIO_MODE_NATIVE,
- .gpio11 = GPIO_MODE_NATIVE,
- .gpio12 = GPIO_MODE_NATIVE,
.gpio13 = GPIO_MODE_GPIO,
- .gpio14 = GPIO_MODE_NATIVE,
- .gpio15 = GPIO_MODE_NATIVE,
- .gpio16 = GPIO_MODE_NATIVE,
.gpio17 = GPIO_MODE_GPIO,
.gpio18 = GPIO_MODE_GPIO,
.gpio19 = GPIO_MODE_GPIO,
.gpio20 = GPIO_MODE_GPIO,
.gpio21 = GPIO_MODE_GPIO,
.gpio22 = GPIO_MODE_GPIO,
- .gpio23 = GPIO_MODE_NATIVE,
.gpio24 = GPIO_MODE_GPIO,
- .gpio25 = GPIO_MODE_NATIVE,
- .gpio26 = GPIO_MODE_NATIVE,
.gpio27 = GPIO_MODE_GPIO,
.gpio28 = GPIO_MODE_GPIO,
- .gpio29 = GPIO_MODE_NATIVE,
- .gpio30 = GPIO_MODE_NATIVE,
- .gpio31 = GPIO_MODE_NATIVE,
};
const struct pch_gpio_set1 pch_gpio_set1_direction = {
- .gpio0 = GPIO_DIR_OUTPUT,
.gpio1 = GPIO_DIR_INPUT,
.gpio2 = GPIO_DIR_INPUT,
.gpio3 = GPIO_DIR_INPUT,
@@ -59,238 +45,71 @@ const struct pch_gpio_set1 pch_gpio_set1_direction = {
.gpio7 = GPIO_DIR_INPUT,
.gpio8 = GPIO_DIR_INPUT,
.gpio9 = GPIO_DIR_OUTPUT,
- .gpio10 = GPIO_DIR_INPUT,
- .gpio11 = GPIO_DIR_INPUT,
- .gpio12 = GPIO_DIR_OUTPUT,
.gpio13 = GPIO_DIR_INPUT,
- .gpio14 = GPIO_DIR_INPUT,
- .gpio15 = GPIO_DIR_OUTPUT,
- .gpio16 = GPIO_DIR_OUTPUT,
.gpio17 = GPIO_DIR_INPUT,
.gpio18 = GPIO_DIR_INPUT,
.gpio19 = GPIO_DIR_OUTPUT,
.gpio20 = GPIO_DIR_OUTPUT,
.gpio21 = GPIO_DIR_INPUT,
.gpio22 = GPIO_DIR_OUTPUT,
- .gpio23 = GPIO_DIR_INPUT,
.gpio24 = GPIO_DIR_INPUT,
- .gpio25 = GPIO_DIR_OUTPUT,
- .gpio26 = GPIO_DIR_OUTPUT,
.gpio27 = GPIO_DIR_OUTPUT,
.gpio28 = GPIO_DIR_OUTPUT,
- .gpio29 = GPIO_DIR_INPUT,
- .gpio30 = GPIO_DIR_INPUT,
- .gpio31 = GPIO_DIR_INPUT,
};
const struct pch_gpio_set1 pch_gpio_set1_level = {
- .gpio0 = GPIO_LEVEL_HIGH,
- .gpio1 = GPIO_LEVEL_HIGH,
- .gpio2 = GPIO_LEVEL_HIGH,
- .gpio3 = GPIO_LEVEL_HIGH,
- .gpio4 = GPIO_LEVEL_HIGH,
- .gpio5 = GPIO_LEVEL_HIGH,
- .gpio6 = GPIO_LEVEL_LOW,
- .gpio7 = GPIO_LEVEL_LOW,
- .gpio8 = GPIO_LEVEL_HIGH,
.gpio9 = GPIO_LEVEL_HIGH,
- .gpio10 = GPIO_LEVEL_HIGH,
- .gpio11 = GPIO_LEVEL_HIGH,
- .gpio12 = GPIO_LEVEL_LOW,
- .gpio13 = GPIO_LEVEL_HIGH,
- .gpio14 = GPIO_LEVEL_HIGH,
- .gpio15 = GPIO_LEVEL_HIGH,
- .gpio16 = GPIO_LEVEL_LOW,
- .gpio17 = GPIO_LEVEL_HIGH,
- .gpio18 = GPIO_LEVEL_LOW,
.gpio19 = GPIO_LEVEL_HIGH,
.gpio20 = GPIO_LEVEL_HIGH,
- .gpio21 = GPIO_LEVEL_HIGH,
.gpio22 = GPIO_LEVEL_HIGH,
- .gpio23 = GPIO_LEVEL_HIGH,
- .gpio24 = GPIO_LEVEL_HIGH,
- .gpio25 = GPIO_LEVEL_HIGH,
- .gpio26 = GPIO_LEVEL_LOW,
.gpio27 = GPIO_LEVEL_LOW,
.gpio28 = GPIO_LEVEL_LOW,
- .gpio29 = GPIO_LEVEL_HIGH,
- .gpio30 = GPIO_LEVEL_HIGH,
- .gpio31 = GPIO_LEVEL_HIGH,
};
-const struct pch_gpio_set1 pch_gpio_set1_blink = {
- .gpio0 = GPIO_NO_BLINK,
- .gpio1 = GPIO_NO_BLINK,
- .gpio2 = GPIO_NO_BLINK,
- .gpio3 = GPIO_NO_BLINK,
- .gpio4 = GPIO_NO_BLINK,
- .gpio5 = GPIO_NO_BLINK,
- .gpio6 = GPIO_NO_BLINK,
- .gpio7 = GPIO_NO_BLINK,
- .gpio8 = GPIO_NO_BLINK,
- .gpio9 = GPIO_NO_BLINK,
- .gpio10 = GPIO_NO_BLINK,
- .gpio11 = GPIO_NO_BLINK,
- .gpio12 = GPIO_NO_BLINK,
- .gpio13 = GPIO_NO_BLINK,
- .gpio14 = GPIO_NO_BLINK,
- .gpio15 = GPIO_NO_BLINK,
- .gpio16 = GPIO_NO_BLINK,
- .gpio17 = GPIO_NO_BLINK,
- .gpio18 = GPIO_NO_BLINK,
- .gpio19 = GPIO_NO_BLINK,
- .gpio20 = GPIO_NO_BLINK,
- .gpio21 = GPIO_NO_BLINK,
- .gpio22 = GPIO_NO_BLINK,
- .gpio23 = GPIO_NO_BLINK,
- .gpio24 = GPIO_NO_BLINK,
- .gpio25 = GPIO_NO_BLINK,
- .gpio26 = GPIO_NO_BLINK,
- .gpio27 = GPIO_NO_BLINK,
- .gpio28 = GPIO_NO_BLINK,
- .gpio29 = GPIO_NO_BLINK,
- .gpio30 = GPIO_NO_BLINK,
- .gpio31 = GPIO_NO_BLINK,
+const struct pch_gpio_set1 pch_gpio_set1_invert = {
+ .gpio1 = GPIO_INVERT,
+ .gpio8 = GPIO_INVERT,
};
-const struct pch_gpio_set1 pch_gpio_set1_invert = {
- .gpio0 = GPIO_NO_INVERT,
- .gpio1 = GPIO_INVERT,
- .gpio2 = GPIO_NO_INVERT,
- .gpio3 = GPIO_NO_INVERT,
- .gpio4 = GPIO_NO_INVERT,
- .gpio5 = GPIO_NO_INVERT,
- .gpio6 = GPIO_NO_INVERT,
- .gpio7 = GPIO_NO_INVERT,
- .gpio8 = GPIO_INVERT,
- .gpio9 = GPIO_NO_INVERT,
- .gpio10 = GPIO_NO_INVERT,
- .gpio11 = GPIO_NO_INVERT,
- .gpio12 = GPIO_NO_INVERT,
- .gpio13 = GPIO_NO_INVERT,
- .gpio14 = GPIO_NO_INVERT,
- .gpio15 = GPIO_NO_INVERT,
- .gpio16 = GPIO_NO_INVERT,
- .gpio17 = GPIO_NO_INVERT,
- .gpio18 = GPIO_NO_INVERT,
- .gpio19 = GPIO_NO_INVERT,
- .gpio20 = GPIO_NO_INVERT,
- .gpio21 = GPIO_NO_INVERT,
- .gpio22 = GPIO_NO_INVERT,
- .gpio23 = GPIO_NO_INVERT,
- .gpio24 = GPIO_NO_INVERT,
- .gpio25 = GPIO_NO_INVERT,
- .gpio26 = GPIO_NO_INVERT,
- .gpio27 = GPIO_NO_INVERT,
- .gpio28 = GPIO_NO_INVERT,
- .gpio29 = GPIO_NO_INVERT,
- .gpio30 = GPIO_NO_INVERT,
- .gpio31 = GPIO_NO_INVERT,
+const struct pch_gpio_set1 pch_gpio_set1_blink = {
};
const struct pch_gpio_set2 pch_gpio_set2_mode = {
- .gpio32 = GPIO_MODE_NATIVE,
.gpio33 = GPIO_MODE_GPIO,
.gpio34 = GPIO_MODE_GPIO,
- .gpio35 = GPIO_MODE_NATIVE,
.gpio36 = GPIO_MODE_GPIO,
.gpio37 = GPIO_MODE_GPIO,
.gpio38 = GPIO_MODE_GPIO,
.gpio39 = GPIO_MODE_GPIO,
- .gpio40 = GPIO_MODE_NATIVE,
.gpio41 = GPIO_MODE_GPIO,
.gpio42 = GPIO_MODE_GPIO,
- .gpio43 = GPIO_MODE_NATIVE,
- .gpio44 = GPIO_MODE_NATIVE,
- .gpio45 = GPIO_MODE_NATIVE,
- .gpio46 = GPIO_MODE_NATIVE,
- .gpio47 = GPIO_MODE_NATIVE,
.gpio48 = GPIO_MODE_GPIO,
.gpio49 = GPIO_MODE_GPIO,
- .gpio50 = GPIO_MODE_NATIVE,
- .gpio51 = GPIO_MODE_NATIVE,
- .gpio52 = GPIO_MODE_NATIVE,
- .gpio53 = GPIO_MODE_NATIVE,
- .gpio54 = GPIO_MODE_NATIVE,
- .gpio55 = GPIO_MODE_NATIVE,
.gpio56 = GPIO_MODE_GPIO,
.gpio57 = GPIO_MODE_GPIO,
- .gpio58 = GPIO_MODE_NATIVE,
- .gpio59 = GPIO_MODE_NATIVE,
- .gpio60 = GPIO_MODE_NATIVE,
- .gpio61 = GPIO_MODE_NATIVE,
- .gpio62 = GPIO_MODE_NATIVE,
- .gpio63 = GPIO_MODE_NATIVE,
};
const struct pch_gpio_set2 pch_gpio_set2_direction = {
- .gpio32 = GPIO_DIR_INPUT,
.gpio33 = GPIO_DIR_OUTPUT,
.gpio34 = GPIO_DIR_OUTPUT,
- .gpio35 = GPIO_DIR_OUTPUT,
.gpio36 = GPIO_DIR_INPUT,
.gpio37 = GPIO_DIR_INPUT,
.gpio38 = GPIO_DIR_INPUT,
.gpio39 = GPIO_DIR_INPUT,
- .gpio40 = GPIO_DIR_INPUT,
.gpio41 = GPIO_DIR_OUTPUT,
.gpio42 = GPIO_DIR_OUTPUT,
- .gpio43 = GPIO_DIR_INPUT,
- .gpio44 = GPIO_DIR_INPUT,
- .gpio45 = GPIO_DIR_INPUT,
- .gpio46 = GPIO_DIR_INPUT,
- .gpio47 = GPIO_DIR_INPUT,
.gpio48 = GPIO_DIR_INPUT,
.gpio49 = GPIO_DIR_OUTPUT,
- .gpio50 = GPIO_DIR_INPUT,
- .gpio51 = GPIO_DIR_OUTPUT,
- .gpio52 = GPIO_DIR_INPUT,
- .gpio53 = GPIO_DIR_OUTPUT,
- .gpio54 = GPIO_DIR_INPUT,
- .gpio55 = GPIO_DIR_OUTPUT,
.gpio56 = GPIO_DIR_INPUT,
.gpio57 = GPIO_DIR_INPUT,
- .gpio58 = GPIO_DIR_INPUT,
- .gpio59 = GPIO_DIR_INPUT,
- .gpio60 = GPIO_DIR_INPUT,
- .gpio61 = GPIO_DIR_OUTPUT,
- .gpio62 = GPIO_DIR_OUTPUT,
- .gpio63 = GPIO_DIR_OUTPUT,
};
const struct pch_gpio_set2 pch_gpio_set2_level = {
- .gpio32 = GPIO_LEVEL_HIGH,
.gpio33 = GPIO_LEVEL_HIGH,
.gpio34 = GPIO_LEVEL_LOW,
- .gpio35 = GPIO_LEVEL_LOW,
- .gpio36 = GPIO_LEVEL_HIGH,
- .gpio37 = GPIO_LEVEL_LOW,
- .gpio38 = GPIO_LEVEL_HIGH,
- .gpio39 = GPIO_LEVEL_LOW,
- .gpio40 = GPIO_LEVEL_HIGH,
.gpio41 = GPIO_LEVEL_HIGH,
.gpio42 = GPIO_LEVEL_HIGH,
- .gpio43 = GPIO_LEVEL_HIGH,
- .gpio44 = GPIO_LEVEL_HIGH,
- .gpio45 = GPIO_LEVEL_HIGH,
- .gpio46 = GPIO_LEVEL_HIGH,
- .gpio47 = GPIO_LEVEL_HIGH,
- .gpio48 = GPIO_LEVEL_HIGH,
.gpio49 = GPIO_LEVEL_HIGH,
- .gpio50 = GPIO_LEVEL_HIGH,
- .gpio51 = GPIO_LEVEL_HIGH,
- .gpio52 = GPIO_LEVEL_HIGH,
- .gpio53 = GPIO_LEVEL_HIGH,
- .gpio54 = GPIO_LEVEL_HIGH,
- .gpio55 = GPIO_LEVEL_HIGH,
- .gpio56 = GPIO_LEVEL_HIGH,
- .gpio57 = GPIO_LEVEL_LOW,
- .gpio58 = GPIO_LEVEL_HIGH,
- .gpio59 = GPIO_LEVEL_HIGH,
- .gpio60 = GPIO_LEVEL_HIGH,
- .gpio61 = GPIO_LEVEL_LOW,
- .gpio62 = GPIO_LEVEL_LOW,
- .gpio63 = GPIO_LEVEL_LOW,
};
const struct pch_gpio_map mainboard_gpio_map = {