Hi,
ARM and X86 basically use the same 8250 driver. Let's consolidate them and
introduce an architecture independent UART driver structure.
Tested on a Jetson TK1 and X86 Qemu VM.
Untested: MMIO-based access on x86.
This shrinks hypervisor code by 46 lines. (at least something... and avoids
redundancies)
Ralf
since v1:
- refactor write_character to write_char
- use type safe static inline functions instead of #defines
- use proper tags in commit subjects
Ralf Ramsauer (7):
arm-common: uart: remove superfluous includes
arm-common: rename struct uart_chip's write to write_char
arm-common: add reg_{in,out,dist} to struct uart_chip
core: introduce common uart structure
core: make 8250 uart driver available to all architectures
core: establish a common uart interface
core: arm: uart: remove include guards
hypervisor/Makefile | 1 +
hypervisor/arch/arm-common/Kbuild | 2 +-
hypervisor/arch/arm-common/dbg-write.c | 27 +-------
hypervisor/arch/arm-common/include/asm/uart.h | 21 +-----
hypervisor/arch/arm-common/uart-pl011.c | 7 +-
hypervisor/arch/arm-common/uart-xuartps.c | 7 +-
hypervisor/arch/x86/Kbuild | 2 +-
hypervisor/arch/x86/dbg-write.c | 32 +++++++--
hypervisor/arch/x86/include/asm/uart.h | 14 ----
hypervisor/arch/x86/uart.c | 94 ---------------------------
hypervisor/include/jailhouse/uart.h | 33 ++++++++++
hypervisor/{arch/arm-common => }/uart-8250.c | 41 +++++++-----
hypervisor/uart.c | 38 +++++++++++
13 files changed, 139 insertions(+), 180 deletions(-)
delete mode 100644 hypervisor/arch/x86/include/asm/uart.h
delete mode 100644 hypervisor/arch/x86/uart.c
create mode 100644 hypervisor/include/jailhouse/uart.h
rename hypervisor/{arch/arm-common => }/uart-8250.c (54%)
create mode 100644 hypervisor/uart.c
--
2.12.0
--
You received this message because you are subscribed to the Google Groups
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
