coreboot/payloads/libpayload
Yogesh Lal 61528884ad libpayload: ipq808x: stale interrupt shall not be cleared unconditionally
The serial driver hangs in cases when FIFO has more than single word to be
processed. Easiest way to reproduce is to paste a string of greater than 4
characters in cli.

Clearing the RXSTALE interrupt without draining all the characters from FIFO
leads to the issue as the driver is dependent on msm_boot_uart_dm_read
function to reinitialize for next transfer.

Logically the driver is organized in such a manner that next transfer never
gets initiated till rx_data_read < total_rx_data. Clearing the RXSTALE without
consideration of total number of characters (or words) unprocessed makes the
msm_boot_uart_dm_read to return on the first if conditional. Thus the driver is
stuck forever.

A quick fix is to avoid clearing the stale interrupt. Reset is handled whenever
a new transfer is initialized in msm_boot_uart_dm_init_rx_transfer.

BUG=chrome-os-partner:29542
TEST=manual
	-Paste a string greater than 4 characters in cli.

Change-Id: I016afb01a77cd14764f0176f6bf144fb29796c2f
Signed-off-by: Yogesh Lal <ylal@codeaurora.org>
Reviewed-on: https://chromium-review.googlesource.com/209512
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Commit-Queue: Vadim Bendebury <vbendeb@chromium.org>
Tested-by: Vadim Bendebury <vbendeb@chromium.org>
2014-07-26 02:43:49 +00:00
..
arch libpayload arm64: Add gdb stub for arm64 2014-06-20 21:41:05 +00:00
bin libpayload: Add support for arm64 in libpayload 2014-06-07 01:28:57 +00:00
configs libpayload: Renormalize config.veyron 2014-07-17 06:52:04 +00:00
crypto libpayload: Change CONFIG_* to CONFIG_LP_* in the kconfig. 2013-08-14 17:05:33 -07:00
curses libpayload: Change CONFIG_* to CONFIG_LP_* in the kconfig. 2013-08-14 17:05:33 -07:00
drivers libpayload: ipq808x: stale interrupt shall not be cleared unconditionally 2014-07-26 02:43:49 +00:00
gdb libpayload: Let GDB stub read/write memory with aligned MMIO words 2014-07-22 01:24:00 +00:00
include libpayload: Expand setbits_le32() and fix readl() const-ness 2014-07-19 01:08:25 +00:00
libc libpayload: improve us timer accuracy 2014-07-11 01:55:41 +00:00
libcbfs libpayload: Fix pointer related casts 2014-05-14 00:25:48 +00:00
liblzma libpayload: Make lzma truncation non-fatal. 2013-08-19 19:04:35 -07:00
libpci libpayload: minor cleanups 2012-04-04 00:40:31 +02:00
sample fix compilation of hello.elf example payload. 2010-08-28 23:23:47 +00:00
tests libpayload: Change CONFIG_X86_SERIAL_CONSOLE to CONFIG_8250_SERIAL_CONSOLE 2013-09-27 14:05:53 +00:00
util libpayload: Add support for arm64 in libpayload 2014-06-07 01:28:57 +00:00
Config.in libpayload: Support board veyron 2014-07-02 00:47:36 +00:00
Doxyfile Run doxygen -u on doxygen configuration files 2010-06-28 10:40:38 +00:00
LICENSE_GPL libpayload: Introduce new Kconfig to explicitly allow GPL code 2014-06-13 20:37:29 +00:00
LICENSES libpayload: Introduce new Kconfig to explicitly allow GPL code 2014-06-13 20:37:29 +00:00
Makefile libpayload: Add support for arm64 in libpayload 2014-06-07 01:28:57 +00:00
Makefile.inc libpayload: Add remote GDB support 2014-06-11 03:33:56 +00:00
README libpayload, superiotool: README: Prepend coreboot/ to path of change directory line 2013-04-04 17:22:15 +02:00

-------------------------------------------------------------------------------
libpayload README
-------------------------------------------------------------------------------

libpayload is a minimal library to support standalone payloads
that can be booted with firmware like coreboot. It handles the setup
code, and provides common C library symbols such as malloc() and printf().

Note: This is _not_ a standard library for use with an operating system,
rather it's only useful for coreboot payload development!
See http://coreboot.org for details on coreboot.


Installation
------------

 $ git clone http://review.coreboot.org/p/coreboot.git

 $ cd coreboot/payloads/libpayload

 $ make menuconfig

 $ make

 $ sudo make install (optional, will install into /opt per default)

As libpayload is for 32bit x86 systems only, you might have to install the
32bit libgcc version, otherwise your payloads will fail to compile.
On Debian systems you'd do 'apt-get install gcc-multilib' for example.


Usage
-----

Here's an example of a very simple payload (hello.c) and how to build it:

 #include <libpayload.h>

 int main(void)
 {
     printf("Hello, world!\n");
     return 0;
 }

Building the payload using the 'lpgcc' compiler wrapper:

 $ lpgcc -o hello.elf hello.c

Please see the sample/ directory for details.


Website and Mailing List
------------------------

The main website is http://www.coreboot.org/Libpayload.

For additional information, patches, and discussions, please join the
coreboot mailing list at http://coreboot.org/Mailinglist, where most
libpayload developers are subscribed.


Copyright and License
---------------------

See LICENSES.