diff options
| author | Caveh Jalali <caveh@chromium.org> | 2021-06-10 01:56:46 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-06-12 02:26:00 +0000 |
| commit | 077bb92c4a41db95d1dd1f824c19f87ac0b1d715 (patch) | |
| tree | 4d2c32cfdef977c991cc96ff15ce0e4d881e11c2 /baseboard/intelrvp/adlrvp.c | |
| parent | fd2276ad5b6403338df5dd096195cb5ceadaeae6 (diff) | |
| download | chrome-ec-077bb92c4a41db95d1dd1f824c19f87ac0b1d715.tar.gz | |
bb_retimer: Return status from bb_retimer_power_handle()
This allows the implementation of bb_retimer_power_handle() to return a
status value indicating whether the request was successful. The default
implementation simply controls a GPIO and is expected to succeed
unconditionally. More complex implementations may run into failure cases
that leave the BB unreachable. When this happens, device initialization
returns an error so the caller can take mitigating action.
USB MUX operations tend to be called from timing sensitive code paths in
the TCPM, so careful error handling helps avoid cascading problems like
PD negotiation failures.
BRANCH=none
BUG=b:181743576,b:188826559
TEST=buildall passes, PD still works on brya
Change-Id: If79078be26e47d758e2cd6cc385ff2b34fecff63
Signed-off-by: Caveh Jalali <caveh@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2954198
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Boris Mittelberg <bmbm@google.com>
Diffstat (limited to 'baseboard/intelrvp/adlrvp.c')
| -rw-r--r-- | baseboard/intelrvp/adlrvp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/baseboard/intelrvp/adlrvp.c b/baseboard/intelrvp/adlrvp.c index fcb590b16e..d992154c49 100644 --- a/baseboard/intelrvp/adlrvp.c +++ b/baseboard/intelrvp/adlrvp.c @@ -247,7 +247,7 @@ void board_overcurrent_event(int port, int is_overcurrented) ioex_set_level(oc_signal, is_overcurrented ? 0 : 1); } -__override void bb_retimer_power_handle(const struct usb_mux *me, int on_off) +__override int bb_retimer_power_handle(const struct usb_mux *me, int on_off) { /* Handle retimer's power domain.*/ if (on_off) { @@ -273,6 +273,7 @@ __override void bb_retimer_power_handle(const struct usb_mux *me, int on_off) msleep(1); ioex_set_level(bb_controls[me->usb_port].usb_ls_en_gpio, 0); } + return EC_SUCCESS; } static void board_connect_c0_sbu_deferred(void) |