diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2020-03-18 19:03:58 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-03-31 23:24:38 +0000 |
| commit | ddbada80f6561cf721fc967ffab965e77b3ae5c3 (patch) | |
| tree | 8f30a78ba10ef8cc5847f167b7319675f1f1c580 /common/console.c | |
| parent | a795c626a156a92813e7992bc14c764b1ae1f994 (diff) | |
| download | chrome-ec-ddbada80f6561cf721fc967ffab965e77b3ae5c3.tar.gz | |
Prepare for transitioning to packet mode console
A very few changes are needed to support the packet mode:
- provide functions to report how much room is left in USB/UART
transmit buffers;
- compile out cprintf/cprints/cputs just in case to be able to catch
cases where util_precompile.py fails to convert them for whatever
reason;
- do not add CR to every LF, this messes up packet transmissions, and
the terminal is doing the right thing anyways
- there is a problem with the USB channel in packet mode: the device
reboots as soon as an attempt to send something to the host is
undertaken. The problem can be rectified by disabling the deferred
function path in the Cr50 console USB channel. A bug was open to
track it down, but in packet mode using deferred function in this
path is less critical, as the amount of sent data always is at
least as much as the packet header size
BUG=b:149964350, b:152116489
TEST=with the rest of the patches applied packet mode console works
fine. When packet mode is disabled the conventional mode console
works fine.
Change-Id: Ib010cede36adc87cf80f49e5d76ec9e274d9e608
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2114238
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Namyoon Woo <namyoon@chromium.org>
Diffstat (limited to 'common/console.c')
| -rw-r--r-- | common/console.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/common/console.c b/common/console.c index 8f42dffbf5..9ee3dc5b13 100644 --- a/common/console.c +++ b/common/console.c @@ -279,10 +279,7 @@ static void console_init(void) static int console_putc(int c) { - int rv1 = uart_putc(c); - int rv2 = usb_putc(c); - - return rv1 == EC_SUCCESS ? rv2 : rv1; + return ccprintf("%c", c); } #ifndef CONFIG_EXPERIMENTAL_CONSOLE |