waddledoo: Add JSL support

waddledoo is using the Intel Jasperlake chipset.  This commit enables
support for that chipset as well as adding the other related power
sequencing signals.  JSL uses the same chipset driver as ICL, but
waddledoo doesn't have a couple of the power good pins that the driver
assumes.  Therefore, waddledoo overrides these power good signals using
other logic.

BUG=b:147257114,b:146172102
BRANCH=None
TEST=`make -j BOARD=waddledoo`

Change-Id: Ib4baa7ff38a4f59f751ecb16c22522eda71b6f4f
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1987837
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Auto-Submit: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
Commit-Queue: Diana Z <dzigterman@chromium.org>

4 files changed, 30 insertions, 14 deletions

diff --git a/board/waddledoo/board.c b/board/waddledoo/board.c
index a302d4d529..8c5a865035 100644
--- a/board/waddledoo/board.c
+++ b/board/waddledoo/board.c
@@ -9,10 +9,21 @@
 #include "button.h"
 #include "common.h"
 #include "compile_time_macros.h"
+#include "extpower.h"
 #include "gpio.h"
 #include "i2c.h"
 #include "lid_switch.h"
+#include "power.h"
 #include "power_button.h"
 #include "switch.h"
 
 #include "gpio_list.h"
+
+int extpower_is_present(void)
+{
+	/*
+	 * TODO(b:146651593) We can likely use the charger IC to determine VBUS
+	 * presence.
+	 */
+	return 1;
+}
diff --git a/board/waddledoo/board.h b/board/waddledoo/board.h
index 328488ddb5..f7a595d545 100644
--- a/board/waddledoo/board.h
+++ b/board/waddledoo/board.h
@@ -11,19 +11,6 @@
 #define VARIANT_DEDEDE_EC_NPCX796FC
 #include "baseboard.h"
 
-/*
- * Remapping of schematic GPIO names to common GPIO names expected (hardcoded)
- * in the EC code base.
- */
-#define GPIO_EC_INT_L       GPIO_EC_AP_MKBP_INT_L
-#define GPIO_ENTERING_RW    GPIO_EC_ENTERING_RW
-#define GPIO_PCH_PWRBTN_L   GPIO_EC_AP_PWR_BTN_ODL
-#define GPIO_PCH_WAKE_L     GPIO_EC_AP_WAKE_ODL
-#define GPIO_POWER_BUTTON_L GPIO_H1_EC_PWR_BTN_ODL
-#define GPIO_VOLUME_UP_L    GPIO_VOLUP_BTN_ODL
-#define GPIO_VOLUME_DOWN_L  GPIO_VOLDN_BTN_ODL
-#define GPIO_WP             GPIO_EC_WP_OD
-
 /* I2C configuration */
 #define I2C_PORT_EEPROM     NPCX_I2C_PORT7_0
 #define I2C_PORT_BATTERY    NPCX_I2C_PORT5_0
diff --git a/board/waddledoo/ec.tasklist b/board/waddledoo/ec.tasklist
index d2983d770d..57bcccc28f 100644
--- a/board/waddledoo/ec.tasklist
+++ b/board/waddledoo/ec.tasklist
@@ -9,6 +9,7 @@
 
 #define CONFIG_TASK_LIST \
 	TASK_ALWAYS(HOOKS, hook_task, NULL, VENTI_TASK_STACK_SIZE) \
+	TASK_NOTEST(CHIPSET, chipset_task, NULL, VENTI_TASK_STACK_SIZE) \
 	TASK_NOTEST(KEYPROTO, keyboard_protocol_task, NULL, TASK_STACK_SIZE) \
 	TASK_ALWAYS(CONSOLE, console_task, NULL, VENTI_TASK_STACK_SIZE) \
 	TASK_ALWAYS(HOSTCMD, host_command_task, NULL, VENTI_TASK_STACK_SIZE) \
diff --git a/board/waddledoo/gpio.inc b/board/waddledoo/gpio.inc
index b4f8d994b8..68d27b4025 100644
--- a/board/waddledoo/gpio.inc
+++ b/board/waddledoo/gpio.inc
@@ -10,6 +10,17 @@
  * Note: Those with interrupt handlers must be declared first.
  */
 
+GPIO_INT(SLP_S0_L,		 PIN(D, 5), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(SLP_S3_L,		 PIN(A, 5), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(SLP_S4_L,		 PIN(D, 4), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(SLP_SUS_L,		 PIN(D, 7), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(RSMRST_PWRGD_L,	 PIN(C, 6), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(VCCIN_AUX_VID1,	 PIN(C, 7), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(CPU_C10_GATE_L,	 PIN(6, 7), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(VCCIN_AUX_VID0,	 PIN(F, 4), GPIO_INT_BOTH | GPIO_SEL_1P8V, power_signal_interrupt)
+GPIO_INT(PG_VCCIO_EXT_OD,	 PIN(B, 0), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(PG_PP5000_U_OD,	 PIN(E, 2), GPIO_INT_BOTH, power_signal_interrupt)
+GPIO_INT(PG_DRAM_OD,		 PIN(E, 4), GPIO_INT_BOTH, power_signal_interrupt)
 GPIO_INT(H1_EC_PWR_BTN_ODL,	 PIN(0, 1), GPIO_INT_BOTH | GPIO_PULL_UP, power_button_interrupt)
 GPIO_INT(VOLDN_BTN_ODL,		 PIN(4, 0), GPIO_INT_BOTH | GPIO_PULL_UP | GPIO_SEL_1P8V, button_interrupt)
 /* TODO(b:147257497) Fix this and EN_PP5000_U before board rev 0. */
@@ -59,7 +70,6 @@ GPIO(EC_PROCHOT_ODL,	     PIN(F, 1), GPIO_ODR_HIGH | GPIO_PULL_UP | GPIO_SEL_1P8
 GPIO(EC_ENTERING_RW,	     PIN(E, 3), GPIO_OUT_LOW)
 GPIO(ALL_SYS_PWRGD,	     PIN(A, 0), GPIO_OUT_LOW)
 GPIO(SYS_RST_ODL,	     PIN(C, 5), GPIO_ODR_HIGH)
-GPIO(RSMRST_PWRGD_L,	     PIN(C, 6), GPIO_OUT_HIGH)
 GPIO(CCD_MODE_ODL,	     PIN(E, 5), GPIO_INPUT)
 
 GPIO(USB_C0_RST_ODL,	     PIN(9, 7), GPIO_OUT_HIGH) /* currently unused */
@@ -67,6 +77,13 @@ GPIO(EC_AP_USB_C1_HDMI_HPD,  PIN(9, 6), GPIO_OUT_LOW)
 GPIO(EC_AP_USB_C0_HPD,	     PIN(9, 3), GPIO_OUT_LOW)
 GPIO(HDMI_SEL_L,	     PIN(7, 2), GPIO_OUT_HIGH)
 
+/*
+ * Waddledoo doesn't have these physical pins coming to the EC but uses other
+ * logic.
+ */
+UNIMPLEMENTED(PG_EC_DSW_PWROK)
+UNIMPLEMENTED(PG_EC_ALL_SYS_PWRGD)
+
 /* Alternate Functions */
 /* ADC */
 ALTERNATE(PIN_MASK(F, BIT(0)), 0, MODULE_ADC, 0) /* ADC9 */