Hello Cedric,

On 11/27/23 10:31, Cédric Le Goater wrote:
Hello Ninad,

On 10/26/23 18:47, Ninad Palsule wrote:
Hello,

Please review the patch-set version 7.
I have incorporated review comments from Cedric, Philippe and Thomas.


I reworked v7 with the suggestions I made in patches 1-6. Please check :

  https://github.com/legoater/qemu/commits/aspeed-8.2

I will have more questions on the mappings because some parts are really
unclear.

I forgot to mention in my last mail. If I build against your aspeed-8.2 branch then rainier machine is failing to boot.

The same images are working with qemu master branch.

fsi/qemu-system-arm -M rainier-bmc -nographic -kernel ./fitImage-linux.bin -dtb ./aspeed-bmc-ibm-rainier.dtb -initrd ./obmc-phosphor-initramfs.rootfs.cpio.xz -drive file=./obmc-phosphor-image.rootfs.wic.qcow2,if=sd,index=2 -append 'rootwait console=ttyS4,115200n8 root=PARTLABEL=rofs-a' -net nic -net user,hostfwd=:127.0.0.1:3222-:22,hostfwd=:127.0.0.1:2234-:1234 -trace 'fsi*'
                        |
                        |
Starting systemd-udevd version 254^
[   50.630407] /dev/disk/by-partlabel/rofs-a: Can't open blockdev
mount: mounting /dev/disk/by-partlabel/rofs-a on /mnt/rofs failed: No such file or directory
/bin/sh: can't access tty; job control turned off


Thanks,

C.



Ninad Palsule (10):
   hw/fsi: Introduce IBM's Local bus
   hw/fsi: Introduce IBM's scratchpad
   hw/fsi: Introduce IBM's cfam,fsi-slave
   hw/fsi: Introduce IBM's FSI
   hw/fsi: IBM's On-chip Peripheral Bus
   hw/fsi: Aspeed APB2OPB interface
   hw/arm: Hook up FSI module in AST2600
   hw/fsi: Added qtest
   hw/fsi: Added FSI documentation
   hw/fsi: Update MAINTAINER list

  MAINTAINERS                        |   8 +
  docs/specs/fsi.rst                 | 138 +++++++++++++++
  docs/specs/index.rst               |   1 +
  meson.build                        |   1 +
  hw/fsi/trace.h                     |   1 +
  include/hw/arm/aspeed_soc.h        |   4 +
  include/hw/fsi/aspeed-apb2opb.h    |  33 ++++
  include/hw/fsi/cfam.h              |  34 ++++
  include/hw/fsi/engine-scratchpad.h |  27 +++
  include/hw/fsi/fsi-master.h        |  30 ++++
  include/hw/fsi/fsi-slave.h         |  29 +++
  include/hw/fsi/fsi.h               |  36 ++++
  include/hw/fsi/lbus.h              |  43 +++++
  include/hw/fsi/opb.h               |  33 ++++
  hw/arm/aspeed_ast2600.c            |  19 ++
  hw/fsi/aspeed-apb2opb.c            | 272 +++++++++++++++++++++++++++++
  hw/fsi/cfam.c                      | 173 ++++++++++++++++++
  hw/fsi/engine-scratchpad.c         |  93 ++++++++++
  hw/fsi/fsi-master.c                | 161 +++++++++++++++++
  hw/fsi/fsi-slave.c                 |  78 +++++++++
  hw/fsi/fsi.c                       |  25 +++
  hw/fsi/lbus.c                      |  74 ++++++++
  hw/fsi/opb.c                       |  74 ++++++++
  tests/qtest/aspeed-fsi-test.c      | 205 ++++++++++++++++++++++
  hw/Kconfig                         |   1 +
  hw/arm/Kconfig                     |   1 +
  hw/fsi/Kconfig                     |  23 +++
  hw/fsi/meson.build                 |   6 +
  hw/fsi/trace-events                |  13 ++
  hw/meson.build                     |   1 +
  tests/qtest/meson.build            |   1 +
  31 files changed, 1638 insertions(+)
  create mode 100644 docs/specs/fsi.rst
  create mode 100644 hw/fsi/trace.h
  create mode 100644 include/hw/fsi/aspeed-apb2opb.h
  create mode 100644 include/hw/fsi/cfam.h
  create mode 100644 include/hw/fsi/engine-scratchpad.h
  create mode 100644 include/hw/fsi/fsi-master.h
  create mode 100644 include/hw/fsi/fsi-slave.h
  create mode 100644 include/hw/fsi/fsi.h
  create mode 100644 include/hw/fsi/lbus.h
  create mode 100644 include/hw/fsi/opb.h
  create mode 100644 hw/fsi/aspeed-apb2opb.c
  create mode 100644 hw/fsi/cfam.c
  create mode 100644 hw/fsi/engine-scratchpad.c
  create mode 100644 hw/fsi/fsi-master.c
  create mode 100644 hw/fsi/fsi-slave.c
  create mode 100644 hw/fsi/fsi.c
  create mode 100644 hw/fsi/lbus.c
  create mode 100644 hw/fsi/opb.c
  create mode 100644 tests/qtest/aspeed-fsi-test.c
  create mode 100644 hw/fsi/Kconfig
  create mode 100644 hw/fsi/meson.build
  create mode 100644 hw/fsi/trace-events



Reply via email to