All of lore.kernel.org
 help / color / mirror / Atom feed
* [PULL 0/2] Trivial branch for 7.0 patches
@ 2022-03-18 13:43 Laurent Vivier
  2022-03-18 13:43 ` [PULL 1/2] hw/pci/pci.c: Fix typos of "Firewire", and of "controller" on same line Laurent Vivier
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Laurent Vivier @ 2022-03-18 13:43 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, Laurent Vivier

The following changes since commit 1d60bb4b14601e38ed17384277aa4c30c57925d3:

  Merge tag 'pull-request-2022-03-15v2' of https://gitlab.com/thuth/qemu into staging (2022-03-16 10:43:58 +0000)

are available in the Git repository at:

  https://gitlab.com/laurent_vivier/qemu.git tags/trivial-branch-for-7.0-pull-request

for you to fetch changes up to a2d860bb546aa766f6c28a47fabedafbfa191cc2:

  virtio/virtio-balloon: Prefer Object* over void* parameter (2022-03-18 13:57:50 +0100)

----------------------------------------------------------------
Trivial branch pull request 20220318

----------------------------------------------------------------

Bernhard Beschow (1):
  virtio/virtio-balloon: Prefer Object* over void* parameter

Rebecca Cran (1):
  hw/pci/pci.c: Fix typos of "Firewire", and of "controller" on same
    line

 hw/pci/pci.c               |  2 +-
 hw/virtio/virtio-balloon.c | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

-- 
2.35.1



^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PULL 1/2] hw/pci/pci.c: Fix typos of "Firewire", and of "controller" on same line
  2022-03-18 13:43 [PULL 0/2] Trivial branch for 7.0 patches Laurent Vivier
@ 2022-03-18 13:43 ` Laurent Vivier
  2022-03-18 13:43 ` [PULL 2/2] virtio/virtio-balloon: Prefer Object* over void* parameter Laurent Vivier
  2022-03-19 11:28 ` [PULL 0/2] Trivial branch for 7.0 patches Peter Maydell
  2 siblings, 0 replies; 4+ messages in thread
From: Laurent Vivier @ 2022-03-18 13:43 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, Peter Maydell, Rebecca Cran, Laurent Vivier

From: Rebecca Cran <quic_rcran@quicinc.com>

Signed-off-by: Rebecca Cran <quic_rcran@quicinc.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20220318101124.912-1-quic_rcran@quicinc.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
 hw/pci/pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/pci/pci.c b/hw/pci/pci.c
index 5cb1232e271d..dae9119bfe5f 100644
--- a/hw/pci/pci.c
+++ b/hw/pci/pci.c
@@ -1697,7 +1697,7 @@ static const pci_class_desc pci_class_descriptions[] =
     { 0x0902, "Mouse", "mouse"},
     { 0x0A00, "Dock station", "dock", 0x00ff},
     { 0x0B00, "i386 cpu", "cpu", 0x00ff},
-    { 0x0c00, "Fireware contorller", "fireware"},
+    { 0x0c00, "Firewire controller", "firewire"},
     { 0x0c01, "Access bus controller", "access-bus"},
     { 0x0c02, "SSA controller", "ssa"},
     { 0x0c03, "USB controller", "usb"},
-- 
2.35.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PULL 2/2] virtio/virtio-balloon: Prefer Object* over void* parameter
  2022-03-18 13:43 [PULL 0/2] Trivial branch for 7.0 patches Laurent Vivier
  2022-03-18 13:43 ` [PULL 1/2] hw/pci/pci.c: Fix typos of "Firewire", and of "controller" on same line Laurent Vivier
@ 2022-03-18 13:43 ` Laurent Vivier
  2022-03-19 11:28 ` [PULL 0/2] Trivial branch for 7.0 patches Peter Maydell
  2 siblings, 0 replies; 4+ messages in thread
From: Laurent Vivier @ 2022-03-18 13:43 UTC (permalink / raw)
  To: qemu-devel
  Cc: qemu-trivial, Michael S . Tsirkin, Bernhard Beschow,
	Laurent Vivier, David Hildenbrand

From: Bernhard Beschow <shentey@gmail.com>

*opaque is an alias to *obj. Using the ladder makes the code consistent with
with other devices, e.g. accel/kvm/kvm-all and accel/tcg/tcg-all. It also
makes the cast more typesafe.

Signed-off-by: Bernhard Beschow <shentey@gmail.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20220301222301.103821-2-shentey@gmail.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
 hw/virtio/virtio-balloon.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c
index e6c1b0aa46fe..163d244eb499 100644
--- a/hw/virtio/virtio-balloon.c
+++ b/hw/virtio/virtio-balloon.c
@@ -242,7 +242,7 @@ static void balloon_stats_get_all(Object *obj, Visitor *v, const char *name,
                                   void *opaque, Error **errp)
 {
     Error *err = NULL;
-    VirtIOBalloon *s = opaque;
+    VirtIOBalloon *s = VIRTIO_BALLOON(obj);
     int i;
 
     if (!visit_start_struct(v, name, NULL, 0, &err)) {
@@ -277,7 +277,7 @@ static void balloon_stats_get_poll_interval(Object *obj, Visitor *v,
                                             const char *name, void *opaque,
                                             Error **errp)
 {
-    VirtIOBalloon *s = opaque;
+    VirtIOBalloon *s = VIRTIO_BALLOON(obj);
     visit_type_int(v, name, &s->stats_poll_interval, errp);
 }
 
@@ -285,7 +285,7 @@ static void balloon_stats_set_poll_interval(Object *obj, Visitor *v,
                                             const char *name, void *opaque,
                                             Error **errp)
 {
-    VirtIOBalloon *s = opaque;
+    VirtIOBalloon *s = VIRTIO_BALLOON(obj);
     int64_t value;
 
     if (!visit_type_int(v, name, &value, errp)) {
@@ -1015,12 +1015,12 @@ static void virtio_balloon_instance_init(Object *obj)
     s->free_page_hint_notify.notify = virtio_balloon_free_page_hint_notify;
 
     object_property_add(obj, "guest-stats", "guest statistics",
-                        balloon_stats_get_all, NULL, NULL, s);
+                        balloon_stats_get_all, NULL, NULL, NULL);
 
     object_property_add(obj, "guest-stats-polling-interval", "int",
                         balloon_stats_get_poll_interval,
                         balloon_stats_set_poll_interval,
-                        NULL, s);
+                        NULL, NULL);
 }
 
 static const VMStateDescription vmstate_virtio_balloon = {
-- 
2.35.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PULL 0/2] Trivial branch for 7.0 patches
  2022-03-18 13:43 [PULL 0/2] Trivial branch for 7.0 patches Laurent Vivier
  2022-03-18 13:43 ` [PULL 1/2] hw/pci/pci.c: Fix typos of "Firewire", and of "controller" on same line Laurent Vivier
  2022-03-18 13:43 ` [PULL 2/2] virtio/virtio-balloon: Prefer Object* over void* parameter Laurent Vivier
@ 2022-03-19 11:28 ` Peter Maydell
  2 siblings, 0 replies; 4+ messages in thread
From: Peter Maydell @ 2022-03-19 11:28 UTC (permalink / raw)
  To: Laurent Vivier; +Cc: qemu-trivial, qemu-devel

On Fri, 18 Mar 2022 at 14:02, Laurent Vivier <laurent@vivier.eu> wrote:
>
> The following changes since commit 1d60bb4b14601e38ed17384277aa4c30c57925d3:
>
>   Merge tag 'pull-request-2022-03-15v2' of https://gitlab.com/thuth/qemu into staging (2022-03-16 10:43:58 +0000)
>
> are available in the Git repository at:
>
>   https://gitlab.com/laurent_vivier/qemu.git tags/trivial-branch-for-7.0-pull-request
>
> for you to fetch changes up to a2d860bb546aa766f6c28a47fabedafbfa191cc2:
>
>   virtio/virtio-balloon: Prefer Object* over void* parameter (2022-03-18 13:57:50 +0100)
>
> ----------------------------------------------------------------
> Trivial branch pull request 20220318


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/7.0
for any user-visible changes.

-- PMM


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-03-19 11:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-18 13:43 [PULL 0/2] Trivial branch for 7.0 patches Laurent Vivier
2022-03-18 13:43 ` [PULL 1/2] hw/pci/pci.c: Fix typos of "Firewire", and of "controller" on same line Laurent Vivier
2022-03-18 13:43 ` [PULL 2/2] virtio/virtio-balloon: Prefer Object* over void* parameter Laurent Vivier
2022-03-19 11:28 ` [PULL 0/2] Trivial branch for 7.0 patches Peter Maydell

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.