[PATCH v4 0/7] hw/arm/raspi: Run U-Boot on the raspi machines

Philippe Mathieu-Daudé posted 7 patches 4 years, 3 months ago
Test FreeBSD passed
Test docker-mingw@fedora passed
Test checkpatch passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200120235159.18510-1-f4bug@amsat.org
hw/arm/raspi.c                            |  2 -
python/qemu/machine.py                    |  9 +++-
tests/acceptance/avocado_qemu/__init__.py | 59 +++++++++++++++++------
tests/acceptance/boot_linux_console.py    | 54 +++++++++++++++++++++
4 files changed, 107 insertions(+), 17 deletions(-)
[PATCH v4 0/7] hw/arm/raspi: Run U-Boot on the raspi machines
Posted by Philippe Mathieu-Daudé 4 years, 3 months ago
Following Laurent report:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg639950.html

The SYS_timer is already merged, see:
https://git.qemu.org/?p=qemu.git;a=commit;h=d05be883fc
"hw/timer/bcm2835: Add the BCM2835 SYS_timer"

The first patch should fix Laurent other issue.
Then few python patches are require to break into U-Boot console,
and the last patches add U-Boot tests for Raspi2 and Raspi3.

Laurent, if you successfully test U-Boot with this patchset again,
do you mind replying with a "Tested-by:" tag?

Regards,

Phil.

Since v3:
- rewrote '-smp' fix.
- tests use Debian 'trustable' u-boot.elf

previous feedbacks from Peter on v3:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg655415.html

v3: https://www.mail-archive.com/qemu-devel@nongnu.org/msg653807.html
Supersedes: <20191019234715.25750-1-f4bug@amsat.org>

Philippe Mathieu-Daudé (7):
  hw/arm/raspi: Remove obsolete use of -smp to set the soc
    'enabled-cpus'
  Acceptance tests: Extract _console_interaction()
  Acceptance tests: Add interrupt_interactive_console_until_pattern()
  python/qemu/machine: Allow to use other serial consoles than default
  tests/boot_linux_console: Test booting U-Boot on the Raspberry Pi 2
  tests/boot_linux_console: Test booting U-Boot on the Raspberry Pi 3
  tests/boot_linux_console: Tag Emcraft Smartfusion2 as running 'u-boot'

 hw/arm/raspi.c                            |  2 -
 python/qemu/machine.py                    |  9 +++-
 tests/acceptance/avocado_qemu/__init__.py | 59 +++++++++++++++++------
 tests/acceptance/boot_linux_console.py    | 54 +++++++++++++++++++++
 4 files changed, 107 insertions(+), 17 deletions(-)

-- 
2.21.1


Re: [PATCH v4 0/7] hw/arm/raspi: Run U-Boot on the raspi machines
Posted by Philippe Mathieu-Daudé 4 years, 3 months ago
Hi Peter,

(Cc'ed Wainer from the Python part).

On 1/21/20 12:51 AM, Philippe Mathieu-Daudé wrote:
> Following Laurent report:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg639950.html
> 
> The SYS_timer is already merged, see:
> https://git.qemu.org/?p=qemu.git;a=commit;h=d05be883fc
> "hw/timer/bcm2835: Add the BCM2835 SYS_timer"
> 
> The first patch should fix Laurent other issue.
> Then few python patches are require to break into U-Boot console,
> and the last patches add U-Boot tests for Raspi2 and Raspi3.
> 
> Laurent, if you successfully test U-Boot with this patchset again,
> do you mind replying with a "Tested-by:" tag?
> 
> Regards,
> 
> Phil.
> 
> Since v3:
> - rewrote '-smp' fix.
> - tests use Debian 'trustable' u-boot.elf
> 
> previous feedbacks from Peter on v3:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg655415.html
> 
> v3: https://www.mail-archive.com/qemu-devel@nongnu.org/msg653807.html
> Supersedes: <20191019234715.25750-1-f4bug@amsat.org>
> 
> Philippe Mathieu-Daudé (7):
>    hw/arm/raspi: Remove obsolete use of -smp to set the soc 'enabled-cpus'

While the first patch is reviewed by Alistair, the rest (acceptance 
tests) still requires an eye from Cleber/Eduardo.

Can you queue the first patch via your qemu-arm tree?

Thanks,

Phil.

>    Acceptance tests: Extract _console_interaction()
>    Acceptance tests: Add interrupt_interactive_console_until_pattern()
>    python/qemu/machine: Allow to use other serial consoles than default
>    tests/boot_linux_console: Test booting U-Boot on the Raspberry Pi 2
>    tests/boot_linux_console: Test booting U-Boot on the Raspberry Pi 3
>    tests/boot_linux_console: Tag Emcraft Smartfusion2 as running 'u-boot'
> 
>   hw/arm/raspi.c                            |  2 -
>   python/qemu/machine.py                    |  9 +++-
>   tests/acceptance/avocado_qemu/__init__.py | 59 +++++++++++++++++------
>   tests/acceptance/boot_linux_console.py    | 54 +++++++++++++++++++++
>   4 files changed, 107 insertions(+), 17 deletions(-)
> 


Re: [PATCH v4 0/7] hw/arm/raspi: Run U-Boot on the raspi machines
Posted by Peter Maydell 4 years, 2 months ago
On Tue, 28 Jan 2020 at 06:53, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>
> Hi Peter,
>
> (Cc'ed Wainer from the Python part).
>
> On 1/21/20 12:51 AM, Philippe Mathieu-Daudé wrote:
> > Following Laurent report:
> > https://www.mail-archive.com/qemu-devel@nongnu.org/msg639950.html
> >
> > The SYS_timer is already merged, see:
> > https://git.qemu.org/?p=qemu.git;a=commit;h=d05be883fc
> > "hw/timer/bcm2835: Add the BCM2835 SYS_timer"
> >
> > The first patch should fix Laurent other issue.
> > Then few python patches are require to break into U-Boot console,
> > and the last patches add U-Boot tests for Raspi2 and Raspi3.
> >
> > Laurent, if you successfully test U-Boot with this patchset again,
> > do you mind replying with a "Tested-by:" tag?
> >
> > Regards,
> >
> > Phil.
> >
> > Since v3:
> > - rewrote '-smp' fix.
> > - tests use Debian 'trustable' u-boot.elf
> >
> > previous feedbacks from Peter on v3:
> > https://www.mail-archive.com/qemu-devel@nongnu.org/msg655415.html
> >
> > v3: https://www.mail-archive.com/qemu-devel@nongnu.org/msg653807.html
> > Supersedes: <20191019234715.25750-1-f4bug@amsat.org>
> >
> > Philippe Mathieu-Daudé (7):
> >    hw/arm/raspi: Remove obsolete use of -smp to set the soc 'enabled-cpus'
>
> While the first patch is reviewed by Alistair, the rest (acceptance
> tests) still requires an eye from Cleber/Eduardo.
>
> Can you queue the first patch via your qemu-arm tree?

OK, I've queued patch 1 (and dropped the rest from my
to-review queue on the assumption that somebody else will
pick them up and/or you'll send a respin at some point).

-- PMM