summaryrefslogtreecommitdiff
path: root/board/hoho
diff options
context:
space:
mode:
authorTodd Broch <tbroch@chromium.org>2014-10-13 15:45:50 -0700
committerchrome-internal-fetch <chrome-internal-fetch@google.com>2014-10-30 21:48:06 +0000
commita3913d99d7a40062144b96d505bba56a62986faa (patch)
tree35b6531d59c0889a36d77253faa30bfa1ea86b4e /board/hoho
parentd4f3279db9d31159624449828362e45062bd7703 (diff)
downloadchrome-ec-a3913d99d7a40062144b96d505bba56a62986faa.tar.gz
pd: dingdong/hoho: HPD over USB PD.
HPD needs to be transported of USB PD as both SBU lines are consumed for differential AUX signalling. This CL does the following: 1. Enables GPIO DP_HPD as interrupt 2. Sends debounced HPD across CC via the SVDM DP status message BRANCH=none BUG=chrome-os-partner:31192,chrome-os-partner:31193 TEST=manual, From servo w/ GPIO attached to HPD drove the following transactions after inserting with HPD low initially: # e1: hpd_high # e2: hpd_low # non-registered glitch # e3: hpd_high followed by hpd_low # e4: hpd high # non-registered glitch # e5: hpd_irq # e6: hpd_irq # e7: hpd_irq # e8: hpd_low followed by hpd_high From fruitpie console (marked up to show result of above) ----> enter-mode, dp status, dp config [6.774108 DONE] SVDM/1 [4] ff018144 [6.777467 DONE] SVDM/2 [16] ff018150 00000002 [6.780637 DONE] SVDM/1 [17] ff018051 ----> attentions start arriving ----> e1 [18.966741 DONE] SVDM/2 [6] ff018106 0000008a ----> e2 [33.724367 DONE] SVDM/2 [6] ff018106 0000000a ----> e3 [64.550398 DONE] SVDM/2 [6] ff018106 0000008a ----> e3 [64.752452 DONE] SVDM/2 [6] ff018106 0000000a ----> e4 [74.247127 DONE] SVDM/2 [6] ff018106 0000008a ----> e5 [88.906254 DONE] SVDM/2 [6] ff018106 0000010a ----> e6 [100.938738 DONE] SVDM/2 [6] ff018106 0000010a ----> e7 [123.693414 DONE] SVDM/2 [6] ff018106 0000010a ----> e8 [130.050074 DONE] SVDM/2 [6] ff018106 0000000a ----> e8 [130.254087 DONE] SVDM/2 [6] ff018106 0000008a Change-Id: I976c268467ece84cedab7ba4943fb59d1e48c113 Signed-off-by: Todd Broch <tbroch@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/223262 Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'board/hoho')
-rw-r--r--board/hoho/board.c81
-rw-r--r--board/hoho/gpio.inc2
-rw-r--r--board/hoho/usb_pd_policy.c13
3 files changed, 92 insertions, 4 deletions
diff --git a/board/hoho/board.c b/board/hoho/board.c
index c521d2c6f8..f7ce18da4b 100644
--- a/board/hoho/board.c
+++ b/board/hoho/board.c
@@ -7,7 +7,6 @@
#include "adc.h"
#include "adc_chip.h"
#include "common.h"
-#include "console.h"
#include "gpio.h"
#include "hooks.h"
#include "i2c.h"
@@ -16,10 +15,80 @@
#include "usb.h"
#include "usb_bb.h"
#include "usb_pd.h"
+#include "timer.h"
#include "util.h"
+static volatile uint64_t hpd_prev_ts;
+static volatile int hpd_prev_level;
+
+void hpd_event(enum gpio_signal signal);
#include "gpio_list.h"
+/**
+ * Hotplug detect deferred task
+ *
+ * Called after level change on hpd GPIO to evaluate (and debounce) what event
+ * has occurred. There are 3 events that occur on HPD:
+ * 1. low : downstream display sink is deattached
+ * 2. high : downstream display sink is attached
+ * 3. irq : downstream display sink signalling an interrupt.
+ *
+ * The debounce times for these various events are:
+ * 100MSEC : min pulse width of level value.
+ * 2MSEC : min pulse width of IRQ low pulse. Max is level debounce min.
+ *
+ * lvl(n-2) lvl(n-1) lvl prev_delta now_delta event
+ * ----------------------------------------------------
+ * 1 0 1 <2ms n/a low glitch (ignore)
+ * 1 0 1 >2ms <100ms irq
+ * x 0 1 n/a >100ms high
+ * 0 1 0 <100ms n/a high glitch (ignore)
+ * x 1 0 n/a >100ms low
+ */
+
+void hpd_irq_deferred(void)
+{
+ pd_send_hpd(0, hpd_irq);
+}
+DECLARE_DEFERRED(hpd_irq_deferred);
+
+void hpd_lvl_deferred(void)
+{
+ int level = gpio_get_level(GPIO_DP_HPD);
+
+ if (level != hpd_prev_level)
+ /* It's a glitch while in deferred or canceled action */
+ return;
+
+ pd_send_hpd(0, (level) ? hpd_high : hpd_low);
+}
+DECLARE_DEFERRED(hpd_lvl_deferred);
+
+void hpd_event(enum gpio_signal signal)
+{
+ timestamp_t now = get_time();
+ int level = gpio_get_level(signal);
+ uint64_t cur_delta = now.val - hpd_prev_ts;
+
+ /* store current time */
+ hpd_prev_ts = now.val;
+
+ /* All previous hpd level events need to be re-triggered */
+ hook_call_deferred(hpd_lvl_deferred, -1);
+
+ /* It's a glitch. Previous time moves but level is the same. */
+ if (cur_delta < HPD_DEBOUNCE_IRQ)
+ return;
+
+ if ((!hpd_prev_level && level) && (cur_delta < HPD_DEBOUNCE_LVL))
+ /* It's an irq */
+ hook_call_deferred(hpd_irq_deferred, 0);
+ else if (cur_delta >= HPD_DEBOUNCE_LVL)
+ hook_call_deferred(hpd_lvl_deferred, HPD_DEBOUNCE_LVL);
+
+ hpd_prev_level = level;
+}
+
/* Initialize board. */
void board_config_pre_init(void)
{
@@ -60,14 +129,22 @@ static void board_init_spi2(void)
/* Enable clocks to SPI2 module */
STM32_RCC_APB1ENR |= STM32_RCC_PB1_SPI2;
}
+#endif /* CONFIG_SPI_FLASH */
/* Initialize board. */
static void board_init(void)
{
+ timestamp_t now;
+#ifdef CONFIG_SPI_FLASH
board_init_spi2();
+#endif
+ now = get_time();
+ hpd_prev_level = gpio_get_level(GPIO_DP_HPD);
+ hpd_prev_ts = now.val;
+ gpio_enable_interrupt(GPIO_DP_HPD);
}
+
DECLARE_HOOK(HOOK_INIT, board_init, HOOK_PRIO_DEFAULT);
-#endif /* CONFIG_SPI_FLASH */
/* ADC channels */
const struct adc_t adc_channels[] = {
diff --git a/board/hoho/gpio.inc b/board/hoho/gpio.inc
index 204a1ac332..0aad148307 100644
--- a/board/hoho/gpio.inc
+++ b/board/hoho/gpio.inc
@@ -4,7 +4,7 @@
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
*/
-GPIO(DP_HPD, A, 0, GPIO_INPUT, NULL)
+GPIO(DP_HPD, A, 0, GPIO_INT_BOTH, hpd_event)
GPIO(USB_C_CC1_PD, A, 1, GPIO_ANALOG, NULL)
GPIO(MCDP_RESET_L, A, 3, GPIO_OUT_HIGH, NULL)
GPIO(PD_DAC_REF, A, 4, GPIO_ANALOG, NULL)
diff --git a/board/hoho/usb_pd_policy.c b/board/hoho/usb_pd_policy.c
index b747766aa9..f52416b19d 100644
--- a/board/hoho/usb_pd_policy.c
+++ b/board/hoho/usb_pd_policy.c
@@ -12,8 +12,8 @@
#include "registers.h"
#include "task.h"
#include "timer.h"
-#include "util.h"
#include "usb_pd.h"
+#include "util.h"
#include "version.h"
#define CPRINTF(format, args...) cprintf(CC_USBPD, format, ## args)
@@ -32,6 +32,9 @@ const int pd_snk_pdo_cnt = ARRAY_SIZE(pd_snk_pdo);
/* Desired voltage requested as a sink (in millivolts) */
static unsigned select_mv = 5000;
+/* Whether alternate mode has been entered or not */
+static int alt_mode;
+
int pd_choose_voltage(int cnt, uint32_t *src_caps, uint32_t *rdo,
uint32_t *curr_limit, uint32_t *supply_voltage)
{
@@ -182,12 +185,20 @@ static int svdm_enter_mode(int port, uint32_t *payload)
if ((PD_VDO_VID(payload[0]) != USB_SID_DISPLAYPORT) ||
(PD_VDO_OPOS(payload[0]) != OPOS))
return 0; /* will generate a NAK */
+
+ alt_mode = OPOS;
return 1;
}
+int pd_alt_mode(int port)
+{
+ return alt_mode;
+}
+
static int svdm_exit_mode(int port, uint32_t *payload)
{
gpio_set_level(GPIO_PD_SBU_ENABLE, 0);
+ alt_mode = 0;
return 1; /* Must return ACK */
}