[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/5] hw: aspeed: Init all UART's with serial devices
From: |
Cédric Le Goater |
Subject: |
Re: [PATCH v2 0/5] hw: aspeed: Init all UART's with serial devices |
Date: |
Mon, 16 May 2022 09:18:46 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.0 |
On 5/16/22 08:23, Peter Delevoryas wrote:
v2:
- Rebased on Cedric's irq proposal. [1]
- Added "Introduce common UART init function" patch
- Added "Add uarts_num SoC attribute" patch
- Rewrote last commit's message for clarity
Looks good to me.
I tried testing this by running acceptance tests, particularly the
boot_linux_console.py file, but I had to disable the raspi2_initrd case.
It's not related to my changes (A/B tested and it fails on upstream/master
too), but thought I would mention that.>
I also manually tested several machines:
AST2400:
https://github.com/facebook/openbmc/releases/download/v2021.49.0/wedge100.mtd
AST2500:
https://github.com/facebook/openbmc/releases/download/v2021.49.0/fby3.mtd
AST2600:
https://github.com/facebook/openbmc/releases/download/v2021.49.0/fuji.mtd
AST1030:
https://github.com/peterdelevoryas/OpenBIC/releases/download/oby35-cl-2022.13.01/Y35BCL.elf
[1] 20220516055620.2380197-1-clg@kaod.org/">https://lore.kernel.org/qemu-devel/20220516055620.2380197-1-clg@kaod.org/
I have quite a few images which I run manually. OpenBMC is providing
images, Aspeed also. Joel did a small tool for the IBM rainier :
https://github.com/shenki/qemu-boot-test
Having an automated framework for Aspeed machines pulling images
from different places would be nice but we cannot put all under
QEMU.
I like the buildroot CI using the QEMU boards. See a pipeline example:
https://gitlab.com/legoater/buildroot/-/pipelines/539559209
but that's buildroot only and integrating a custom uboot might be
difficult.
For my ppc needs, I did a quick-and-dirty non-regression test suite :
https://github.com/legoater/qemu-ppc-boot
Anyway, I haven't found a solution nor spent much time on it but
I am interested !
C.
Peter Delevoryas (5):
hw: aspeed: Add missing UART's
hw: aspeed: Add uarts_num SoC attribute
hw: aspeed: Ensure AST1030 respects uart-default
hw: aspeed: Introduce common UART init function
hw: aspeed: Init all UART's with serial devices
hw/arm/aspeed_ast10x0.c | 32 +++++++++++++++++++++++++++-----
hw/arm/aspeed_ast2600.c | 27 ++++++++++++++++++++++-----
hw/arm/aspeed_soc.c | 33 +++++++++++++++++++++++++++++----
include/hw/arm/aspeed_soc.h | 10 ++++++++++
4 files changed, 88 insertions(+), 14 deletions(-)
- [PATCH v2 1/5] hw: aspeed: Add missing UART's, (continued)
- [PATCH v2 1/5] hw: aspeed: Add missing UART's, Peter Delevoryas, 2022/05/16
- [PATCH v2 3/5] hw: aspeed: Ensure AST1030 respects uart-default, Peter Delevoryas, 2022/05/16
- [PATCH v2 4/5] hw: aspeed: Introduce common UART init function, Peter Delevoryas, 2022/05/16
- [PATCH v2 2/5] hw: aspeed: Add uarts_num SoC attribute, Peter Delevoryas, 2022/05/16
- [PATCH v2 5/5] hw: aspeed: Init all UART's with serial devices, Peter Delevoryas, 2022/05/16
- Re: [PATCH v2 0/5] hw: aspeed: Init all UART's with serial devices,
Cédric Le Goater <=