From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54024) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YcHOx-00021U-2D for qemu-devel@nongnu.org; Sun, 29 Mar 2015 13:53:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YcHOw-0003Xh-6L for qemu-devel@nongnu.org; Sun, 29 Mar 2015 13:53:35 -0400 From: =?UTF-8?q?Andreas=20F=C3=A4rber?= Date: Sun, 29 Mar 2015 19:53:15 +0200 Message-Id: <1427651603-9210-1-git-send-email-afaerber@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH RFC for-2.3? 0/8] prep: Fix pc87312 for -device usage List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: qemu-block@nongnu.org, mark.cave-ayland@ilande.co.uk, armbru@redhat.com, hpoussin@reactos.org, qemu-ppc@nongnu.org, jsnow@redhat.com, =?UTF-8?q?Andreas=20F=C3=A4rber?= Hello Markus et al., This series attempts to fix the -device pc87312 issues you reported. I can't add alias properties for devices that don't get created before re= alize. Therefore this involves moving code for various ISA devices, to enable us to initialize the objects early for alias properties and realizing them as part of the composite device once the configuration is known, also fix= ing error propagation while at it. Probably needs a further iteration. Yes, it's terribly invasive, that's why the code was as it is. But the code movements are quite trivial, as long as no in-air conflicts = occur. A follow-up would be to respin my old ISA enabled/disabled series to allo= w inactive ISADevices sitting on an ISABus. Regards, Andreas Cc: Markus Armbruster Cc: Herv=C3=A9 Poussineau Cc: qemu-ppc@nongnu.org Cc: qemu-block@nongnu.org Cc: John Snow Andreas F=C3=A4rber (8): parallel: Factor out header for ISAParallelState struct pc87312: Create isa-parallel in-place and add alias par0-chardev property serial: Move ISASerialState to header pc87312: Create UARTs in-place and add alias properties fdb: Move FDCtrlISABus to header pc87312: Create FDC in-place ide: Move ISAIDEState to header pc87312: Create IDE in-place hw/block/fdc.c | 87 ----------------------- hw/char/parallel.c | 30 +------- hw/char/serial-isa.c | 12 ---- hw/ide/internal.h | 155 ---------------------------------------= - hw/ide/isa.c | 13 ---- hw/isa/pc87312.c | 107 ++++++++++++++++------------ hw/ppc/prep.c | 33 +++++++++ include/hw/block/fdc.h | 88 +++++++++++++++++++++++ include/hw/char/parallel.h | 62 ++++++++++++++++ include/hw/char/serial.h | 14 ++++ include/hw/ide.h | 173 +++++++++++++++++++++++++++++++++++++++= ++++++ include/hw/isa/pc87312.h | 23 +++--- 12 files changed, 442 insertions(+), 355 deletions(-) create mode 100644 include/hw/char/parallel.h --=20 2.1.4