* [Qemu-devel] [PATCH 0/2] Trivial MemoryRegionOps patches
@ 2012-02-25 12:37 Stefan Weil
2012-02-25 12:37 ` [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops Stefan Weil
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Stefan Weil @ 2012-02-25 12:37 UTC (permalink / raw)
To: qemu-trivial; +Cc: qemu-devel
Please apply http://patchwork.ozlabs.org/patch/139660/ to the qemu-trivial
patch queue. Here are two additional trivial patches which also touch
MemoryRegionOps and make the code more uniform.
[PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops
[PATCH 2/2] sh7750: Remove redundant 'struct' from MemoryRegionOps
Regards,
Stefan W.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops
2012-02-25 12:37 [Qemu-devel] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Weil
@ 2012-02-25 12:37 ` Stefan Weil
2012-02-25 23:16 ` Andreas Färber
2012-02-25 12:37 ` [Qemu-devel] [PATCH 2/2] sh7750: Remove redundant 'struct' from MemoryRegionOps Stefan Weil
2012-03-05 13:05 ` [Qemu-devel] [Qemu-trivial] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Hajnoczi
2 siblings, 1 reply; 5+ messages in thread
From: Stefan Weil @ 2012-02-25 12:37 UTC (permalink / raw)
To: qemu-trivial; +Cc: Stefan Weil, qemu-devel, Alexander Graf
spin_rw_ops is only used in hw/ppce500_spin.c.
Cc: Alexander Graf <agraf@suse.de>
Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
hw/ppce500_spin.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/ppce500_spin.c b/hw/ppce500_spin.c
index 6b8a189..6ed676b 100644
--- a/hw/ppce500_spin.c
+++ b/hw/ppce500_spin.c
@@ -182,7 +182,7 @@ static uint64_t spin_read(void *opaque, target_phys_addr_t addr, unsigned len)
}
}
-const MemoryRegionOps spin_rw_ops = {
+static const MemoryRegionOps spin_rw_ops = {
.read = spin_read,
.write = spin_write,
.endianness = DEVICE_BIG_ENDIAN,
--
1.7.9
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Qemu-devel] [PATCH 2/2] sh7750: Remove redundant 'struct' from MemoryRegionOps
2012-02-25 12:37 [Qemu-devel] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Weil
2012-02-25 12:37 ` [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops Stefan Weil
@ 2012-02-25 12:37 ` Stefan Weil
2012-03-05 13:05 ` [Qemu-devel] [Qemu-trivial] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Hajnoczi
2 siblings, 0 replies; 5+ messages in thread
From: Stefan Weil @ 2012-02-25 12:37 UTC (permalink / raw)
To: qemu-trivial; +Cc: Stefan Weil, qemu-devel
The 'struct' is not needed, and all other MemoryRegionOps don't use it.
Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
hw/sh7750.c | 2 +-
hw/sh_intc.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/sh7750.c b/hw/sh7750.c
index 4f4d8e7..e712928 100644
--- a/hw/sh7750.c
+++ b/hw/sh7750.c
@@ -712,7 +712,7 @@ static void sh7750_mmct_write(void *opaque, target_phys_addr_t addr,
}
}
-static const struct MemoryRegionOps sh7750_mmct_ops = {
+static const MemoryRegionOps sh7750_mmct_ops = {
.read = sh7750_mmct_read,
.write = sh7750_mmct_write,
.endianness = DEVICE_NATIVE_ENDIAN,
diff --git a/hw/sh_intc.c b/hw/sh_intc.c
index b24ec77..7d31ced 100644
--- a/hw/sh_intc.c
+++ b/hw/sh_intc.c
@@ -283,7 +283,7 @@ static void sh_intc_write(void *opaque, target_phys_addr_t offset,
#endif
}
-static const struct MemoryRegionOps sh_intc_ops = {
+static const MemoryRegionOps sh_intc_ops = {
.read = sh_intc_read,
.write = sh_intc_write,
.endianness = DEVICE_NATIVE_ENDIAN,
--
1.7.9
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops
2012-02-25 12:37 ` [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops Stefan Weil
@ 2012-02-25 23:16 ` Andreas Färber
0 siblings, 0 replies; 5+ messages in thread
From: Andreas Färber @ 2012-02-25 23:16 UTC (permalink / raw)
To: Stefan Weil; +Cc: qemu-trivial, qemu-devel, Alexander Graf
Am 25.02.2012 13:37, schrieb Stefan Weil:
> spin_rw_ops is only used in hw/ppce500_spin.c.
>
> Cc: Alexander Graf <agraf@suse.de>
> Signed-off-by: Stefan Weil <sw@weilnetz.de>
Acked-by: Andreas Färber <afaerber@suse.de>
/-F
> ---
> hw/ppce500_spin.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/hw/ppce500_spin.c b/hw/ppce500_spin.c
> index 6b8a189..6ed676b 100644
> --- a/hw/ppce500_spin.c
> +++ b/hw/ppce500_spin.c
> @@ -182,7 +182,7 @@ static uint64_t spin_read(void *opaque, target_phys_addr_t addr, unsigned len)
> }
> }
>
> -const MemoryRegionOps spin_rw_ops = {
> +static const MemoryRegionOps spin_rw_ops = {
> .read = spin_read,
> .write = spin_write,
> .endianness = DEVICE_BIG_ENDIAN,
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [Qemu-trivial] [PATCH 0/2] Trivial MemoryRegionOps patches
2012-02-25 12:37 [Qemu-devel] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Weil
2012-02-25 12:37 ` [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops Stefan Weil
2012-02-25 12:37 ` [Qemu-devel] [PATCH 2/2] sh7750: Remove redundant 'struct' from MemoryRegionOps Stefan Weil
@ 2012-03-05 13:05 ` Stefan Hajnoczi
2 siblings, 0 replies; 5+ messages in thread
From: Stefan Hajnoczi @ 2012-03-05 13:05 UTC (permalink / raw)
To: Stefan Weil; +Cc: qemu-trivial, qemu-devel
On Sat, Feb 25, 2012 at 01:37:11PM +0100, Stefan Weil wrote:
> Please apply http://patchwork.ozlabs.org/patch/139660/ to the qemu-trivial
> patch queue. Here are two additional trivial patches which also touch
> MemoryRegionOps and make the code more uniform.
>
> [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops
> [PATCH 2/2] sh7750: Remove redundant 'struct' from MemoryRegionOps
Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches
Stefan
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-03-05 13:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-02-25 12:37 [Qemu-devel] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Weil
2012-02-25 12:37 ` [Qemu-devel] [PATCH 1/2] ppc: Add missing 'static' to spin_rw_ops Stefan Weil
2012-02-25 23:16 ` Andreas Färber
2012-02-25 12:37 ` [Qemu-devel] [PATCH 2/2] sh7750: Remove redundant 'struct' from MemoryRegionOps Stefan Weil
2012-03-05 13:05 ` [Qemu-devel] [Qemu-trivial] [PATCH 0/2] Trivial MemoryRegionOps patches Stefan Hajnoczi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.