Hi Guenter, On Fri, Nov 2, 2018 at 3:52 AM Guenter Roeck <li...@roeck-us.net> 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 (debian-ker...@lists.debian.org) (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 root@rpi3:~# 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 > > >
