[coreboot-gerrit] Change in coreboot[master]: libpayload: Make 8250 UART driver relocation safe

[Nico Huber (Code Review)]

Hello build bot (Jenkins), Reto Buerki, Angel Pons, Julius Werner,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/37513

to look at the new patch set (#2).

Change subject: libpayload: Make 8250 UART driver relocation safe
......................................................................

libpayload: Make 8250 UART driver relocation safe

`lib_sysinfo->serial` is a virtual pointer into coreboot tables.
It's not valid across relocation. Accessing the wrong value during
relocation of FILO resulted in a hang with DEBUG_SEGMENT and UART
console enabled. Work around that by caching the whole table entry
locally.

An alternative would be to revise `sysinfo`, to contain no virtual
pointers to anything outside the payload.

Change-Id: I03adaf57b83a177316d7778f7e06df8eb6f9158e
Signed-off-by: Nico Huber 
---
M payloads/libpayload/drivers/serial/8250.c
1 file changed, 10 insertions(+), 11 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/37513/2