[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv3 0/5] fw_cfg: qdev-related tidy-ups
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-devel] [PATCHv3 0/5] fw_cfg: qdev-related tidy-ups |
Date: |
Fri, 16 Jun 2017 14:10:20 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 16/06/17 13:18, address@hidden wrote:
> In file included from /tmp/qemu-test/src/include/hw/loader.h:5,
> from /tmp/qemu-test/src/device_tree.c:27:
> /tmp/qemu-test/src/include/hw/nvram/fw_cfg.h:75: error: redefinition of
> typedef ‘FWCfgState’
> /tmp/qemu-test/src/include/qemu/typedefs.h:35: note: previous declaration of
> ‘FWCfgState’ was here
> /tmp/qemu-test/src/include/hw/nvram/fw_cfg.h:84: error: redefinition of
> typedef ‘FWCfgIoState’
> /tmp/qemu-test/src/include/qemu/typedefs.h:33: note: previous declaration of
> ‘FWCfgIoState’ was here
> /tmp/qemu-test/src/include/hw/nvram/fw_cfg.h:94: error: redefinition of
> typedef ‘FWCfgMemState’
> /tmp/qemu-test/src/include/qemu/typedefs.h:34: note: previous declaration of
> ‘FWCfgMemState’ was here
> make: *** [device_tree.o] Error 1
> make: *** Waiting for unfinished jobs....
> In file included from /tmp/qemu-test/src/include/hw/loader.h:5,
> from /tmp/qemu-test/src/vl.c:73:
> /tmp/qemu-test/src/include/hw/nvram/fw_cfg.h:75: error: redefinition of
> typedef ‘FWCfgState’
> /tmp/qemu-test/src/include/qemu/typedefs.h:35: note: previous declaration of
> ‘FWCfgState’ was here
> /tmp/qemu-test/src/include/hw/nvram/fw_cfg.h:84: error: redefinition of
> typedef ‘FWCfgIoState’
> /tmp/qemu-test/src/include/qemu/typedefs.h:33: note: previous declaration of
> ‘FWCfgIoState’ was here
> /tmp/qemu-test/src/include/hw/nvram/fw_cfg.h:94: error: redefinition of
> typedef ‘FWCfgMemState’
> /tmp/qemu-test/src/include/qemu/typedefs.h:34: note: previous declaration of
> ‘FWCfgMemState’ was here
> make: *** [vl.o] Error 1
> tests/docker/Makefile.include:118: recipe for target 'docker-run' failed
> make[1]: *** [docker-run] Error 2
> make[1]: Leaving directory '/var/tmp/patchew-tester-tmp-mjmbvk4m/src'
> tests/docker/Makefile.include:149: recipe for target 'address@hidden' failed
> make: *** address@hidden Error 2
> === OUTPUT END ===
During the rework I accidentally picked up a change from an experimental
branch which (among other changes) added typedefs to the FWCfg*State
structs which I though were needed, but really aren't because they are
already defined in typedefs.h.
Revised version to follow shortly.
ATB,
Mark.
- [Qemu-devel] [PATCHv3 0/5] fw_cfg: qdev-related tidy-ups, Mark Cave-Ayland, 2017/06/16
- [Qemu-devel] [PATCHv3 1/5] fw_cfg: don't map the fw_cfg IO ports in fw_cfg_io_realize(), Mark Cave-Ayland, 2017/06/16
- [Qemu-devel] [PATCHv3 3/5] fw_cfg: move assert() and linking of fw_cfg device to the machine into instance_init(), Mark Cave-Ayland, 2017/06/16
- [Qemu-devel] [PATCHv3 4/5] fw_cfg: move qdev_init_nofail() from fw_cfg_init1() to callers, Mark Cave-Ayland, 2017/06/16
- [Qemu-devel] [PATCHv3 5/5] fw_cfg: move QOM type defines and fw_cfg types into fw_cfg.h, Mark Cave-Ayland, 2017/06/16
- [Qemu-devel] [PATCHv3 2/5] fw_cfg: move setting of FW_CFG_VERSION_DMA bit to fw_cfg_init1(), Mark Cave-Ayland, 2017/06/16
- Re: [Qemu-devel] [PATCHv3 0/5] fw_cfg: qdev-related tidy-ups, no-reply, 2017/06/16
- Re: [Qemu-devel] [PATCHv3 0/5] fw_cfg: qdev-related tidy-ups,
Mark Cave-Ayland <=