On 2017-03-02 05:56, Ralf Ramsauer wrote:
> 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
>
Thanks, applied all. I've augmented patch 7 with the removal of #ifndef
__ASSEMBLY__ which is also not needed (and likely never was).
Jan
--
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Corporate Competence Center Embedded Linux
--
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.
