Hi Laurent, On 11/1/21 16:43, Philippe Mathieu-Daudé wrote: > On 11/1/21 16:31, Richard Henderson wrote: >> On 11/1/21 4:27 AM, Laurent Vivier wrote: >>> From: Philippe Mathieu-Daudé <[email protected]> >>> >>> Signed-off-by: Philippe Mathieu-Daudé <[email protected]> >>> Reviewed-by: Damien Hedde <[email protected]> >>> Message-Id: <[email protected]> >>> Signed-off-by: Laurent Vivier <[email protected]> >>> --- >>> hw/hppa/lasi.c | 10 +++++++++- >>> hw/input/lasips2.c | 38 ++++++++++++++++++++++++++++---------- >>> include/hw/input/lasips2.h | 17 +++++++++++++---- >>> 3 files changed, 50 insertions(+), 15 deletions(-) >> >> This has broken the qtest-hppa device-introspection test: >> >> ERROR:../src/qom/object.c:2011:object_get_canonical_path_component: code >> should not be reached >> Broken pipe >> Aborted (core dumped) >> >> Also, the previous patch 5 does not compile on its own: >> >> In file included from ../src/hw/input/lasips2.c:28:0: >> /home/richard.henderson/qemu/src/include/hw/input/lasips2.h:17:18: >> error: field ‘reg’ has incomplete type >> MemoryRegion reg; >> ^~~ >> ninja: build stopped: subcommand failed. >> >> Reverting to patch 4 builds, and passes the introspection test. > > Thank you for the update, I'll have a look.
The last 2 patches are broken, do you mind resending your pullreq without them? Thanks, Phil.
