2 files changed, 37 insertions, 54 deletions

diff --git a/src/soc/mediatek/mt8188/i2c.c b/src/soc/mediatek/mt8188/i2c.c
index 1de37c5052..edfe510202 100644
--- a/src/soc/mediatek/mt8188/i2c.c
+++ b/src/soc/mediatek/mt8188/i2c.c
@@ -53,41 +53,34 @@ struct mtk_i2c mtk_i2c_bus_controller[] = {
 _Static_assert(ARRAY_SIZE(mtk_i2c_bus_controller) == I2C_BUS_NUMBER,
 	       "Wrong size of mtk_i2c_bus_controller");
 
-struct pad_func {
-	gpio_t gpio;
-	u8 func;
-};
-
-#define PAD_FUNC(name, func) {GPIO(name), PAD_##name##_FUNC_##func}
-
 static const struct pad_func i2c_funcs[I2C_BUS_NUMBER][2] = {
 	[0] = {
-		PAD_FUNC(SDA0, SDA0),
-		PAD_FUNC(SCL0, SCL0),
+		PAD_FUNC_UP(SDA0, SDA0),
+		PAD_FUNC_UP(SCL0, SCL0),
 	},
 	[1] = {
-		PAD_FUNC(SDA1, SDA1),
-		PAD_FUNC(SCL1, SCL1),
+		PAD_FUNC_UP(SDA1, SDA1),
+		PAD_FUNC_UP(SCL1, SCL1),
 	},
 	[2] = {
-		PAD_FUNC(SDA2, SDA2),
-		PAD_FUNC(SCL2, SCL2),
+		PAD_FUNC_UP(SDA2, SDA2),
+		PAD_FUNC_UP(SCL2, SCL2),
 	},
 	[3] = {
-		PAD_FUNC(SDA3, SDA3),
-		PAD_FUNC(SCL3, SCL3),
+		PAD_FUNC_UP(SDA3, SDA3),
+		PAD_FUNC_UP(SCL3, SCL3),
 	},
 	[4] = {
-		PAD_FUNC(SDA4, SDA4),
-		PAD_FUNC(SCL4, SCL4),
+		PAD_FUNC_UP(SDA4, SDA4),
+		PAD_FUNC_UP(SCL4, SCL4),
 	},
 	[5] = {
-		PAD_FUNC(SDA5, SDA5),
-		PAD_FUNC(SCL5, SCL5),
+		PAD_FUNC_UP(SDA5, SDA5),
+		PAD_FUNC_UP(SCL5, SCL5),
 	},
 	[6] = {
-		PAD_FUNC(SDA6, SDA6),
-		PAD_FUNC(SCL6, SCL6),
+		PAD_FUNC_UP(SDA6, SDA6),
+		PAD_FUNC_UP(SCL6, SCL6),
 	},
 };
 
@@ -98,7 +91,7 @@ static void mtk_i2c_set_gpio_pinmux(uint8_t bus)
 	const struct pad_func *ptr = i2c_funcs[bus];
 	for (size_t i = 0; i < 2; i++) {
 		gpio_set_mode(ptr[i].gpio, ptr[i].func);
-		gpio_set_pull(ptr[i].gpio, GPIO_PULL_ENABLE, GPIO_PULL_UP);
+		gpio_set_pull(ptr[i].gpio, GPIO_PULL_ENABLE, ptr[i].select);
 	}
 }
 
diff --git a/src/soc/mediatek/mt8188/spi.c b/src/soc/mediatek/mt8188/spi.c
index 0596bb9b7f..994663d02c 100644
--- a/src/soc/mediatek/mt8188/spi.c
+++ b/src/soc/mediatek/mt8188/spi.c
@@ -41,61 +41,51 @@ struct mtk_spi_bus spi_bus[SPI_BUS_NUMBER] = {
 	},
 };
 
-struct pad_func {
-	gpio_t gpio;
-	u8 func;
-	enum pull_select select;
-};
-
-#define PAD_FUNC_SEL(name, func, sel) {GPIO(name), PAD_##name##_FUNC_##func, sel}
-#define PAD_FUNC(name, func) {GPIO(name), PAD_##name##_FUNC_##func}
-#define PAD_FUNC_GPIO(name) {GPIO(name), 0}
-
 static const struct pad_func pad_funcs[SPI_BUS_NUMBER][4] = {
 	{
-		PAD_FUNC(SPIM0_MISO, SPIM0_MISO),
+		PAD_FUNC_DOWN(SPIM0_MISO, SPIM0_MISO),
 		PAD_FUNC_GPIO(SPIM0_CSB),
-		PAD_FUNC(SPIM0_MOSI, SPIM0_MOSI),
-		PAD_FUNC(SPIM0_CLK, SPIM0_CLK),
+		PAD_FUNC_DOWN(SPIM0_MOSI, SPIM0_MOSI),
+		PAD_FUNC_DOWN(SPIM0_CLK, SPIM0_CLK),
 	},
 	{
-		PAD_FUNC(SPIM1_MISO, SPIM1_MISO),
+		PAD_FUNC_DOWN(SPIM1_MISO, SPIM1_MISO),
 		PAD_FUNC_GPIO(SPIM1_CSB),
-		PAD_FUNC(SPIM1_MOSI, SPIM1_MOSI),
-		PAD_FUNC(SPIM1_CLK, SPIM1_CLK),
+		PAD_FUNC_DOWN(SPIM1_MOSI, SPIM1_MOSI),
+		PAD_FUNC_DOWN(SPIM1_CLK, SPIM1_CLK),
 	},
 	{
-		PAD_FUNC(SPIM2_MISO, SPIM2_MISO),
+		PAD_FUNC_DOWN(SPIM2_MISO, SPIM2_MISO),
 		PAD_FUNC_GPIO(SPIM2_CSB),
-		PAD_FUNC(SPIM2_MOSI, SPIM2_MOSI),
-		PAD_FUNC(SPIM2_CLK, SPIM2_CLK),
+		PAD_FUNC_DOWN(SPIM2_MOSI, SPIM2_MOSI),
+		PAD_FUNC_DOWN(SPIM2_CLK, SPIM2_CLK),
 	},
 	{
-		PAD_FUNC(DPI_D15, SPIM3_MISO),
+		PAD_FUNC_DOWN(DPI_D15, SPIM3_MISO),
 		PAD_FUNC_GPIO(DPI_D12),
-		PAD_FUNC(DPI_D14, SPIM3_MOSI),
-		PAD_FUNC(DPI_D13, SPIM3_CLK),
+		PAD_FUNC_DOWN(DPI_D14, SPIM3_MOSI),
+		PAD_FUNC_DOWN(DPI_D13, SPIM3_CLK),
 	},
 	{
-		PAD_FUNC(GPIO15, SPIM4_MISO),
+		PAD_FUNC_DOWN(GPIO15, SPIM4_MISO),
 		PAD_FUNC_GPIO(GPIO12),
-		PAD_FUNC(GPIO14, SPIM4_MOSI),
-		PAD_FUNC(GPIO13, SPIM4_CLK),
+		PAD_FUNC_DOWN(GPIO14, SPIM4_MOSI),
+		PAD_FUNC_DOWN(GPIO13, SPIM4_CLK),
 	},
 	{
-		PAD_FUNC(GPIO03, SPIM5_MISO),
+		PAD_FUNC_DOWN(GPIO03, SPIM5_MISO),
 		PAD_FUNC_GPIO(GPIO00),
-		PAD_FUNC(GPIO02, SPIM5_MOSI),
-		PAD_FUNC(GPIO01, SPIM5_CLK),
+		PAD_FUNC_DOWN(GPIO02, SPIM5_MOSI),
+		PAD_FUNC_DOWN(GPIO01, SPIM5_CLK),
 	},
 };
 
 static const struct pad_func nor_pinmux[4] = {
 	/* GPIO 125 ~ 128 */
-	PAD_FUNC_SEL(DMIC1_CLK, SPINOR_CK, GPIO_PULL_DOWN),
-	PAD_FUNC_SEL(DMIC1_DAT, SPINOR_CS, GPIO_PULL_UP),
-	PAD_FUNC_SEL(DMIC1_DAT_R, SPINOR_IO0, GPIO_PULL_DOWN),
-	PAD_FUNC_SEL(DMIC2_CLK, SPINOR_IO1, GPIO_PULL_DOWN),
+	PAD_FUNC(DMIC1_CLK, SPINOR_CK, GPIO_PULL_DOWN),
+	PAD_FUNC(DMIC1_DAT, SPINOR_CS, GPIO_PULL_UP),
+	PAD_FUNC(DMIC1_DAT_R, SPINOR_IO0, GPIO_PULL_DOWN),
+	PAD_FUNC(DMIC2_CLK, SPINOR_IO1, GPIO_PULL_DOWN),
 };
 
 void mtk_snfc_init(void)