[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 0/5] Remove sysbus_add_memory and sysbus_del_memo
From: |
Peter Maydell |
Subject: |
[Qemu-devel] [PATCH v2 0/5] Remove sysbus_add_memory and sysbus_del_memory |
Date: |
Mon, 25 Feb 2013 17:08:57 +0000 |
The functions sysbus_add_memory and sysbus_del_memory are odd wrappers
around around memory_region_add/del_subregion, and their presence
is an encouragement to devices to try to map their own memory
regions into the system address space.
Since they're only used in a couple of places in the milkymist
and musicpal platforms, rewrite those uses to have the sysbus
devices expose the memory regions as sysbus mmio regions, and
then have the creator of the device (ie board code) map them
in the right places. Then we can remove the functions altogether.
The series includes a trivial patch to sysbus to make the init
method optional, since (as part of the move towards using only
instance_init and realize) it's now possible to have a simple
functional device which only needs an instance_init method
and no realize or init [the musicpal-misc device introduced
in patch 2 being one such example].
Tested on both musicpal and milkymist.
I rather suspect sysbus_add_io and sysbus_del_io should also be
removed, but since their users are in PPC and x86 platforms I'll
let somebody else do that part :-)
Changes v1->v2:
* updated 'qdevify musicpal-misc' to drop unneeded typedef
and QOM macros, as per review discussion
Peter Maydell (5):
sysbus: make SysBusDeviceClass::init optional
musicpal: qdevify musicpal-misc
milkymist-minimac2: Just expose buffers as a sysbus mmio region
milkymist-softusb: Don't map RAM memory regions in the device itself
sysbus: Remove sysbus_add_memory and sysbus_del_memory
hw/milkymist-hw.h | 6 +++---
hw/milkymist-minimac2.c | 5 +----
hw/milkymist-softusb.c | 21 +++++++++++----------
hw/musicpal.c | 28 +++++++++++++++++++++++-----
hw/sysbus.c | 21 +++------------------
hw/sysbus.h | 5 -----
6 files changed, 41 insertions(+), 45 deletions(-)
--
1.7.9.5
- [Qemu-devel] [PATCH v2 0/5] Remove sysbus_add_memory and sysbus_del_memory,
Peter Maydell <=
- [Qemu-devel] [PATCH v2 4/5] milkymist-softusb: Don't map RAM memory regions in the device itself, Peter Maydell, 2013/02/25
- [Qemu-devel] [PATCH v2 2/5] musicpal: qdevify musicpal-misc, Peter Maydell, 2013/02/25
- [Qemu-devel] [PATCH v2 5/5] sysbus: Remove sysbus_add_memory and sysbus_del_memory, Peter Maydell, 2013/02/25
- [Qemu-devel] [PATCH v2 1/5] sysbus: make SysBusDeviceClass::init optional, Peter Maydell, 2013/02/25
- [Qemu-devel] [PATCH v2 3/5] milkymist-minimac2: Just expose buffers as a sysbus mmio region, Peter Maydell, 2013/02/25