[coreboot-gerrit] Change in coreboot[master]: mb/lenovo/t400: Trim down the gpio.h file
Arthur Heymans (Code Review)
gerrit at coreboot.org
Sun Sep 16 16:02:35 CEST 2018
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/28627
Change subject: mb/lenovo/t400: Trim down the gpio.h file
......................................................................
mb/lenovo/t400: Trim down the gpio.h file
Some settings like direction are only used if the mode is GPIO.
Change-Id: I4efc54dfef3721b528b90d49f490014d9132cdf8
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/mainboard/lenovo/t400/gpio.h
1 file changed, 20 insertions(+), 201 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/28627/1
diff --git a/src/mainboard/lenovo/t400/gpio.h b/src/mainboard/lenovo/t400/gpio.h
index 6757c94..261c912 100644
--- a/src/mainboard/lenovo/t400/gpio.h
+++ b/src/mainboard/lenovo/t400/gpio.h
@@ -16,8 +16,7 @@
#include <southbridge/intel/common/gpio.h>
-const struct pch_gpio_set1 pch_gpio_set1_mode = {
- .gpio0 = GPIO_MODE_NATIVE,
+static const struct pch_gpio_set1 pch_gpio_set1_mode = {
.gpio1 = GPIO_MODE_GPIO,
.gpio2 = GPIO_MODE_GPIO,
.gpio3 = GPIO_MODE_GPIO,
@@ -27,273 +26,93 @@
.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,
- .gpio4 = GPIO_DIR_INPUT,
- .gpio5 = GPIO_DIR_INPUT,
- .gpio6 = GPIO_DIR_INPUT,
- .gpio7 = GPIO_DIR_INPUT,
- .gpio8 = GPIO_DIR_INPUT,
- .gpio9 = GPIO_DIR_OUTPUT,
- .gpio10 = GPIO_DIR_INPUT,
- .gpio11 = GPIO_DIR_INPUT,
- .gpio12 = GPIO_DIR_OUTPUT,
+static const struct pch_gpio_set1 pch_gpio_set1_direction = {
+ .gpio1 = GPIO_DIR_INPUT,
+ .gpio2 = GPIO_DIR_INPUT,
+ .gpio3 = GPIO_DIR_INPUT,
+ .gpio4 = GPIO_DIR_INPUT,
+ .gpio5 = GPIO_DIR_INPUT,
+ .gpio6 = GPIO_DIR_INPUT,
+ .gpio7 = GPIO_DIR_INPUT,
+ .gpio8 = GPIO_DIR_INPUT,
+ .gpio9 = 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,
+static const struct pch_gpio_set1 pch_gpio_set1_level = {
+ .gpio9 = GPIO_LEVEL_HIGH,
.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,
+static 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,
+static const struct pch_gpio_set1 pch_gpio_set1_blink = {
};
-const struct pch_gpio_set2 pch_gpio_set2_mode = {
- .gpio32 = GPIO_MODE_NATIVE,
+static const struct pch_gpio_set2 pch_gpio_set2_mode = {
.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,
+static const struct pch_gpio_set2 pch_gpio_set2_direction = {
.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,
+static const struct pch_gpio_set2 pch_gpio_set2_level = {
.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 t400_gpio_map = {
--
To view, visit https://review.coreboot.org/28627
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4efc54dfef3721b528b90d49f490014d9132cdf8
Gerrit-Change-Number: 28627
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180916/cfbc46d1/attachment-0001.html>
More information about the coreboot-gerrit
mailing list