[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: |
Andrew Jeffery |
Subject: |
Re: [PATCH v2 0/5] hw: aspeed: Init all UART's with serial devices |
Date: |
Mon, 16 May 2022 22:03:10 +0930 |
User-agent: |
Cyrus-JMAP/3.7.0-alpha0-591-gfe6c3a2700-fm-20220427.001-gfe6c3a27 |
On Mon, 16 May 2022, at 16:48, Cédric Le Goater wrote:
> 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.
For what it's worth I run this as a smoke test before pushing updates to
openbmc/qemu:
https://github.com/openbmc/openbmc-build-scripts/blob/master/scripts/test-qemu
Andrew
- Re: [PATCH v2 1/5] hw: aspeed: Add missing UART's, (continued)
- [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, 2022/05/16