[v1] hw/arm/virt: make second UART available

[PATCH qemu.git 0/1] hw/arm/virt: make second UART available

~axelheider posted 1 patch 1 year, 4 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/166982763526.10484.9925072056712598801-0@git.sr.ht

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Ani Sinha <ani@anisinha.ca>, Shannon Zhao <shannon.zhaosl@gmail.com>, Peter Maydell <peter.maydell@linaro.org>

There is a newer version of this series

hw/arm/virt-acpi-build.c | 12 ++++-----
hw/arm/virt.c            | 55 ++++++++++++++++++++++++++++++----------
include/hw/arm/virt.h    |  4 +--
3 files changed, 49 insertions(+), 22 deletions(-)

Expand all Fold all

[PATCH qemu.git 0/1] hw/arm/virt: make second UART available

Posted by ~axelheider 1 year, 4 months ago

This is a follow-up on older attempts to make a second UART
available for the arm-virt machine in normal world. The use case
is, that this give a simple I/O channel in addition to stdout, as this
simplifies various test scenarios. Especially for non-Linux operating
system,s (e.g. seL4) where arm-virt is handy as a generic machine
for testing purposes.

There are existing discussions about this topic at:
- https://lists.gnu.org/archive/html/qemu-arm/2017-12/msg00063.html
- https://lists.nongnu.org/archive/html/qemu-
discuss/2018-11/msg00001.html
- https://lists.gnu.org/archive/html/qemu-devel/2019-12/msg01613.html

This patch tries to address the concerns raised there and also
avoid breaking compatibility with existing setups.

Axel Heider (1):
  hw/arm/virt: make second UART available

 hw/arm/virt-acpi-build.c | 12 ++++-----
 hw/arm/virt.c            | 55 ++++++++++++++++++++++++++++++----------
 include/hw/arm/virt.h    |  4 +--
 3 files changed, 49 insertions(+), 22 deletions(-)

-- 
2.34.5

Re: [PATCH qemu.git 0/1] hw/arm/virt: make second UART available

Posted by Alex Bennée 1 year, 4 months ago

~axelheider <axelheider@git.sr.ht> writes:

> This is a follow-up on older attempts to make a second UART
> available for the arm-virt machine in normal world. The use case
> is, that this give a simple I/O channel in addition to stdout, as this
> simplifies various test scenarios. Especially for non-Linux operating
> system,s (e.g. seL4) where arm-virt is handy as a generic machine
> for testing purposes.
>
> There are existing discussions about this topic at:
> - https://lists.gnu.org/archive/html/qemu-arm/2017-12/msg00063.html
> - https://lists.nongnu.org/archive/html/qemu-
> discuss/2018-11/msg00001.html
> - https://lists.gnu.org/archive/html/qemu-devel/2019-12/msg01613.html
>
> This patch tries to address the concerns raised there and also
> avoid breaking compatibility with existing setups.
>
> Axel Heider (1):
>   hw/arm/virt: make second UART available
>
>  hw/arm/virt-acpi-build.c | 12 ++++-----
>  hw/arm/virt.c            | 55 ++++++++++++++++++++++++++++++----------
>  include/hw/arm/virt.h    |  4 +--
>  3 files changed, 49 insertions(+), 22 deletions(-)

It would also be worth updating ./docs/system/arm/virt.rst to document
this feature. 

-- 
Alex Bennée

Re: [PATCH qemu.git 0/1] hw/arm/virt: make second UART available

Posted by Axel Heider 1 year, 4 months ago

Alex,

> It would also be worth updating ./docs/system/arm/virt.rst to document
> this feature.

Good point. I will add this in the next iteration of the patch.
Until then, the proposed doc changes can be found here:
https://gitlab.com/axel-h/qemu/-/merge_requests/1/diffs

Axel