Am 17.09.2013 18:43, schrieb Stefan Weil:
The patch fixes a warning from gcc (Debian 4.6.3-14+rpi1) 4.6.3:
block/stream.c:141:22: error:
‘copy’ may be used uninitialized in this function [-Werror=uninitialized]
This is not a real bug - a better compiler would not complain.
Now 'copy'
Am 19.09.2013 18:59, schrieb Stefan Weil:
Am 19.09.2013 18:52, schrieb Andreas Färber:
Hello,
As reported yesterday on IRC, since the last round of pulls I am getting
the following warning-treated-as-error:
CCblock/stream.o
/home/andreas/QEMU/qemu/block/stream.c: In function
Acked-by: Michael Walle mich...@walle.cc
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/lm32/milkymist.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/lm32/milkymist.c b/hw/lm32/milkymist.c
index f1744ec..15053c4 100644
--- a/hw/lm32/milkymist.c
+++ b/hw/lm32
() for armv7m, too.
Cc: Anthony Liguori anth...@codemonkey.ws
Cc: Aurélien Jarno aurel...@aurel32.net
Cc: Paolo Bonzini pbonz...@redhat.com
Cc: Peter Maydell peter.mayd...@linaro.org
Andreas Färber (18):
mips_mipssim: Silence BIOS loading warning for qtest
arm/boot: Turn arm_load_kernel
Adopt error_report() while at it.
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: Aurelien Jarno aurel...@aurel32.net
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/tc58128.c | 10 ++
hw/sh4/shix.c | 9 +
2 files changed, 11 insertions(+), 8
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/boot.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/arm/boot.c b/hw/arm/boot.c
index 1e313af..0c3dc5f 100644
--- a/hw/arm/boot.c
+++ b/hw/arm/boot.c
@@ -15,6 +15,7 @@
#include hw/loader.h
#include elf.h
#include sysemu
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/palm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/arm/palm.c b/hw/arm/palm.c
index 3e39044..ff6bfab 100644
--- a/hw/arm/palm.c
+++ b/hw/arm/palm.c
@@ -19,6 +19,7 @@
#include hw/hw.h
#include audio/audio.h
Adopt error_report().
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/armv7m.c | 25 ++---
1 file changed, 14 insertions(+), 11 deletions(-)
diff --git a/hw/arm/armv7m.c b/hw/arm/armv7m.c
index 89a9015..397e8df 100644
--- a/hw/arm/armv7m.c
+++ b/hw/arm/armv7m.c
Replacing the assert() with more user-friendly error handling is left
for a follow-up.
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/unicore32/puv3.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/unicore32/puv3.c b/hw/unicore32/puv3.c
index a900061..e05cbc1 100644
--- a/hw
Simply skip flash setup for now.
Also drop useless debug output.
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/mainstone.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/hw/arm/mainstone.c b/hw/arm/mainstone.c
index b244f7e..9402c84 100644
--- a/hw/arm
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/exynos4_boards.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c
index 2929f9f..26cedec 100644
--- a/hw/arm/exynos4_boards.c
+++ b/hw/arm/exynos4_boards.c
@@ -22,6
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/cris/axis_dev88.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/hw/cris/axis_dev88.c b/hw/cris/axis_dev88.c
index 03058d3..5524088 100644
--- a/hw/cris/axis_dev88.c
+++ b/hw/cris/axis_dev88.c
@@ -32,6 +32,7
or xenfv.
Signed-off-by: Andreas Färber afaer...@suse.de
---
tests/Makefile | 26 ++
tests/qom-test.c | 253 +++
2 files changed, 279 insertions(+)
create mode 100644 tests/qom-test.c
diff --git a/tests/Makefile b/tests/Makefile
index 994fef1
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/m68k/mcf5208.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/m68k/mcf5208.c b/hw/m68k/mcf5208.c
index fb96fe8..6e30c0b 100644
--- a/hw/m68k/mcf5208.c
+++ b/hw/m68k/mcf5208.c
@@ -10,6 +10,7 @@
#include qemu/timer.h
#include hw
Reviewed-by: Aurelien Jarno aurel...@aurel32.net
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/mips/mips_mipssim.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/mips/mips_mipssim.c b/hw/mips/mips_mipssim.c
index 242bab9..239aa6a 100644
--- a/hw/mips
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/m68k/an5206.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/m68k/an5206.c b/hw/m68k/an5206.c
index a8eee44..24f2068 100644
--- a/hw/m68k/an5206.c
+++ b/hw/m68k/an5206.c
@@ -12,6 +12,7 @@
#include hw/loader.h
#include elf.h
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/gumstix.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/hw/arm/gumstix.c b/hw/arm/gumstix.c
index e97fbbd..aeea172 100644
--- a/hw/arm/gumstix.c
+++ b/hw/arm/gumstix.c
@@ -42,6 +42,7 @@
#include hw
Am 30.08.2013 07:28, schrieb Alexey Kardashevskiy:
This fixes XICS according new QOM rules.
This converts ICS's init() callbacks to realize().
This converts legacy qdev_init_nofail() to property_set(realized).
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
Reviewed-by: Andreas
Am 30.08.2013 07:28, schrieb Alexey Kardashevskiy:
This adds missing const specifiers to ICS and ICP TypeInfo's.
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
Reviewed-by: Andreas Färber afaer...@suse.de
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF
Reviewed-by: Aurelien Jarno aurel...@aurel32.net
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/sh4/shix.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/hw/sh4/shix.c b/hw/sh4/shix.c
index 1ff37f5..f008b98 100644
--- a/hw/sh4/shix.c
+++ b/hw/sh4/shix.c
@@ -50,7 +50,6 @@ static
Am 23.09.2013 15:35, schrieb Andreas Färber:
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/boot.c | 4
1 file changed, 4 insertions(+)
Sorry, I forgot that Grant Likely had an alternative patch [1] not
restricted to qtest. Last thing I read was that PMM had similar/further
Am 30.08.2013 07:28, schrieb Alexey Kardashevskiy:
This replaces old-style fprintf with new style error_report.
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
Reviewed-by: Andreas Färber afaer...@suse.de
Thanks,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/omap_sx1.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/arm/omap_sx1.c b/hw/arm/omap_sx1.c
index b0f8664..235fed8 100644
--- a/hw/arm/omap_sx1.c
+++ b/hw/arm/omap_sx1.c
@@ -32,6 +32,7 @@
#include hw/arm/arm.h
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/arm/z2.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/arm/z2.c b/hw/arm/z2.c
index 2e0d5d4..61c9ca7 100644
--- a/hw/arm/z2.c
+++ b/hw/arm/z2.c
@@ -24,6 +24,7 @@
#include ui/console.h
#include audio/audio.h
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/sparc/leon3.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/sparc/leon3.c b/hw/sparc/leon3.c
index 390f3e4..c583c3d 100644
--- a/hw/sparc/leon3.c
+++ b/hw/sparc/leon3.c
@@ -26,6 +26,7 @@
#include hw/ptimer.h
Am 24.09.2013 18:45, schrieb Fabien Chouteau:
Signed-off-by: Fabien Chouteau chout...@adacore.com
---
disas.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
Reviewed-by: Andreas Färber afaer...@suse.de
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
Am 25.09.2013 02:38, schrieb Michael Roth:
Quoting Laszlo Ersek (2013-08-28 05:49:05)
On 08/20/13 13:43, arm...@redhat.com wrote:
From: Markus Armbruster arm...@redhat.com
v2: Nominate for qemu-stable (Andreas)
Markus Armbruster (2):
tests: Fix schema parser test for in-tree build
Hi,
Am 27.09.2013 12:10, schrieb Mian M. Hamayun:
From: John Rigby john.ri...@linaro.org
Just an copy of a15 with a57 substituting a15 for now.
Signed-off-by: John Rigby john.ri...@linaro.org
Signed-off-by: Mian M. Hamayun m.hama...@virtualopensystems.com
---
hw/cpu/Makefile.objs | 1
Am 30.06.2013 09:34, schrieb Andreas Färber:
Am 24.06.2013 08:52, schrieb peter.crosthwa...@xilinx.com:
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite
Am 22.07.2013 19:27, schrieb Peter Maydell:
On 22 July 2013 17:36, Andreas Färber afaer...@suse.de wrote:
Signed-off-by: Andreas Färber afaer...@suse.de
---
HACKING | 1 +
1 file changed, 1 insertion(+)
diff --git a/HACKING b/HACKING
index e73ac79..d9dbb46 100644
--- a/HACKING
+++ b
Am 22.07.2013 22:29, schrieb Anthony Liguori:
Michael S. Tsirkin m...@redhat.com writes:
On Sun, Jul 21, 2013 at 04:09:04PM +0200, Andreas Färber wrote:
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-bridge/ioh3420.c| 23 ++-
hw/pci-bridge
Am 22.07.2013 23:03, schrieb Seiji Aguchi:
Convert stderr messages calling error_get_pretty()
to error_report().
Timestamp is prepended by -msg timstamp option with it.
This is suggested by Luiz Capitulino.
http://marc.info/?l=qemu-develm=137331442628866w=2
Signed-off-by: Seiji
Am 22.07.2013 22:16, schrieb Hervé Poussineau:
BTW, I've a patch to really cleanup i82378 implementation (47
insertions, 175 deletions). Should I send it now, during 1.6 soft freeze?
Yes please, since it's a PReP-only device and relevant to the general
discussion.
Andreas
--
SUSE LINUX
Am 22.07.2013 19:22, schrieb Andreas Färber:
Am 21.07.2013 22:26, schrieb Michael S. Tsirkin:
On Sun, Jul 21, 2013 at 04:09:00PM +0200, Andreas Färber wrote:
Hello Michael et al.,
This series turns PCIBridge, PCIEPort and PCIESlot into abstract QOM types,
so that we can use QOM casts
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-bridge/pci_bridge_dev.c | 22 ++
1 file changed, 14 insertions(+), 8 deletions(-)
diff --git a/hw/pci-bridge/pci_bridge_dev.c b/hw/pci-bridge
Introduce TYPE_PCI_BRIDGE as base type and use PCI_BRIDGE() casts.
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-bridge/dec.c| 4 ++--
hw/pci-bridge/i82801b11.c | 6 +++---
hw/pci-bridge/ioh3420.c| 18 -
hw/pci-bridge
Move PCIEPort's port property to the new type, same for aer_log_max.
Move PCIESlot's chassis and slot properties to the new type.
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-bridge/ioh3420.c| 31 ++-
hw/pci-bridge/xio3130_downstream.c | 31
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-bridge/i82801b11.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
index 88f489a..90fc91d 100644
--- a/hw/pci-bridge/i82801b11.c
+++ b/hw/pci-bridge/i82801b11.c
...@cn.fujitsu.com
Andreas Färber (4):
pci-bridge: Turn PCIBridge into abstract QOM type
pci-bridge-dev: QOM parent field cleanup
pci-bridge/i82801b11: Rename parent field
pcie_port: Turn PCIEPort and PCIESlot into abstract QOM types
hw/pci-bridge/dec.c| 4 +--
hw/pci-bridge
Am 23.07.2013 00:34, schrieb Peter Maydell:
On 22 July 2013 23:05, Anthony Liguori anth...@codemonkey.ws wrote:
Peter Maydell peter.mayd...@linaro.org writes:
We don't currently have any PCI host controller which is:
(a) for ARM
In QEMU? You can make one super easily by just extending
Am 11.07.2013 07:35, schrieb Igor Mammedov:
On Wed, 10 Jul 2013 18:26:17 +0200
Andreas Färber afaer...@suse.de wrote:
Avoids duplicating the calculation.
Signed-off-by: Andreas Färber afaer...@suse.de
---
Have this lying around for a while already, apparently not sent out yet.
linux
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/lsi53c895a.c | 26
of code refactoring
and cleanup. For the handful of devices that were doing this by
mistake, replace the QOM typenames with the corresponding literal
strings.
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
[AF: Use TYPE_PVSCSI for TypeInfo instead]
Signed-off-by: Andreas Färber afaer
-by: Andreas Färber afaer...@suse.de
---
hw/usb/hcd-xhci.c | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xhci.c
index d7a54fd..0566886 100644
--- a/hw/usb/hcd-xhci.c
+++ b/hw/usb/hcd-xhci.c
@@ -482,6 +482,11 @@ struct
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/megasas.c | 15
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/esp-pci.c | 14
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/net/e1000.c | 18
Replace direct uses of E1000State::dev field with QOM casts and rename
it to parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/net/e1000.c | 35 ++-
1 file changed, 22 insertions(+), 13
Replace direct uses of PCIXenPlatformState::pci_dev field with QOM casts
and rename it to parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/xen/xen_platform.c | 21 +
1 file changed, 13 insertions(+), 8
Replace direct uses of MegasasState::dev with QOM casts and rename it to
parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/megasas.c | 46 ++
1 file changed, 26 insertions
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST() and
direct - style casting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
[AF: Renamed to TYPE_VMWARE_VGA and VMWARE_VGA()]
Signed-off-by: Andreas Färber
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Converted remaining accesses and renamed to parent_obj]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-host/piix.c | 25 -
1 file changed, 16 insertions(+), 9 deletions(-)
diff
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
[AF: Replaced another DO_UPCAST()]
Signed-off-by: Andreas Färber afaer...@suse.de
Replace direct uses of PIIX4PMState::dev with QOM casts and rename it to
parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/acpi/piix4.c | 32
1 file changed, 20 insertions(+), 12 deletions
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
[AF: Simplified casts and converted two more DO_UPCAST()s]
Signed-off-by: Andreas
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
[AF: Dropped opaque casts]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/acpi/piix4.c | 31
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and replace FROM_SYSBUS().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/pflash_cfi02.c | 21 -
1 file changed, 12 insertions(+), 9
Replace direct uses of pci_vmsvga_state_s::card with QOM casts and
rename it to parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/display/vmware_vga.c | 23 +--
1 file changed, 13 insertions(+), 10
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Moved sysbus_init_mmio() to instance_init, renamed variable]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/nvram/fw_cfg.c | 27 +--
1 file changed, 17 insertions(+), 10 deletions
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Syntax and wording changes]
Signed-off-by: Andreas Färber afaer...@suse.de
---
include/hw/sysbus.h | 12
1 file changed, 12 insertions(+)
diff --git a/include/hw/sysbus.h b/include/hw/sysbus.h
index
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and avoid DO_UPCAST().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/ide/ahci.c | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
diff
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and avoid FROM_SYSBUS().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/i386/kvm/clock.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions
-by: Andreas Färber afaer...@suse.de
---
hw/ide/ahci.h | 5 +
hw/ide/ich.c | 10 +-
2 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/hw/ide/ahci.h b/hw/ide/ahci.h
index 341a571..f997c67 100644
--- a/hw/ide/ahci.h
+++ b/hw/ide/ahci.h
@@ -305,6 +305,11 @@ typedef struct
Replace direct uses of IVShmemState::dev with QOM casts and rename it to
parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/misc/ivshmem.c | 37 ++---
1 file changed, 22 insertions(+), 15
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and avoid DO_UPCAST().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/esp.c | 21 +
1 file changed, 13 insertions(+), 8 deletions
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and avoid DO_UPCAST().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Avoid remaining OHCIPCIState::pci_dev uses, rename parent fields]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/usb/hcd-ohci.c | 39
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed argument]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/pflash_cfi01.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/hw/block/pflash_cfi01.c b/hw/block
Rename fdctrl_init_common() to fdctrl_realize_common() and let
fdctrl_connect_drives() propagate an Error through it.
Reviewed-by: Hu Tao hu...@cn.fujitsu.com
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/fdc.c | 40
From: Hu Tao hu...@cn.fujitsu.com
This works since commit 4ce5dae88ecf2bafa0cd663de7e923728b1b3672.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Edited]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/isa/isa-bus.c | 8
1 file changed, 8 deletions(-)
diff --git a/hw/isa/isa
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed SysBusDevice variable]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/fdc.c | 63 ++
1 file changed, 41 insertions(+), 22 deletions
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Drop sysbus_esp_init()]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/esp.c | 19 +++
1 file changed, 11 insertions(+), 8 deletions(-)
diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
index
From: Hu Tao hu...@cn.fujitsu.com
And split off MemoryRegion initialization into instance_init.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-host/q35.c | 29 +++--
1 file changed, 15 insertions(+), 14 deletions
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Avoided repeated SYS_BUS_DEVICE() casts]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/ide/ahci.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/hw/ide/ahci.c b/hw/ide
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and replace FROM_SYSBUS().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/pflash_cfi01.c | 21 -
1 file changed, 12 insertions(+), 9
Only apply legacy command line handling when the device has not been
hot-plugged. Propagate failure of legacy command line handling.
Cc: qemu-sta...@nongnu.org
Acked-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/megasas.c | 5 -
1 file
From: Hu Tao hu...@cn.fujitsu.com
Use type constant if possible and avoid DO_UPCAST().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/nvram/fw_cfg.c | 17 ++---
1 file changed, 10 insertions(+), 7 deletions
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Avoided SYS_BUS_DEVICE() in loop]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/timer/hpet.c | 32 +++-
1 file changed, 19 insertions(+), 13 deletions(-)
diff --git a/hw/timer
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed variable]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/i386/kvmvapic.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/hw/i386/kvmvapic.c b/hw/i386/kvmvapic.c
index
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant, avoid FROM_SYSBUS().
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field and renamed variable]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/timer/hpet.c | 19 +--
1 file changed, 13 insertions
Let scsi_bus_legacy_add_drive() and scsi_bus_legacy_handle_cmdline()
return an Error**. Prepare qdev initfns for QOM realize error model.
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci/pci-hotplug-old.c | 2 +-
hw/scsi/esp-pci.c| 7 ++-
hw/scsi/esp.c| 6
Replace direct uses of XHCIState::pci_dev with QOM casts and rename it
to parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/usb/hcd-xhci.c | 83 ---
1 file changed, 48
Replace direct uses of LSIState::dev with QOM casts and rename it to
parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/lsi53c895a.c | 46 +++---
1 file changed, 27 insertions
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/block/pflash_cfi02.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c
index
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-host/piix.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/hw/pci-host/piix.c b/hw/pci-host/piix.c
index
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/i386/kvm/clock.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/hw/i386/kvm/clock.c b/hw/i386/kvm/clock.c
index 6265458..e89e2f7 100644
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/net/rtl8139.c | 26
Replace direct uses of PCITestDevState::dev with QOM casts and rename it
to parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/misc/pci-testdev.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git
From: Hu Tao hu...@cn.fujitsu.com
Introduce type constant and avoid DO_UPCAST(), container_of(),
and use DEVICE() to avoid accessing parent qdev directly.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Renamed parent field and avoided repeated SYS_BUS_DEVICE() casts]
Signed-off-by: Andreas
Am 22.07.2013 19:36, schrieb Peter Maydell:
On 22 July 2013 18:32, Andreas Färber afaer...@suse.de wrote:
Am 22.07.2013 19:27, schrieb Peter Maydell:
Here's my suggestion for this paragraph (ie to replace
both the Use target_ulong... and Use vaddr sentences
above):
===begin===
For CPU
Replace direct uses of AHCIPCIState::card with QOM casts and rename it
to parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/ide/ahci.c | 11 ++-
hw/ide/ahci.h | 5 -
hw/ide/ich.c | 22 +++---
3
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Tweaked error message]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/intc/ioapic_common.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/hw/intc/ioapic_common.c b/hw/intc
with DeviceClass::realize
Andreas Färber (15):
net/e1000: QOM parent field cleanup
net/rtl8139: QOM parent field cleanup
usb/hcd-xhci: QOM parent field cleanup
scsi/lsi53c895a: QOM parent field cleanup
scsi
From: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/usb/hcd-ohci.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/hw/usb/hcd-ohci.c b/hw/usb/hcd-ohci.c
index 7da2564..2bab8ff 100644
From: Peter Crosthwaite peter.crosthwa...@xilinx.com
Define and use standard QOM cast macro. Remove usages of DO_UPCAST()
and direct - style upcasting.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
[AF: Renamed parent field, renamed from PC_NET to PCNET]
Signed-off-by: Andreas
From: Hu Tao hu...@cn.fujitsu.com
And remove variables if possible.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
[AF: Converted remaining access and renamed to parent_obj]
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/i386/pc_q35.c | 4 +++-
hw/pci-host/q35.c | 43
Replace direct uses of PCIESPState::dev with QOM casts and rename it to
parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/scsi/esp-pci.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/hw
Replace direct uses of RTL8139State::dev with QOM casts and rename it to
parent_obj.
Acked-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/net/rtl8139.c | 88 +++-
1 file changed, 49
This assures the trivial field initialization is applied for any derived
type - currently only Q35PCIHost.
Signed-off-by: Andreas Färber afaer...@suse.de
---
hw/pci-host/q35.c | 4
hw/pci/pcie_host.c | 7 ---
include/hw/pci/pcie_host.h | 1 -
3 files changed, 4
From: Andreas Färber andreas.faer...@web.de
Split the SysBusDevice initfn into instance_init and realizefn.
Signed-off-by: Andreas Färber andreas.faer...@web.de
---
hw/timer/arm_mptimer.c | 25 +++--
1 file changed, 15 insertions(+), 10 deletions(-)
diff --git a/hw/timer
501 - 600 of 10507 matches
Mail list logo