qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Strahinja Jankovic <strahinjapjankovic@gmail.com>
To: "Philippe Mathieu-Daudé" <philmd@linaro.org>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Beniamino Galvani <b.galvani@gmail.com>,
	 Niek Linnenbank <nieklinnenbank@gmail.com>,
	qemu-arm@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [PATCH v2 0/7] Enable Cubieboard A10 boot SPL from SD card
Date: Sun, 18 Dec 2022 23:34:14 +0100	[thread overview]
Message-ID: <CABtshVRUevNjOucUwtKW-sccUAADRe-9JBbYHd8UMGf0YPJXSw@mail.gmail.com> (raw)
In-Reply-To: <e05afcc8-8d86-b942-6702-8ba4887c44db@linaro.org>

Hi,


On Sun, Dec 18, 2022 at 11:17 PM Philippe Mathieu-Daudé
<philmd@linaro.org> wrote:
>
> On 18/12/22 22:19, Strahinja Jankovic wrote:
> > This patch series adds missing Allwinner A10 modules needed for
> > successful SPL boot:
> > - Clock controller module
> > - DRAM controller
> > - I2C0 controller (added also for Allwinner H3 since it is the same)
> > - AXP-209 connected to I2C0 bus
> >
> > It also updates Allwinner A10 emulation so SPL is copied from attached
> > SD card if `-kernel` parameter is not passed when starting QEMU
> > (approach adapted from Allwinner H3 implementation).
> >
> > Boot from SD card has been tested with Cubieboard Armbian SD card image and custom
> > Yocto image built for Cubieboard.
> > Example usage for Armbian image:
> > qemu-system-arm -M cubieboard -nographic -sd ~/Armbian_22.11.0-trunk_Cubieboard_kinetic_edge_6.0.7.img
>
> As a follow-up, could you add a test similar to
> test_arm_orangepi_bionic_20_08() in tests/avocado/boot_linux_console.py?
>
> This test could be refactored as do_test_arm_allwinner_armbian(), called
> for orangepi and cubieboard with the corresponding url / hash.

I was planning to do it (I already have a patch for cubieboard), but
there is a problem with finding a stable-enough image for cubieboard
for automated testing.
Unfortunately, Cubieboard does not have supported Armbian images as
OrangePi-PC has (it only has weekly builds). From the images that can
be found on archive.armbian.com, there is only one bionic image
(21.02) and it won't boot because it hangs due to the musb issue (I
tried both with my patches and without, by extracting
kernel/dtb/initrd). Other images are focal, but for some reason, in
those images it is impossible to interrupt U-Boot (tested with 21.08),
so I could not append to bootcmd to make boot process more verbose and
easier to monitor for automated testing.

That is why, for now, I would suggest not updating the SPL/SD boot
test for Cubieboard.

Best regards,
Strahinja


  reply	other threads:[~2022-12-18 22:35 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-18 21:19 [PATCH v2 0/7] Enable Cubieboard A10 boot SPL from SD card Strahinja Jankovic
2022-12-18 21:19 ` [PATCH v2 1/7] hw/misc: Allwinner-A10 Clock Controller Module Emulation Strahinja Jankovic
2022-12-18 21:19 ` [PATCH v2 2/7] hw/misc: Allwinner A10 DRAM Controller Emulation Strahinja Jankovic
2022-12-18 21:19 ` [PATCH v2 3/7] hw/i2c: Allwinner TWI/I2C Emulation Strahinja Jankovic
2022-12-18 21:19 ` [PATCH v2 4/7] hw/misc: Allwinner AXP-209 Emulation Strahinja Jankovic
2022-12-18 21:28   ` Strahinja Jankovic
2022-12-18 22:07   ` Philippe Mathieu-Daudé
2022-12-18 22:12     ` Strahinja Jankovic
2022-12-18 22:23       ` Philippe Mathieu-Daudé
2022-12-18 22:39         ` Strahinja Jankovic
2022-12-19  7:15           ` Philippe Mathieu-Daudé
2022-12-19 22:05             ` Strahinja Jankovic
2022-12-18 21:19 ` [PATCH v2 5/7] hw/arm: Add AXP-209 to Cubieboard Strahinja Jankovic
2022-12-18 21:55   ` Philippe Mathieu-Daudé
2022-12-18 21:19 ` [PATCH v2 6/7] hw/arm: Allwinner A10 enable SPL load from MMC Strahinja Jankovic
2022-12-18 21:59   ` Philippe Mathieu-Daudé
2022-12-18 22:16     ` Strahinja Jankovic
2022-12-18 21:19 ` [PATCH v2 7/7] docs/system/arm: Update Allwinner with TWI (I2C) Strahinja Jankovic
2022-12-18 22:02   ` Philippe Mathieu-Daudé
2022-12-18 22:08     ` Strahinja Jankovic
2022-12-18 22:17 ` [PATCH v2 0/7] Enable Cubieboard A10 boot SPL from SD card Philippe Mathieu-Daudé
2022-12-18 22:34   ` Strahinja Jankovic [this message]
2022-12-18 22:57     ` Strahinja Jankovic
2022-12-19  7:11       ` Philippe Mathieu-Daudé
2022-12-19 22:03         ` Strahinja Jankovic

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CABtshVRUevNjOucUwtKW-sccUAADRe-9JBbYHd8UMGf0YPJXSw@mail.gmail.com \
    --to=strahinjapjankovic@gmail.com \
    --cc=b.galvani@gmail.com \
    --cc=nieklinnenbank@gmail.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).