[coreboot-gerrit] Change in coreboot[master]: mb/clevo/kbl-u: Move mainboard/gpio.h to variant/gpio.h

[Michael Niewöhner (Code Review)]

Michael Niewöhner has submitted this change. ( https://review.coreboot.org/c/coreboot/+/48297 )

Change subject: mb/clevo/kbl-u: Move mainboard/gpio.h to variant/gpio.h
......................................................................

mb/clevo/kbl-u: Move mainboard/gpio.h to variant/gpio.h

Move mainboard/gpio.h to variant/gpio.h and rename its methods to make
clear that these methods are implemented on variant level.

Change-Id: Ibcb6322067285984bad70761fce34cfcb6ed8e0f
Signed-off-by: Felix Singer 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48297
Reviewed-by: Michael Niewöhner 
Tested-by: build bot (Jenkins) 
---
M src/mainboard/clevo/kbl-u/bootblock.c
D src/mainboard/clevo/kbl-u/include/mainboard/gpio.h
A src/mainboard/clevo/kbl-u/include/variant/gpio.h
M src/mainboard/clevo/kbl-u/ramstage.c
M src/mainboard/clevo/kbl-u/variants/n13xwu/gpio.c
M src/mainboard/clevo/kbl-u/variants/n13xwu/gpio_early.c
6 files changed, 16 insertions(+), 16 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Michael Niewöhner: Looks good to me, approved

diff --git a/src/mainboard/clevo/kbl-u/bootblock.c b/src/mainboard/clevo/kbl-u/bootblock.c
index 067c9ec..8f085be 100644
--- a/src/mainboard/clevo/kbl-u/bootblock.c
+++ b/src/mainboard/clevo/kbl-u/bootblock.c
@@ -1,9 +1,9 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

#include <bootblock_common.h>
-#include <mainboard/gpio.h>
+#include <variant/gpio.h>

void bootblock_mainboard_early_init(void)
 {
-	mainboard_configure_gpios();
+	variant_configure_gpios();
 }
diff --git a/src/mainboard/clevo/kbl-u/include/mainboard/gpio.h b/src/mainboard/clevo/kbl-u/include/mainboard/gpio.h
deleted file mode 100644
index ef2a21c..0000000
--- a/src/mainboard/clevo/kbl-u/include/mainboard/gpio.h
+++ /dev/null
@@ -1,8 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-#ifndef MAINBOARD_GPIO_H
-#define MAINBOARD_GPIO_H
-
-void mainboard_configure_gpios(void);
-
-#endif
diff --git a/src/mainboard/clevo/kbl-u/include/variant/gpio.h b/src/mainboard/clevo/kbl-u/include/variant/gpio.h
new file mode 100644
index 0000000..4258325
--- /dev/null
+++ b/src/mainboard/clevo/kbl-u/include/variant/gpio.h
@@ -0,0 +1,8 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#ifndef VARIANT_GPIO_H
+#define VARIANT_GPIO_H
+
+void variant_configure_gpios(void);
+
+#endif
diff --git a/src/mainboard/clevo/kbl-u/ramstage.c b/src/mainboard/clevo/kbl-u/ramstage.c
index 9368021e..e34d500 100644
--- a/src/mainboard/clevo/kbl-u/ramstage.c
+++ b/src/mainboard/clevo/kbl-u/ramstage.c
@@ -1,7 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

#include <device/device.h>
-#include <mainboard/gpio.h>
+#include <variant/gpio.h>

/*
  * TODO:
@@ -14,7 +14,7 @@

static void init_mainboard(void *chip_info)
 {
-	mainboard_configure_gpios();
+	variant_configure_gpios();
 }

struct chip_operations mainboard_ops = {
diff --git a/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio.c b/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio.c
index e7aadcc..4b25993 100644
--- a/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio.c
+++ b/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio.c
@@ -1,7 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <mainboard/gpio.h>
 #include <soc/gpio.h>
+#include <variant/gpio.h>

static const struct pad_config gpio_table[] = {
    // RCIN#
@@ -497,7 +497,7 @@
    _PAD_CFG_STRUCT(GPP_G7,		0x44000702,	0x0)
 };

-void mainboard_configure_gpios(void)
+void variant_configure_gpios(void)
 {
    gpio_configure_pads(gpio_table, ARRAY_SIZE(gpio_table));
 }
diff --git a/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio_early.c b/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio_early.c
index 00246a4..46b9ff0 100644
--- a/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio_early.c
+++ b/src/mainboard/clevo/kbl-u/variants/n13xwu/gpio_early.c
@@ -1,7 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <mainboard/gpio.h>
 #include <soc/gpio.h>
+#include <variant/gpio.h>

static const struct pad_config early_gpio_table[] = {
    // UART2_RXD
@@ -11,7 +11,7 @@
    _PAD_CFG_STRUCT(GPP_C21,	0x44000700,	0x0)
 };

-void mainboard_configure_gpios(void)
+void variant_configure_gpios(void)
 {
    gpio_configure_pads(early_gpio_table, ARRAY_SIZE(early_gpio_table));
 }