summaryrefslogtreecommitdiff
path: root/src/cpu
diff options
context:
space:
mode:
authorVadim Bendebury <vbendeb@chromium.org>2014-05-27 18:03:38 -0700
committerMarc Jones <marc.jones@se-eng.com>2015-01-03 20:02:37 +0100
commit24a53dddb9b0b545ab9208722ff7b684114b4e91 (patch)
treebffd2066158aa26568c31578acb784a17feeef3a /src/cpu
parent0780d67ffe8ef754ace0912114e2b1e772554990 (diff)
downloadcoreboot-24a53dddb9b0b545ab9208722ff7b684114b4e91.tar.xz
ipq806x: clean up UART driver tx_byte function
The driver as it was copied from u-boot provided a function to transmit multiple characters in one invocation. This feature was not ported to coreboot, there is no need to maintain the complexity when only one character at a time is transmitted. It is also very desirable to get rid of a 1024 byte array allocated on the stack. The array was necessary to allow to convert multiple newline characters in the transmit data flow into two character sequences CRLF. Now just a single word is enough to keep one or two characters to transmit. [EDIT km: newline translation is now part of printk] BUG=chrome-os-partner:27784 TEST=verified that coreboot with the new code prints generates console output. Original-Change-Id: I73869c5f4ca87210b34811b583386554bafff1e7 Original-Signed-off-by: Vadim Bendebury <vbendeb@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/201782 Original-Reviewed-by: Stefan Reinauer <reinauer@chromium.org> Original-Reviewed-by: Trevor Bourget <tbourget@codeaurora.org> (cherry picked from commit eab3dc9d30c7e8355a2563e18ada78e4070e6151) Signed-off-by: Marc Jones <marc.jones@se-eng.com> Change-Id: I4274b8f7188bf9636906b39bcd9ec7adf0e1222e Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: http://review.coreboot.org/8011 Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones <marc.jones@se-eng.com>
Diffstat (limited to 'src/cpu')
0 files changed, 0 insertions, 0 deletions