mb/google,intel: Add ChromeOS GPIOs to onboard.h

Change-Id: Ia473596e3c9a75587cd1288c8816bfef66bef82e
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59000
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>

src/mainboard/google/auron/chromeos.c

@@ -7,8 +7,7 @@
 #include <soc/chromeos.h>
 #include <southbridge/intel/lynxpoint/lp_gpio.h>
 
-/* SPI Write protect is GPIO 16 */
-#define CROS_WP_GPIO	58
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {

src/mainboard/google/auron/onboard.h

@@ -0,0 +1,9 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#ifndef AURON_ONBOARD_H
+#define AURON_ONBOARD_H
+
+/* SPI Write protect is GPIO 58 */
+#define CROS_WP_GPIO	58
+
+#endif

src/mainboard/google/butterfly/chromeos.c

@@ -11,9 +11,8 @@
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
 #include "ec.h"
+#include "onboard.h"
 
-#define WP_GPIO		6
-#define DEVMODE_GPIO	54
 #define FORCE_RECOVERY_MODE	0
 
 void fill_lb_gpios(struct lb_gpios *gpios)

src/mainboard/google/butterfly/onboard.h

@@ -16,4 +16,8 @@
 
 /* 0x00: White LINK LED and Amber ACTIVE LED */
 #define BUTTERFLY_NIC_LED_MODE		0x00
+
+/* SPI write protect, active low */
+#define WP_GPIO		6
+
 #endif

src/mainboard/google/link/chromeos.c

@@ -6,6 +6,7 @@
 #include <southbridge/intel/common/gpio.h>
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {
@@ -26,12 +27,12 @@ void fill_lb_gpios(struct lb_gpios *gpios)
 
 int get_write_protect_state(void)
 {
-	return get_gpio(57);
+	return get_gpio(GPIO_SPI_WP);
 }
 
 static const struct cros_gpio cros_gpios[] = {
-	CROS_GPIO_REC_AL(9, CROS_GPIO_DEVICE_NAME),
-	CROS_GPIO_WP_AH(57, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_REC_AL(GPIO_REC_MODE, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_WP_AH(GPIO_SPI_WP, CROS_GPIO_DEVICE_NAME),
 };
 
 void mainboard_chromeos_acpi_generate(void)

src/mainboard/google/link/onboard.h

@@ -16,4 +16,8 @@
 #define BOARD_TOUCHSCREEN_I2C_ADDR	0x4a
 #define BOARD_TOUCHSCREEN_IRQ		22
 
+#define GPIO_REC_MODE	9
+
+#define GPIO_SPI_WP	57
+
 #endif

src/mainboard/google/parrot/chromeos.c

@@ -8,10 +8,10 @@
 
 #include <southbridge/intel/bd82x6x/pch.h>
 #include <southbridge/intel/common/gpio.h>
-#include <ec/compal/ene932/ec.h>
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
-#include "ec.h"
+
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {
@@ -20,7 +20,7 @@ void fill_lb_gpios(struct lb_gpios *gpios)
 
 	struct lb_gpio chromeos_gpios[] = {
 		/* Lid switch GPIO active high (open). */
-		{15, ACTIVE_HIGH, get_lid_switch(), "lid"},
+		{GPIO_LID, ACTIVE_HIGH, get_lid_switch(), "lid"},
 
 		/* Power Button */
 		{101, ACTIVE_LOW, (gen_pmcon_1 >> 9) & 1, "power"},
@@ -34,32 +34,27 @@ void fill_lb_gpios(struct lb_gpios *gpios)
 
 int get_lid_switch(void)
 {
-	return get_gpio(15);
+	return get_gpio(GPIO_LID);
 }
 
 int get_write_protect_state(void)
 {
-	return !get_gpio(70);
+	return !get_gpio(GPIO_SPI_WP);
 }
 
 int get_recovery_mode_switch(void)
 {
-	u8 gpio = !get_gpio(68);
-	/* GPIO68, active low. For Servo support
-	 * Treat as active high and let the caller invert if needed. */
-	printk(BIOS_DEBUG, "REC MODE GPIO 68: %x\n", gpio);
-
-	return gpio;
+	return !get_gpio(GPIO_REC_MODE);
 }
 
 static int parrot_ec_running_ro(void)
 {
-	return !get_gpio(68);
+	return get_recovery_mode_switch();
 }
 
 static const struct cros_gpio cros_gpios[] = {
 	CROS_GPIO_REC_AH(CROS_GPIO_VIRTUAL, CROS_GPIO_DEVICE_NAME),
-	CROS_GPIO_WP_AL(70, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_WP_AL(GPIO_SPI_WP, CROS_GPIO_DEVICE_NAME),
 };
 
 void mainboard_chromeos_acpi_generate(void)

src/mainboard/google/parrot/onboard.h

@@ -9,4 +9,13 @@
 #define BOARD_TRACKPAD_IRQ_PVT      20
 #define BOARD_TRACKPAD_WAKE_GPIO    0x1c
 
+#define GPIO_LID	15
+
+/* GPIO68, active low. For Servo support
+ * Treat as active high and let the caller invert if needed. */
+#define GPIO_REC_MODE	68
+
+
+#define GPIO_SPI_WP	70
+
 #endif

src/mainboard/google/slippy/chromeos.c

@@ -6,6 +6,7 @@
 #include <southbridge/intel/common/gpio.h>
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {
@@ -19,12 +20,12 @@ void fill_lb_gpios(struct lb_gpios *gpios)
 
 int get_write_protect_state(void)
 {
-	return get_gpio(58);
+	return get_gpio(GPIO_SPI_WP);
 }
 
 static const struct cros_gpio cros_gpios[] = {
 	CROS_GPIO_REC_AL(CROS_GPIO_VIRTUAL, CROS_GPIO_DEVICE_NAME),
-	CROS_GPIO_WP_AH(58, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_WP_AH(GPIO_SPI_WP, CROS_GPIO_DEVICE_NAME),
 };
 
 void mainboard_chromeos_acpi_generate(void)

src/mainboard/google/slippy/onboard.h

@@ -23,4 +23,7 @@
 #define PEPPY_BOARD_VERSION_PROTO	0
 #define PEPPY_BOARD_VERSION_EVT		1
 
+/* Write protect is active high */
+#define GPIO_SPI_WP	58
+
 #endif

src/mainboard/google/stout/chromeos.c

@@ -12,6 +12,7 @@
 #include <vendorcode/google/chromeos/chromeos.h>
 #include "ec.h"
 #include <ec/quanta/it8518/ec.h>
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {
@@ -35,7 +36,7 @@ void fill_lb_gpios(struct lb_gpios *gpios)
 
 int get_write_protect_state(void)
 {
-	return !get_gpio(7);
+	return !get_gpio(GPIO_SPI_WP);
 }
 
 int get_lid_switch(void)
@@ -77,7 +78,7 @@ int get_recovery_mode_switch(void)
 static const struct cros_gpio cros_gpios[] = {
 	CROS_GPIO_REC_AH(CROS_GPIO_VIRTUAL, CROS_GPIO_DEVICE_NAME),
 	CROS_GPIO_REC_AH(CROS_GPIO_VIRTUAL, CROS_GPIO_DEVICE_NAME),
-	CROS_GPIO_WP_AL(7, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_WP_AL(GPIO_SPI_WP, CROS_GPIO_DEVICE_NAME),
 };
 
 void mainboard_chromeos_acpi_generate(void)

src/mainboard/google/stout/onboard.h

@@ -11,4 +11,7 @@
 #define XHCI_PREBOOT  0  // No PreOS boot support
 #define XHCI_STREAMS  1  // Sure, lets have streams
 
+/* Write protect is active low */
+#define GPIO_SPI_WP	7
+
 #endif

src/mainboard/intel/baskingridge/chromeos.c

@@ -7,12 +7,13 @@
 #include <southbridge/intel/common/gpio.h>
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {
 	struct lb_gpio chromeos_gpios[] = {
 		/* Recovery: GPIO69 - SV_DETECT - J8E3 (silkscreen: J8E2) */
-		{69, ACTIVE_HIGH, get_recovery_mode_switch(), "presence"},
+		{GPIO_REC_MODE, ACTIVE_HIGH, get_recovery_mode_switch(), "presence"},
 
 		/* Hard code the lid switch GPIO to open. */
 		{-1, ACTIVE_HIGH, 1, "lid"},
@@ -32,18 +33,18 @@ int get_recovery_mode_switch(void)
 	 * Recovery: GPIO69, Connected to J8E3, however the silkscreen says
 	 * J8E2. The jump is active high.
 	 */
-	return get_gpio(69);
+	return get_gpio(GPIO_REC_MODE);
 }
 
 int get_write_protect_state(void)
 {
 	/* Write protect is active low, so invert it here */
-	return !get_gpio(22);
+	return !get_gpio(GPIO_SPI_WP);
 }
 
 static const struct cros_gpio cros_gpios[] = {
-	CROS_GPIO_REC_AH(69, CROS_GPIO_DEVICE_NAME),
-	CROS_GPIO_WP_AL(22, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_REC_AH(GPIO_REC_MODE, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_WP_AL(GPIO_SPI_WP, CROS_GPIO_DEVICE_NAME),
 };
 
 void mainboard_chromeos_acpi_generate(void)

src/mainboard/intel/baskingridge/onboard.h

@@ -0,0 +1,12 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#ifndef BASKINGRIDGE_ONBOARD_H
+#define BASKINGRIDGE_ONBOARD_H
+
+/* Recovery: GPIO69, active high - SV_DETECT - J8E3 (silkscreen: J8E2) */
+#define GPIO_REC_MODE	69
+
+/* Write protect is active low */
+#define GPIO_SPI_WP	22
+
+#endif

src/mainboard/intel/emeraldlake2/chromeos.c

@@ -7,12 +7,13 @@
 #include <southbridge/intel/common/gpio.h>
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {
 	struct lb_gpio chromeos_gpios[] = {
 		/* Recovery: GPIO22 */
-		{22, ACTIVE_LOW, !get_recovery_mode_switch(), "presence"},
+		{GPIO_REC_MODE, ACTIVE_LOW, !get_recovery_mode_switch(), "presence"},
 
 		/* Hard code the lid switch GPIO to open. */
 		{-1, ACTIVE_HIGH, 1, "lid"},
@@ -29,18 +30,18 @@ void fill_lb_gpios(struct lb_gpios *gpios)
 int get_recovery_mode_switch(void)
 {
 	/* Recovery: GPIO22, active low */
-	return !get_gpio(22);
+	return !get_gpio(GPIO_REC_MODE);
 }
 
 int get_write_protect_state(void)
 {
 	/* Write protect is active low, so invert it here */
-	return !get_gpio(48);
+	return !get_gpio(GPIO_SPI_WP);
 }
 
 static const struct cros_gpio cros_gpios[] = {
-	CROS_GPIO_REC_AL(22, CROS_GPIO_DEVICE_NAME),
-	CROS_GPIO_WP_AL(48, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_REC_AL(GPIO_REC_MODE, CROS_GPIO_DEVICE_NAME),
+	CROS_GPIO_WP_AL(GPIO_SPI_WP, CROS_GPIO_DEVICE_NAME),
 };
 
 void mainboard_chromeos_acpi_generate(void)

src/mainboard/intel/emeraldlake2/onboard.h

@@ -0,0 +1,12 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#ifndef EMERALDLAKE2_ONBOARD_H
+#define EMERALDLAKE2_ONBOARD_H
+
+/* Recovery: GPIO22, active low */
+#define GPIO_REC_MODE	22
+
+/* Write protect is active low */
+#define GPIO_SPI_WP	48
+
+#endif

src/mainboard/intel/strago/chromeos.c

@@ -6,10 +6,7 @@
 #include <types.h>
 #include <vendorcode/google/chromeos/chromeos.h>
 
-#define WP_GPIO	GP_E_22
-
-#define ACTIVE_LOW	0
-#define ACTIVE_HIGH	1
+#include "onboard.h"
 
 void fill_lb_gpios(struct lb_gpios *gpios)
 {

src/mainboard/intel/strago/onboard.h

@@ -29,6 +29,9 @@
 #define JACK_DETECT_GPIO_INDEX		95
 /* SCI: Gpio index in N bank */
 #define BOARD_SCI_GPIO_INDEX		15
+
+#define WP_GPIO				GP_E_22
+
 /* Trackpad: Gpio index in N bank */
 #define BOARD_TRACKPAD_GPIO_INDEX	18