[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-arm] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem) |
Date: |
Fri, 2 Nov 2018 08:32:00 +0100 |
Hi Guenter,
On Fri, Nov 2, 2018 at 3:52 AM Guenter Roeck <address@hidden> wrote:
>
> On 11/1/18 5:12 PM, Philippe Mathieu-Daudé wrote:
> > Hi,
> >
> > This series is a mix of a previous work I had for the raspi, and a patch
> > from
> > Guenter: https://lists.gnu.org/archive/html/qemu-devel/2018-07/msg03153.html
> >
> > The final patch keep Guenter ideas and comments, but is mostly a rewrite.
> > I dropped the A2W code from this work, it doesn't seems unuseful to me.
> >
> > Guenter can you test this series?
> >
>
> arm/raspi2 works, but aarch64/raspi3 stalls.
Thanks for testing it!
So I suppose the A2W is required. And I'm probably using a too old kernel,
I'm using the Buster preview following Peter's post [1]:
[ 0.000000] Linux version 4.14.0-3-arm64
(address@hidden) (gcc version 7.2.0 (Debian 7.2.0-18))
#1 SMP Debian 4.14.12-2 (2018-01-06)
[ 0.000000] Machine model: Raspberry Pi 3 Model B
[...]
[ 8.044215] systemd[1]: Detected architecture arm64.
Welcome to Debian GNU/Linux buster/sid!
Debian GNU/Linux buster/sid rpi3 ttyAMA0
rpi3 login: root
Password:
Linux rpi3 4.14.0-3-arm64 #1 SMP Debian 4.14.12-2 (2018-01-06) aarch64
address@hidden:~#
I'll look for a newer kernel.
BTW I use these QEMU command line options while testing:
qemu-system-aarch64 \
-d unimp,guest_errors \
-trace bcm2835_cprman_rd_\* -trace bcm2835_cprman_wr_\* \
...
And the cmdline suggested by Peter:
-append "rw earlycon=pl011,0x3f201000 console=ttyAMA0 loglevel=8
root=/dev/mmcblk0p2 fsck.repair=yes net.ifnames=0 rootwait memtest=1"
[1]
https://translatedcode.wordpress.com/2018/04/25/debian-on-qemus-raspberry-pi-3-model/
>
> [ 45.683302] Run /sbin/init as init process
> [ 50.745961] random: dd: uninitialized urandom read (512 bytes read)
> [ 77.478266] Writes: Total: 2074828 Max/Min: 0/0 Fail: 0
>
> ... then nothing else until I abort the session.
>
> This is with the series applied on top of master.
>
> Guenter
>
> > I kept Guenter S-o-b and (C), is that OK? (Guenter?, Peter?)
> >
> > - patches 1, 7: obvious updates in MAINTAINERS
> > - patches 3, 4: simple cleanups
> > - patches 2, 5: add UNIMP code
> > - patch 6: add the cprman (KISS init values from Guenter)
> >
> > Peter: can you take patch #1 for 3.1?
> >
> > Regards,
> >
> > Phil.
> >
> > Philippe Mathieu-Daudé (7):
> > MAINTAINERS: Add an entry for the Raspberry Pi machines
> > hw/misc/bcm2835_property: Handle the 'domain state' property
> > hw/arm/bcm2835: Use 0x prefix for hex numbers
> > hw/arm/bcm2835: Rename some definitions
> > hw/arm/bcm2835: Add various unimplemented peripherals
> > hw/arm/bcm2835: Add basic support for cprman (clock subsystem)
> > MAINTAINERS: Volunteer to review Raspi patches
> >
> > MAINTAINERS | 7 +
> > hw/arm/bcm2835_peripherals.c | 42 +++-
> > hw/char/bcm2835_aux.c | 2 +-
> > hw/intc/bcm2836_control.c | 4 +-
> > hw/misc/Makefile.objs | 1 +
> > hw/misc/bcm2835_cprman.c | 277 +++++++++++++++++++++++++++
> > hw/misc/bcm2835_property.c | 8 +-
> > hw/misc/trace-events | 8 +
> > include/hw/arm/bcm2835_peripherals.h | 11 ++
> > include/hw/arm/raspi_platform.h | 6 +-
> > include/hw/misc/bcm2835_cprman.h | 28 +++
> > 11 files changed, 387 insertions(+), 7 deletions(-)
> > create mode 100644 hw/misc/bcm2835_cprman.c
> > create mode 100644 include/hw/misc/bcm2835_cprman.h
> >
>
- [Qemu-arm] [PATCH for 3.2 v2 7/7] MAINTAINERS: Volunteer to review Raspi patches, (continued)
[Qemu-arm] [PATCH for 3.2 v2 6/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem), Philippe Mathieu-Daudé, 2018/11/01
Re: [Qemu-arm] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem), Guenter Roeck, 2018/11/01
- Re: [Qemu-arm] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem),
Philippe Mathieu-Daudé <=
Re: [Qemu-arm] [Qemu-devel] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem), no-reply, 2018/11/03
Re: [Qemu-arm] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem), Peter Maydell, 2018/11/05
Re: [Qemu-arm] [Qemu-devel] [PATCH for 3.2 v2 0/7] hw/arm/bcm2835: Add basic support for cprman (clock subsystem), no-reply, 2018/11/05