diff options
| author | Randall Spangler <rspangler@chromium.org> | 2013-12-17 13:43:17 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-12-19 00:12:28 +0000 |
| commit | 6ab8e91658f1efc894b648cc0748af8d804915e4 (patch) | |
| tree | c33da7914793452ecb37084e505230171d4eff9e /common/uart_buffering.c | |
| parent | e5935f17d1798a1f19c6003e57f140446774484f (diff) | |
| download | chrome-ec-6ab8e91658f1efc894b648cc0748af8d804915e4.tar.gz | |
cleanup: Remove checkpatch warnings
This make minor syntactic changes and renames some camel-cased symbols
to keep checkpatch from complaining. The goal is to reduce the
temptation to use 'repo upload --no-verify'.
This is a big furball of find/replace, but no functional changes.
BUG=chromium:322144
BRANCH=none
TEST=build all boards; pass unit tests
Change-Id: I0269b7dd95836ef9a6e33f88c003ab0f24f842a0
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/180495
Diffstat (limited to 'common/uart_buffering.c')
| -rw-r--r-- | common/uart_buffering.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/common/uart_buffering.c b/common/uart_buffering.c index d49425229e..7d0b51ccf4 100644 --- a/common/uart_buffering.c +++ b/common/uart_buffering.c @@ -95,7 +95,7 @@ void uart_process_output(void) return; /* If DMA is still busy, nothing to do. */ - if(!uart_tx_dma_ready()) + if (!uart_tx_dma_ready()) return; /* If a previous DMA transfer completed, free up the buffer it used */ @@ -106,7 +106,7 @@ void uart_process_output(void) } /* Disable DMA-done interrupt if nothing to send */ - if(head == tx_buf_tail) { + if (head == tx_buf_tail) { uart_tx_stop(); return; } @@ -301,7 +301,7 @@ void uart_flush_output(void) int uart_getc(void) { /* Look for a non-flow-control character */ - while(rx_buf_tail != rx_buf_head) { + while (rx_buf_tail != rx_buf_head) { int c = rx_buf[rx_buf_tail]; rx_buf_tail = RX_BUF_NEXT(rx_buf_tail); |