Hi Peter,

On 10/5/19 5:47 PM, Philippe Mathieu-Daudé wrote:
Hi all,

Yesterday Peter Maydell asked on IRC if I had any working Exynos4
image. I looked at some old backuped notes and could boot Guenter
initrd with BusyBox.
I'll use this cover letter to share my notes, they might help to
have this board fully usable again.

This board is listed as "Odd Fixes". Since we have it covered, I
thought it was worthwhile to have it covered by tests to avoid
more regressions.

Frédéric Basse used this board last year:
https://fredericb.info/2018/03/emulating-exynos-4210-bootrom-in-qemu.html

I'll have a look a these particular commits he added:

- https://github.com/frederic/qemu-exynos-bootrom/commit/9be5c9f2253dbc04ee

    sd: add sd clock support to SDHC_CLKCON

- https://github.com/frederic/qemu-exynos-bootrom/commit/6f045949ee2fdec624

    sd: always reply to ACMD41 (SD_APP_OP_COND)

Guenter also carries on this patch:

- https://github.com/groeck/qemu/commit/0a80543cc910d

   hw/timer/exynos4210_mct: Initialize timer before starting it

   When booting a recent Linux kernel, the qemu message "Timer with period
   zero, disabling" is seen, apparently because a timer is started before
   being initialized.  Fix the problem by initializing the offending timer
   before starting it.

It might also be interesting to use Krzysztof's initramfs image:
https://github.com/krzk/tools/blob/master/run-qemu.sh#L29

The 1st test added works fine, however the 2nd (SD card) is not
reliable so it is disabled. We might need to adapt the ADMA patch
Igor sent once:
https://patchwork.ozlabs.org/patch/181854/

If you want to run the Avocado tests, you need these other patches
pending review:

- https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg06439.html
   "tests/boot_linux_console: Extract the gunzip() helper"

- https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg06438.html
   "python/qemu/machine: Allow to use other serial consoles than default"
   (only for the 2nd disabled test)

Regards,

Phil.

Based-on: 20190926173428.10713-16-f4...@amsat.org

Philippe Mathieu-Daudé (5):
   tests/boot_linux_console: Add initrd test for the Exynos4210
   hw/sd/sdhci: Add a comment to distinct the i.MX eSDHC functions
   hw/sd/sdhci: Add dummy Samsung SDHCI controller
   hw/arm/exynos4210: Use the Samsung s3c SDHCI controller
   tests/boot_linux_console: Add sdcard test for the Exynos4210

Can you take patches 2-4 from this series? (C part, not Python).
All these patches have been reviewed.

Thanks,

Phil.

Reply via email to