Hi Tim, On 6/5/25 04:14, Tim Lee wrote:
This fails on top of current master, please take a look:$ QTEST_LOG=1 QTEST_QEMU_BINARY=./qemu-system-aarch64 ./tests/qtest/npcm8xx_pspi-test # random seed: R02S03f79fc48ba73b76c881f93f90b015e9 1..3 # Start of aarch64 tests # Start of npcm8xx_pspi tests # starting QEMU: exec ./qemu-system-aarch64 -qtest unix:/tmp/qtest-32530.sock -qtest-log /dev/fd/2 -chardev socket,path=/tmp/qtest-32530.qmp,id=char0 -mon chardev=char0,mode=control -display none -audio none -machine npcm845-evb -accel qtest [I 0.000000] OPENED [R +0.034918] endianness [S +0.034944] OK little {"QMP": {"version": {"qemu": {"micro": 50, "minor": 0, "major": 10}, "package": "v10.0.0-530-g88d6459dae"}, "capabilities": ["oob"]}} {"execute": "qmp_capabilities"} {"return": {}} [R +0.037373] writel 0xf0201002 0x1 [S +0.037396] OK [R +0.037417] readl 0xf0201002 [S +0.037426] OK 0x0000000000000000 ** ERROR:../tests/qtest/npcm8xx_pspi-test.c:45:test_init: assertion failed (pspi_read_ctl(qts, pspi) == CTL_SPIEN): (0x00000000 == 0x00000001) Bail out! [I +0.037909] CLOSED Aborted (core dumped)Thank you for testing it. I think the failure seems to be related to the following commit which, has not been merged yet. https://patchew.org/QEMU/[email protected]/ Here is our test result for reference. Thanks.
It helps and saves maintainers' time if you group related / dependent patches as a series, i.e.: patch introducing a feature with the patch testing the feature. Alternatively when big series are split, you can mention their dependence using the 'Based-on' tag: Based-on: [email protected] Regards, Phil.
